Actions
Bug #15572
closedReporting context is not re-evaluated when using multiple directives of the same technique
Status:
Released
Priority:
N/A
Assignee:
Category:
Web - Technique editor
Target version:
Pull Request:
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
Updated by Nicolas CHARLES over 5 years ago
- Status changed from New to In progress
- Assignee set to Nicolas CHARLES
Updated by Nicolas CHARLES over 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
Updated by Nicolas CHARLES over 5 years ago
- Status changed from Pending technical review to Pending release
Applied in changeset 830cf43cc07e377d8fcb934c75b1b498a1e052d2.
Updated by Vincent MEMBRÉ over 5 years ago
- Fix check changed from To do to Error - Blocking
Updated by Vincent MEMBRÉ over 5 years ago
- Fix check changed from Error - Blocking to To do
Updated by Nicolas CHARLES over 5 years ago
- Fix check changed from To do to Checked
Updated by Vincent MEMBRÉ over 5 years ago
This bug has been fixed in Rudder 5.0.13 which was released today.
Updated by Vincent MEMBRÉ over 5 years ago
- Status changed from Pending release to Released
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