Bug #8409
closedWhen a node send reports with an unknow configID, no expected reports are showned
Description
When a node sends reports with a bad config ID (for example, because someone changed the value of rudder_node_config_id to something that that node never had), then we get a VersionNotFound with the message "run received but no version in db", and no expected reports at all displayed above.
The correct behavior should be to:
- display a clear message explaining what is going wrong (better than the current one, at least with links toward documentation about how to debug/solve the situation)
- display the list of directive configured on the node but WITHOUT compliance, to make it clear that the reports received are not compatible with the expected ones. And so, the user know what directives are actually wanted on the node.
We also thought to:
- get the expected config id for that node,
- get expected reports for that config id,
- behave exactly like when we have a bad config id, with missing/unknown (perhaps it could be made better with a merge between expected reports / what we get + a big WARNING: the config ID IS NOT CORRECT, THERE IS A PROBLEM).
But the last part lead to quite hard to understand compliance tree (basically, if you don't know how the calculus are implemented, it's really hard to understand what you are seing)
Updated by François ARMAND over 8 years ago
- Status changed from New to In progress
Updated by Vincent MEMBRÉ over 8 years ago
- Target version changed from 3.0.17 to 2.11.23
Updated by Vincent MEMBRÉ over 8 years ago
- Target version changed from 2.11.23 to 302
Updated by Alexis Mousset over 8 years ago
- Target version changed from 302 to 3.1.12
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/1118
Updated by François ARMAND over 8 years ago
- Description updated (diff)
Corrected the description text and closed the pull request that doesn't match what we want.
Updated by François ARMAND over 8 years ago
- Status changed from Pending technical review to New
- Assignee deleted (
Vincent MEMBRÉ)
Updated by François ARMAND over 8 years ago
- Pull Request changed from https://github.com/Normation/rudder/pull/1118 to https://github.com/Normation/rudder/pull/1131
Updated by François ARMAND over 8 years ago
- Status changed from New to Pending release
- % Done changed from 0 to 100
Applied in changeset rudder|15db9f11bbd39456b8142d909605c351b74c8201.
Updated by Alexis Mousset over 8 years ago
- Status changed from Pending release to Released
This bug has been fixed in Rudder 3.1.12 and 3.2.5 which were released today.
- 3.1: Announce Changelog
- 3.2: Announce Changelog
- Download: https://www.rudder-project.org/site/get-rudder/downloads/