Architecture #11913
closedno reporting if value to report contains a variable
Description
Let's say you want your technique to edit a file /tmp/${bar} with a file_ensure_lines_present method where bar is either a variable definedby another method or a parameter of the technique (value defined in a Directive in 4.3). You will never get a report for this method since the variable won't have any value while expanding reporting file (expected_reports.csv)
We should comppletely change the way reporting is made on ncf techniques; by replacing the csv file system by creating a context where the reporting method will get the value of the current technique/directive/method.
Values extracted from CSV are:
- The technique name
- Component to report
- Value of the component to report
- The reporting identifier (aka the tracking key) composed of:
- THe Rule ID
- The Directive ID
Thus will largeky simplify logger_rudder method and remove a lot of magic.
Context needs to be set befor elogger method is called.
The techniqe needs to change the reproting context before each method call ( component to report, and it's value, and the technique name)
*
Rudder needs to set Directive/Rule context before calling the technique bundle.
Updated by Vincent MEMBRÉ almost 7 years ago
- Status changed from New to In progress
Updated by Vincent MEMBRÉ almost 7 years ago
- Status changed from In progress to Pending technical review
- Assignee changed from Vincent MEMBRÉ to Benoît PECCATTE
- Pull Request set to https://github.com/Normation/ncf/pull/674
Updated by Vincent MEMBRÉ almost 7 years ago
- Status changed from Pending technical review to Pending release
Applied in changeset f61dd73868c3cdfc81eadcd519157808747d8e25.
Updated by Nicolas CHARLES almost 7 years ago
- Related to Bug #12087: command_execution_result does not report anything added
Updated by Vincent MEMBRÉ almost 7 years ago
- Status changed from Pending release to Released
This bug has been fixed in Rudder 4.3.0~beta1 which was released today.
- 4.3.0~beta1: Announce Changelog
- Download: https://www.rudder-project.org/site/get-rudder/downloads/
Updated by Vincent MEMBRÉ almost 7 years ago
- Related to Architecture #11915: Remove expected reports.csv logic added
Updated by Alexis Mousset over 2 years ago
- Project changed from 41 to Rudder
- Category changed from Technique editor - API to Web - Technique editor