Project

General

Profile

Actions

Bug #15572

closed

Reporting context is not re-evaluated when using multiple directives of the same technique

Added by Félix DALLIDET about 5 years ago. Updated over 2 years ago.

Status:
Released
Priority:
N/A
Category:
Web - Technique editor
Target version:
Severity:
UX impact:
User visibility:
Effort required:
Priority:
0
Name check:
To do
Fix check:
Checked
Regression:

Description

I ended up with missing reports while defining quite simple configuration in the technique editor.
The faulty technique is attached to the ticket.

PB:
When defining multiple directives from this technique only the first instance executed was correctly reporting, all other directives
were reporting wrongly on the "Condition from variable existence" with a component_name set to "None".

This is due to the promiser of _method_reporting_context which was:

Condition from variable existence_context_0

Which is not re-evaluated at each bundle call since only depending on static values: the 0 for the promiser and the class_parameter for the parameters which is common to all directives.
To avoid this, we should keep the incremental suffix but also add an arbitrary value in the promiser, depending on the directive we are executing. For instance, the promiser id.

Which fixed the problem with a promiser like:

"Condition from variable existence_context_${report_data.directive_id}_0" 


Files

Disable kernel module disabled.json (1.12 KB) Disable kernel module disabled.json Félix DALLIDET, 2019-08-23 15:39

Subtasks 2 (0 open2 closed)

Bug #15573: Same a parent ticket for branch 5.1ReleasedVincent MEMBRÉActions
Bug #15576: Don't parse _${report_data.directive} as report componentReleasedNicolas CHARLESActions
Actions #1

Updated by Nicolas CHARLES about 5 years ago

  • Status changed from New to In progress
  • Assignee set to Nicolas CHARLES
Actions #2

Updated by Nicolas CHARLES about 5 years ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from Nicolas CHARLES to Vincent MEMBRÉ
  • Pull Request set to https://github.com/Normation/ncf/pull/1049
Actions #3

Updated by Nicolas CHARLES about 5 years ago

  • Status changed from Pending technical review to Pending release

Applied in changeset commit:830cf43cc07e377d8fcb934c75b1b498a1e052d2.

Actions #4

Updated by Vincent MEMBRÉ about 5 years ago

  • Fix check changed from To do to Error - Blocking
Actions #5

Updated by Vincent MEMBRÉ about 5 years ago

  • Fix check changed from Error - Blocking to To do
Actions #6

Updated by Nicolas CHARLES about 5 years ago

  • Fix check changed from To do to Checked
Actions #7

Updated by Vincent MEMBRÉ about 5 years ago

This bug has been fixed in Rudder 5.0.13 which was released today.

Actions #8

Updated by Vincent MEMBRÉ about 5 years ago

  • Status changed from Pending release to Released
Actions #9

Updated by Alexis Mousset over 2 years ago

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

Also available in: Atom PDF