User story #2596: Remove/update inventory promises - Rudder - Issue Tracker

Actions

Copy link

User story #2596

closed

Remove/update inventory promises

Added by Vincent MEMBRÉ over 12 years ago. Updated over 5 years ago.

Status:

Rejected

Priority:

Assignee:

Category:

System techniques

Target version:

Ideas (not version specific)

Pull Request:

UX impact:

Suggestion strength:

User visibility:

Effort required:

Name check:

Fix check:

Regression:

Description

As we got new informations from inventories in 2.5 we don't need to add some informations anymore.

we should remove the promises adding virtual machines informations

Depending on the version used for inventory, we should not add cpu informations anymore.
we may check the fusion inventory version used before computing those informations
( using fusioninventory-agent --version returns the version used to make inventories)

The inventory parser is already capable treating case where the informations is provided but useless,
but it will less things to process for the agent

Actions

Copy link

Updated by Jonathan CLARKE over 12 years ago

Status changed from New to Discussion
Assignee changed from Jonathan CLARKE to Vincent MEMBRÉ
Priority changed from N/A to 3

Vincent MEMBRÉ wrote:

As we got new informations from inventories in 2.5 we don't need to add some informations anymore.

we should remove the promises adding virtual machines informations

Depending on the version used for inventory, we should not add cpu informations anymore.
we may check the fusion inventory version used before computing those informations
( using fusioninventory-agent --version returns the version used to make inventories)

Above which version exactly should we not include CPU information?

Actions

Copy link

Updated by Vincent MEMBRÉ over 12 years ago

Since version 2.2.1, cpu informations are complete in fusion

Actions

Copy link

Updated by Vincent MEMBRÉ over 12 years ago

We also need to remove Policies informations, they are now useless in 2.5

Actions

Copy link

Updated by Vincent MEMBRÉ over 12 years ago

Assignee changed from Vincent MEMBRÉ to Jonathan CLARKE

Actions

Copy link

Updated by Jonathan CLARKE over 12 years ago

Target version changed from 47 to 50

Actions

Copy link

Updated by Jonathan CLARKE over 12 years ago

Target version changed from 50 to 2.4.0~beta3

Actions

Copy link

Updated by Nicolas CHARLES over 12 years ago

The list of techniques has been removed, but not the virtual machine nor the Cpu informations. should they be ?

Actions

Copy link

Updated by Vincent MEMBRÉ over 12 years ago

virtual machines could be removed (due to the use of option --scan-homedirs in fusion)

for cpu informations we should match if version on fusion inventory used is >= 2.2.1.
If not then run cpu informations

Actions

Copy link

Updated by Nicolas PERRON over 12 years ago

Nicolas CHARLES wrote:

The list of techniques has been removed, but not the virtual machine nor the Cpu informations. should they be ?

I don't see any commit about it, how could it be possible ?

Actions

Copy link

#10

Updated by Vincent MEMBRÉ over 12 years ago

Target version changed from 2.4.0~beta3 to 2.4.0~beta4

Using old promises is not a problem as inventory endpoint will handle the case where useless content has been added, so it's not blocking for 2.4.0 beta3 but i think we have to handle it before release of 2.4.0 (need to check fusion inventory version before execution promises / remove useless promises)

then we can change it to beta4

Actions

Copy link

#11