Project

General

Profile

Actions

Bug #18952

closed

Static group which depends on dynamic subgroup must be forbiden

Added by François ARMAND about 3 years ago. Updated 3 months 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
Actions #1

Updated by Nicolas CHARLES about 3 years ago

  • 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
Actions #2

Updated by Vincent MEMBRÉ about 3 years ago

  • Target version changed from 6.1.10 to 6.1.11
Actions #3

Updated by Vincent MEMBRÉ about 3 years ago

  • Target version changed from 6.1.11 to 6.1.12
Actions #4

Updated by Vincent MEMBRÉ about 3 years ago

  • Target version changed from 6.1.12 to 6.1.13
Actions #5

Updated by Vincent MEMBRÉ almost 3 years ago

  • Target version changed from 6.1.13 to 6.1.14
Actions #6

Updated by Vincent MEMBRÉ almost 3 years ago

  • Target version changed from 6.1.14 to 6.1.15
Actions #7

Updated by Vincent MEMBRÉ almost 3 years ago

  • Target version changed from 6.1.15 to 6.1.16
Actions #8

Updated by Vincent MEMBRÉ over 2 years ago

  • Target version changed from 6.1.16 to 6.1.17
Actions #9

Updated by Vincent MEMBRÉ over 2 years ago

  • Target version changed from 6.1.17 to 6.1.18
Actions #10

Updated by Vincent MEMBRÉ over 2 years ago

  • Target version changed from 6.1.18 to 6.1.19
Actions #11

Updated by Vincent MEMBRÉ about 2 years ago

  • Target version changed from 6.1.19 to 6.1.20
Actions #12

Updated by Vincent MEMBRÉ almost 2 years ago

  • Target version changed from 6.1.20 to 6.1.21
Actions #13

Updated by Vincent MEMBRÉ almost 2 years ago

  • Target version changed from 6.1.21 to old 6.1 issues to relocate
Actions #14

Updated by François ARMAND 5 months ago

  • Target version changed from old 6.1 issues to relocate to 7.3.10
Actions #15

Updated by Vincent MEMBRÉ 4 months ago

  • Target version changed from 7.3.10 to 7.3.11
Actions #16

Updated by François ARMAND 4 months ago

  • Status changed from New to In progress
Actions #17

Updated by François ARMAND 4 months ago

  • 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
Actions #18

Updated by Anonymous 4 months ago

  • Status changed from Pending technical review to Pending release

Updated by François ARMAND 3 months ago

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

Actions #20

Updated by François ARMAND 3 months ago

  • Subtask #24086 added
Actions #21

Updated by Alexis Mousset 3 months ago

  • Fix check changed from Error - Blocking to Error - Fixed
Actions #22

Updated by Vincent MEMBRÉ 3 months ago

  • 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: Atom PDF