Project

General

Profile

Actions

Architecture #25092

closed

Architecture #24963: Persist compliance in base to know last state for a long time

Refactor CachedReportingService to make persistance simpler

Added by François ARMAND 5 months ago. Updated 4 months ago.

Status:
Released
Priority:
N/A
Category:
Architecture - Code maintenance
Target version:
Effort required:
Name check:
To do
Fix check:
To do
Regression:
No

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.

Actions

Also available in: Atom PDF