Project

General

Profile

Bug #8739

group management technique doesn't ensure optional user is in group

Added by Florian Heigl about 2 years ago. Updated 10 months ago.

Status:
Released
Priority:
N/A
Category:
Techniques
Target version:
Target version (plugin):
Severity:
Major - prevents use of part of Rudder | no simple workaround
User visibility:
Getting started - demo | first install | level 1 Techniques
Effort required:
Priority:
79
Tags:

Description

if the group already exists
"option Users included in the group (optional)"
doesn't work UNLESS you also select enforce the content of the group

groups.png (103 KB) groups.png Florian Heigl, 2016-07-21 22:47
1039

Associated revisions

Revision 4c0728ec (diff)
Added by Alexis MOUSSET 10 months ago

Fixes #8739: group management technique doesn't ensure optional user is in group

History

#1 Updated by Alexis MOUSSET about 2 years ago

  • Target version set to 2.11.24

Indeed, the "Enforce the group content" boolean is used as a condition to edit the group content.

We should have:
  • an option to append the users list to the group
  • an option to enforce only these users.

We should also rename the body named "groups_file_append" that does not append users but set the whole content of the group.

#2 Updated by Vincent MEMBRÉ almost 2 years ago

  • Target version changed from 2.11.24 to 308

#3 Updated by Vincent MEMBRÉ almost 2 years ago

  • Target version changed from 308 to 3.1.14

#4 Updated by Vincent MEMBRÉ almost 2 years ago

  • Target version changed from 3.1.14 to 3.1.15

#5 Updated by Vincent MEMBRÉ almost 2 years ago

  • Target version changed from 3.1.15 to 3.1.16

#6 Updated by Vincent MEMBRÉ almost 2 years ago

  • Target version changed from 3.1.16 to 3.1.17

#7 Updated by Vincent MEMBRÉ over 1 year ago

  • Target version changed from 3.1.17 to 3.1.18

#8 Updated by Vincent MEMBRÉ over 1 year ago

  • Target version changed from 3.1.18 to 3.1.19

#9 Updated by Benoît PECCATTE over 1 year ago

  • Severity set to Major - prevents use of part of Rudder | no simple workaround
  • User visibility set to Operational - other Techniques | Technique editor | Rudder settings
  • Priority set to 0

#10 Updated by François ARMAND over 1 year ago

  • Severity changed from Major - prevents use of part of Rudder | no simple workaround to Minor - inconvenience | misleading | easy workaround
  • User visibility changed from Operational - other Techniques | Technique editor | Rudder settings to Getting started - demo | first install | level 1 Techniques
  • Priority changed from 0 to 31

The behavior is the expected one, but the label for the checkbox must be explicit about what it does (obviously, it is not).

#11 Updated by Vincent MEMBRÉ over 1 year ago

  • Target version changed from 3.1.19 to 3.1.20

#12 Updated by Jonathan CLARKE over 1 year ago

  • Priority changed from 31 to 30

#13 Updated by Vincent MEMBRÉ over 1 year ago

  • Target version changed from 3.1.20 to 3.1.21

#14 Updated by Vincent MEMBRÉ about 1 year ago

  • Target version changed from 3.1.21 to 3.1.22

#15 Updated by Benoît PECCATTE about 1 year ago

  • Priority changed from 30 to 29

#16 Updated by Benoît PECCATTE about 1 year ago

  • Priority changed from 29 to 43

#17 Updated by Vincent MEMBRÉ about 1 year ago

  • Target version changed from 3.1.22 to 3.1.23
  • Priority changed from 43 to 42

#18 Updated by Vincent MEMBRÉ 12 months ago

  • Target version changed from 3.1.23 to 3.1.24

#19 Updated by François ARMAND 10 months ago

  • Tags set to Sponsored
  • Severity changed from Minor - inconvenience | misleading | easy workaround to Major - prevents use of part of Rudder | no simple workaround
  • Priority changed from 42 to 79

There is in fact no easy workaround to be able to ensure that an user is in a group.

#20 Updated by Alexis MOUSSET 10 months ago

  • Status changed from New to In progress
  • Assignee set to Alexis MOUSSET

#21 Updated by Alexis MOUSSET 10 months ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from Alexis MOUSSET to Nicolas CHARLES
  • Pull Request set to https://github.com/Normation/rudder-techniques/pull/1205

#22 Updated by Alexis MOUSSET 10 months ago

  • Status changed from Pending technical review to Pending release

#23 Updated by Vincent MEMBRÉ 10 months ago

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 3.1.24, 4.1.8 and 4.2.1 which were released today.

Also available in: Atom PDF