Project

General

Profile

Actions

Architecture #11913

closed

no reporting if value to report contains a variable

Added by Vincent MEMBRÉ almost 7 years ago. Updated over 2 years ago.

Status:
Released
Priority:
N/A
Category:
Web - Technique editor
Target version:
Effort required:
Name check:
Fix check:
Regression:

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.


Subtasks 14 (0 open14 closed)

Architecture #11914: Add Rudder reporting context before calling technique bundleReleasedVincent MEMBRÉActions
Architecture #11929: Remove calls related to expected resports csv files from rudder techniquesReleasedBenoît PECCATTEActions
Architecture #11930: Add a way to prevent reporting of methodsReleasedBenoît PECCATTEActions
Architecture #12172: Prevent reporting of inner methods in Rudder ReleasedBenoît PECCATTEActions
Architecture #12180: typo in new loggerReleasedBenoît PECCATTEActions
Architecture #12181: typo in new loggerReleasedBenoît PECCATTEActions
Architecture #12185: Broken policies after changes in the loggerRejectedActions
Architecture #11939: Adapt associated testsReleasedNicolas CHARLESActions
Bug #12035: ncf does not report on the correct component name in Rudder 4.3ReleasedNicolas CHARLESActions
Architecture #12044: Add a script to rewrite a technique from command lineReleasedVincent MEMBRÉActions
Bug #12078: ncf techniques are not reporting on the correct technique nameReleasedBenoît PECCATTEActions
Bug #12079: Add information about technique name/id to directive contextReleasedBenoît PECCATTEActions
Bug #12085: Add information about technique name/id to directive contextReleasedBenoît PECCATTEActions
Bug #12084: Broken tests after parent changeReleasedVincent MEMBRÉActions

Related issues 2 (0 open2 closed)

Related to Rudder - Bug #12087: command_execution_result does not report anythingReleasedBenoît PECCATTEActions
Related to Rudder - Architecture #11915: Remove expected reports.csv logicReleasedVincent MEMBRÉActions
Actions #1

Updated by Vincent MEMBRÉ almost 7 years ago

  • Status changed from New to In progress
Actions #2

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
Actions #3

Updated by Vincent MEMBRÉ almost 7 years ago

  • Status changed from Pending technical review to Pending release

Applied in changeset commit:f61dd73868c3cdfc81eadcd519157808747d8e25.

Actions #4

Updated by Nicolas CHARLES almost 7 years ago

  • Related to Bug #12087: command_execution_result does not report anything added
Actions #5

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.

Actions #6

Updated by Vincent MEMBRÉ over 6 years ago

Actions #7

Updated by Alexis Mousset over 2 years ago

  • Project changed from 41 to Rudder
  • Category changed from Technique editor - API to Web - Technique editor
Actions

Also available in: Atom PDF