Bug #3555
closedClass name collision 'cant_send_inventory' between inventory and distributePolicy Techniques
Description
On a fresh install of Rudder server 2.6.0~rc1, I'm seeing these non-compliance errors in /var/log/rudder/compliance/non-compliant-reports.log:
[2013-04-28 18:30:39+0200] N: root [server] S: [result_error] R: root-DP [distributePolicy] D: root-distributePolicy [Distribute Policy] T: distributePolicy/1.0 C: [Send inventories to CMDB] V: [None] Some inventories failed to add successfully to Rudder
However, when I look in /var/rudder/inventories/incoming and /var/rudder/inventories/accepted-nodes-updates/, there are no inventories waiting. This non-compliance log is generated in the distributePolicy/1.0/propagatePromises.cf file, based on the "cant_send_inventory::" class.
After some investigating, I found this when running cf-agent -v:
rudder> -> Additional classes = { cant_send_inventory inventory_sent policy_server root_server site_ok }
This means that the 'cant_send_inventory' class is defined persistently. However, this doesn't come from distributePolicy, but from the inventory Technique, which uses the exact same class name. This should be fine, as it should be cancelled by the inventory Technique when it succeeds (which it has), but for some reason it is not.
An easy fix here is just to use another class name.
Updated by Jonathan CLARKE over 11 years ago
- Target version changed from 2.6.0 to 2.4.5
This can happen since the server can send it's own inventory, so in Rudder 2.4.
Updated by Jonathan CLARKE over 11 years ago
- Status changed from In progress to Pending technical review
- Assignee changed from Jonathan CLARKE to Matthieu CERDA
- Pull Request set to https://github.com/Normation/rudder-techniques/pull/108
Updated by Jonathan CLARKE over 11 years ago
Pull Request ready for review: https://github.com/Normation/rudder-techniques/pull/108
Updated by Matthieu CERDA over 11 years ago
- Assignee changed from Matthieu CERDA to Jonathan CLARKE
- Pull Request changed from https://github.com/Normation/rudder-techniques/pull/108 to https://github.com/Normation/rudder-techniques/pull/109
Pull request modified (commit author changed): https://github.com/Normation/rudder-techniques/pull/109
Updated by Jonathan CLARKE over 11 years ago
- Status changed from Pending technical review to Pending release
- % Done changed from 0 to 100
Applied in changeset policy-templates:commit:4dbe7bdfbec5f3e204c60fa950458411e8487897.
Updated by Matthieu CERDA over 11 years ago
Applied in changeset policy-templates:commit:dd075865da48beb8da0238658cfe682f9a00af9b.
Updated by Nicolas PERRON over 11 years ago
- Status changed from Pending release to Released