Architecture #25092
closedArchitecture #24963: Persist compliance in base to know last state for a long time
Refactor CachedReportingService to make persistance simpler
Description
CachedReportingService is very complicated and mix at least four function (getting runs & expected config, computing compliance and checking compliance, having a cache, and higher level utility methods on NodeStatusReport).
We want to split these different part to have a clear NodeStatusReportRepository
where to add persistance, and a clear ComputeNodeStatusReports
where to modify expiration checks based on new parameter.
This first step should not modify too much current implementation, just simplify things where it's obvious (unused parameters, etc).
It's also time to get ride of the all-commented ReportingServiceTest
, so that we can build a simpler, better suite afterward.
Updated by François ARMAND 6 months ago
- Status changed from New to In progress
- Assignee set to François ARMAND
Updated by François ARMAND 6 months ago
- Status changed from In progress to Pending technical review
- Assignee changed from François ARMAND to Clark ANDRIANASOLO
- Pull Request set to https://github.com/Normation/rudder/pull/5757
Updated by Anonymous 6 months ago
- Status changed from Pending technical review to Pending release
Applied in changeset rudder|61ee46399c33eabb7000b12a53443b3cabfe06c0.
Updated by Alexis Mousset 5 months ago
- Status changed from Pending release to Released
This bug has been fixed in Rudder 8.2.0~alpha1 which was released today.