Bug #7625
closedNot accepted node reports trigger compliance processing
Description
When we are receving new reports for an agent run, we are invalidating compliance for that node and calculating it back. Problem: the process is triggered for any node, even pendind, deleted, or even completly unknown nodes.
This waste resources for nothing, and so only accepted node should trigger the compliance calcul.
For history, I'm letting the old bug description:
For some reason, it seems that pending nodes are taken into consideration when calculating compliance.
So we get message like:
WARN explain_compliance.1e0ab778-0a9e-4d44-902b-60842ed09120 - Node '1e0ab778-0a9e-4d44-902b-60842ed09120' sent reports for run at 'VersionNotFound(2015-12-16T00:16:34.000Z,None)' (with no configuration ID). No expected configuration matches these reports.
They should not, because either it is blattlantly false (we are using nodes that are clearly NOT in any rule to calculate the compliance of the rule), or a waste of time (we are spending time calculating things for nodes, and we are sure that these things won't be used).
Updated by Jonathan CLARKE almost 9 years ago
- Translation missing: en.field_tag_list set to Quick and important
Updated by François ARMAND over 8 years ago
- Subject changed from Pending Nodes are considered for compliance to Not accepted node reports trigger compliance processing
- Description updated (diff)
- Status changed from New to In progress
Changing bug text now that we know it has nothing to do with pending nodes.
Updated by François ARMAND over 8 years ago
- Status changed from In progress to Pending technical review
- Assignee changed from François ARMAND to Vincent MEMBRÉ
- Pull Request set to https://github.com/Normation/rudder/pull/1025
Updated by François ARMAND over 8 years ago
- Status changed from Pending technical review to Pending release
- % Done changed from 0 to 100
Applied in changeset rudder|73bdd79346f9941ff5b7434a6bc3d1f92ece16bc.
Updated by Vincent MEMBRÉ over 8 years ago
- Status changed from Pending release to Released