Bug #26255: Reporting on user management technique is broken when multiple users are defined in a directive - Rudder - Issue Tracker

Actions

Copy link

Bug #26255

closed

Reporting on user management technique is broken when multiple users are defined in a directive

Added by Nicolas CHARLES about 2 months ago. Updated 28 days ago.

Status:

Released

Priority:

1 (highest)

Assignee:

Alexis Mousset

Category:

Techniques

Target version:

8.1.12

Pull Request:

https://github.com/Normation/rudder-t...

Severity:

UX impact:

User visibility:

Effort required:

Priority:

Name check:

To do

Fix check:

To do

Regression:

Description

if we try to manage 2 users in a same directive, the reporting is broken, because of this code

      "any" usebundle => _method_reporting_context("Users", "${usergroup_user_login[${usergroup_user_index}]}");
      "${usergroup_user_login[${usergroup_user_index}]}" 
            usebundle => user_absent("${usergroup_user_login[${usergroup_user_index}]}"),
                   if => "usermanagement_user_remove_${usergroup_user_index}";

method_reporting_context iterate over all user, and get the last value
then the generic method reports with the last entry defined

happens with technique v9.1 and V10

Subtasks 1 (0 open — 1 closed)

Related issues 1 (1 open — 0 closed)

Actions

Copy link

Updated by Nicolas CHARLES about 2 months ago

Assignee set to Alexis Mousset
Priority changed from To review to 1 (highest)

The only approach would be to wrap the call, like what we do with the technique editor
psudo code would be

methods:
"remove_that_user" usebundle => bundle agent user_absent_&UUID&("${usergroup_user_index}")

(snip)

bundle agent user_absent_&UUID&(usergroup_user_index)
      "any" usebundle => _method_reporting_context("Users", "${usermanagement_&uuid&.usergroup_user_login[${usergroup_user_index}]}");
      "${usergroup_user_login[${usergroup_user_index}]}" 
            usebundle => user_absent("${usermanagement_&uuid&.usergroup_user_login[${usergroup_user_index}]}"),
                   if => "usermanagement_user_remove_${usergroup_user_index}";

Actions

Copy link