Project

General

Profile

Actions

Bug #3555

closed

Class name collision 'cant_send_inventory' between inventory and distributePolicy Techniques

Added by Jonathan CLARKE almost 11 years ago. Updated almost 11 years ago.

Status:
Released
Priority:
1
Category:
System techniques
Target version:
Severity:
UX impact:
User visibility:
Effort required:
Priority:
Name check:
Fix check:
Regression:

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.

Actions #1

Updated by Jonathan CLARKE almost 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.

Actions #2

Updated by Jonathan CLARKE almost 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
Actions #4

Updated by Matthieu CERDA almost 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

Actions #5

Updated by Jonathan CLARKE almost 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.

Actions #6

Updated by Matthieu CERDA almost 11 years ago

Applied in changeset policy-templates:commit:dd075865da48beb8da0238658cfe682f9a00af9b.

Actions #7

Updated by Nicolas PERRON almost 11 years ago

  • Status changed from Pending release to Released
Actions

Also available in: Atom PDF