Project

General

Profile

Actions

Bug #18952

closed

Static group which depends on dynamic subgroup must be forbiden

Bug #18952: Static group which depends on dynamic subgroup must be forbiden

Added by François ARMAND almost 5 years ago. Updated about 2 years ago.

Status:
Released
Priority:
N/A
Category:
Web - Config management
Target version:
Severity:
UX impact:
User visibility:
Effort required:
Priority:
0
Name check:
To do
Fix check:
Error - Fixed
Regression:
No

Description

We need to ensure consistency on a group and its subgroup.

Consistency seems to be: a group with subgroup query and that group MUST BE dynamic, other cases seems to be ok:

- dyn parent / dyn subgroup => OK
- dyn parent / static subgroup => OK (subgroup just never change, or if it change with a new save, parent is computed again)
- static parent / static subgroup => actually, perhaps not good, because if a subgroup is changed, parent won't be recalculated (which would mean that actually, the parent group will be dynamic)
- static parent / dyn subgroup => breaks.

The idea is to forbid saving group as static with subgroup with a helpful message + have an other check during policy generation (and breaks ?)


Files


Subtasks 1 (0 open1 closed)

Bug #24086: You can create a static group which depends on a dynamic oneReleasedClark ANDRIANASOLOActions

Related issues 1 (1 open0 closed)

Related to Rudder - Bug #18942: Non understandable policy generation error with dependant groups when a static depends on a dynamic and are out of syncNewActions

Updated by Nicolas CHARLES almost 5 years ago Actions #1

  • Related to Bug #18942: Non understandable policy generation error with dependant groups when a static depends on a dynamic and are out of sync added

Updated by Vincent MEMBRÉ almost 5 years ago Actions #2

  • Target version changed from 6.1.10 to 6.1.11

Updated by Vincent MEMBRÉ almost 5 years ago Actions #3

  • Target version changed from 6.1.11 to 6.1.12

Updated by Vincent MEMBRÉ almost 5 years ago Actions #4

  • Target version changed from 6.1.12 to 6.1.13

Updated by Vincent MEMBRÉ over 4 years ago Actions #5

  • Target version changed from 6.1.13 to 6.1.14

Updated by Vincent MEMBRÉ over 4 years ago Actions #6

  • Target version changed from 6.1.14 to 6.1.15

Updated by Vincent MEMBRÉ over 4 years ago Actions #7

  • Target version changed from 6.1.15 to 6.1.16

Updated by Vincent MEMBRÉ over 4 years ago Actions #8

  • Target version changed from 6.1.16 to 6.1.17

Updated by Vincent MEMBRÉ over 4 years ago Actions #9

  • Target version changed from 6.1.17 to 6.1.18

Updated by Vincent MEMBRÉ about 4 years ago Actions #10

  • Target version changed from 6.1.18 to 6.1.19

Updated by Vincent MEMBRÉ almost 4 years ago Actions #11

  • Target version changed from 6.1.19 to 6.1.20

Updated by Vincent MEMBRÉ over 3 years ago Actions #12

  • Target version changed from 6.1.20 to 6.1.21

Updated by Vincent MEMBRÉ over 3 years ago Actions #13

  • Target version changed from 6.1.21 to old 6.1 issues to relocate

Updated by François ARMAND about 2 years ago Actions #14

  • Target version changed from old 6.1 issues to relocate to 7.3.10

Updated by Vincent MEMBRÉ about 2 years ago Actions #15

  • Target version changed from 7.3.10 to 7.3.11

Updated by François ARMAND about 2 years ago Actions #16

  • Status changed from New to In progress

Updated by François ARMAND about 2 years ago Actions #17

  • Status changed from In progress to Pending technical review
  • Assignee changed from François ARMAND to Vincent MEMBRÉ
  • Pull Request set to https://github.com/Normation/rudder/pull/5286

Updated by Anonymous about 2 years ago Actions #18

  • Status changed from Pending technical review to Pending release

Updated by François ARMAND about 2 years ago Actions #19

clipboard-202401251803-z4nfh.png
clipboard-202401251804-vor3k.png

The case:
- changing a sub group of a static group from static to dynamic works well:

- the case: forbid using a dynamic sub-group in a static one does not

Updated by François ARMAND about 2 years ago Actions #20

  • Subtask #24086 added

Updated by Alexis Mousset about 2 years ago Actions #21

  • Fix check changed from Error - Blocking to Error - Fixed

Updated by Vincent MEMBRÉ about 2 years ago Actions #22

  • Status changed from Pending release to Released

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

Actions

Also available in: PDF Atom