Architecture #19127
openOverridden directives in the same rule are missing before a full regeneration
Description
So, in #19114, we had missing directive because of a bug.
Now, we have the correct behavior, but only after a full regeneration if the added directive doesn't change actual configuration.
This is because is the added directive is skipped everywhere, the effective configuration doesn't change on any nodes, so the first step of policy generation that looks if it is necessary to compute new generation say "no, it isn't", then we don't have new expected configuration, and it's expected configuration that are used to check if directives are skipped.
That's all because we don't really apply directives to rule, we apply directives to nodes (with a lot of graphe resolution in the middle, involving rules and groups).
Correcting that one would mean that we need to change expected configuration in a case when it's actually not needed (because, well, they don't change) and would lead to poetentially long computation for nothing. I'm not sure about what to do.
Updated by François ARMAND over 3 years ago
- Related to Bug #19114: Overridden directives in the same rule are missing (not even "skipped") added
Updated by Vincent MEMBRÉ over 3 years ago
- Target version changed from 6.2.5 to 6.2.6
Updated by Vincent MEMBRÉ over 3 years ago
- Target version changed from 6.2.6 to 6.2.7
Updated by Vincent MEMBRÉ over 3 years ago
- Target version changed from 6.2.7 to 6.2.8
Updated by Vincent MEMBRÉ over 3 years ago
- Target version changed from 6.2.8 to 6.2.9
Updated by Vincent MEMBRÉ over 3 years ago
- Target version changed from 6.2.9 to 6.2.10
Updated by Vincent MEMBRÉ about 3 years ago
- Target version changed from 6.2.10 to 6.2.11
Updated by Vincent MEMBRÉ about 3 years ago
- Target version changed from 6.2.11 to 6.2.12
Updated by Vincent MEMBRÉ almost 3 years ago
- Target version changed from 6.2.12 to 6.2.13
Updated by François ARMAND almost 3 years ago
- Tracker changed from Bug to Architecture
- Priority deleted (
0)
Updated by Vincent MEMBRÉ over 2 years ago
- Target version changed from 6.2.13 to 6.2.14
Updated by Vincent MEMBRÉ over 2 years ago
- Target version changed from 6.2.14 to 6.2.15
Updated by Vincent MEMBRÉ over 2 years ago
- Target version changed from 6.2.15 to 6.2.16
Updated by Alexis Mousset over 2 years ago
- Target version changed from 6.2.16 to 6.2.17
Updated by Vincent MEMBRÉ about 2 years ago
- Target version changed from 6.2.17 to 997
Updated by Vincent MEMBRÉ about 2 years ago
- Target version changed from 997 to 6.2.18
Updated by Vincent MEMBRÉ about 2 years ago
- Target version changed from 6.2.18 to 6.2.19
Updated by Vincent MEMBRÉ about 2 years ago
- Target version changed from 6.2.19 to 6.2.20
Updated by Vincent MEMBRÉ about 2 years ago
- Target version changed from 6.2.20 to old 6.2 issues to relocate
Updated by Alexis Mousset over 1 year ago
- Target version changed from old 6.2 issues to relocate to 7.2.11
- Regression set to No
Updated by Vincent MEMBRÉ about 1 year ago
- Target version changed from 7.2.11 to 1046
Updated by Alexis Mousset about 1 year ago
- Target version changed from 1046 to Ideas (not version specific)