Project

General

Profile

Actions

Bug #22525

closed

Directives applied twice don't show in rule details (they should be skipped)

Added by Nicolas CHARLES about 1 year ago. Updated about 2 months ago.

Status:
Released
Priority:
N/A
Category:
Web - Compliance & node report
Target version:
Severity:
Minor - inconvenience | misleading | easy workaround
UX impact:
It bothers me each time
User visibility:
Operational - other Techniques | Rudder settings | Plugins
Effort required:
Priority:
84
Name check:
To do
Fix check:
Checked
Regression:
Yes

Description

but it does appears as skipped on the node compliance page

in the rule page: per directive nothing

per node nothing


Files

clipboard-202303231725-lmhyd.png (64.2 KB) clipboard-202303231725-lmhyd.png Nicolas CHARLES, 2023-03-23 17:25
clipboard-202303231727-slfpe.png (64.7 KB) clipboard-202303231727-slfpe.png Nicolas CHARLES, 2023-03-23 17:27
clipboard-202303231727-6tyrs.png (37.4 KB) clipboard-202303231727-6tyrs.png Nicolas CHARLES, 2023-03-23 17:27

Related issues 4 (1 open3 closed)

Related to Rudder - Bug #18931: No skipped directives on Rule screen after parent ticketReleasedNicolas CHARLESActions
Related to Rudder - Bug #15125: Rule shown as "Mixed" whereas applied to one audit node and one (skipped) enforce directiveNewActions
Related to Rudder - Architecture #23656: Rewrite Nodes compliance table in ElmReleasedVincent MEMBRÉActions
Related to Rudder - Bug #24097: Skipped directive does not show in node compliance treeReleasedFrançois ARMANDActions
Actions #1

Updated by Vincent MEMBRÉ about 1 year ago

  • Target version changed from 7.2.6 to 7.2.7
Actions #2

Updated by Vincent MEMBRÉ about 1 year ago

  • Target version changed from 7.2.7 to 7.2.8
Actions #3

Updated by Nicolas CHARLES about 1 year ago

  • Regression changed from No to Yes
Actions #4

Updated by François ARMAND about 1 year ago

  • Severity set to Minor - inconvenience | misleading | easy workaround
  • UX impact set to It bothers me each time
  • User visibility set to Operational - other Techniques | Rudder settings | Plugins
  • Priority changed from 0 to 99
Actions #6

Updated by Elaad FURREEDAN about 1 year ago

  • Status changed from New to In progress
  • Assignee set to Elaad FURREEDAN
Actions #7

Updated by Nicolas CHARLES about 1 year ago

  • Related to Bug #18931: No skipped directives on Rule screen after parent ticket added
Actions #8

Updated by Vincent MEMBRÉ 12 months ago

  • Target version changed from 7.2.8 to 7.2.9
  • Priority changed from 99 to 97
Actions #9

Updated by Vincent MEMBRÉ 11 months ago

  • Target version changed from 7.2.9 to 7.2.10
  • Priority changed from 97 to 96
Actions #10

Updated by Alexis Mousset 11 months ago

  • Target version changed from 7.2.10 to 7.2.11
  • Priority changed from 96 to 95
Actions #11

Updated by Vincent MEMBRÉ 9 months ago

  • Target version changed from 7.2.11 to 1046
  • Priority changed from 95 to 93
Actions #12

Updated by Alexis Mousset 8 months ago

  • Target version changed from 1046 to 7.3.8
  • Priority changed from 93 to 92
Actions #13

Updated by Elaad FURREEDAN 8 months ago

  • Priority changed from 92 to 91
Actions #14

Updated by Elaad FURREEDAN 8 months ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from Elaad FURREEDAN to Nicolas CHARLES
  • Pull Request set to https://github.com/Normation/rudder/pull/5132
Actions #15

Updated by Vincent MEMBRÉ 8 months ago

  • Target version changed from 7.3.8 to 7.3.9
Actions #16

Updated by Vincent MEMBRÉ 8 months ago

  • Target version changed from 7.3.9 to 7.3.10
Actions #17

Updated by François ARMAND 7 months ago

  • Related to Bug #15125: Rule shown as "Mixed" whereas applied to one audit node and one (skipped) enforce directive added
Actions #18

Updated by Elaad FURREEDAN 7 months ago

  • Status changed from Pending technical review to In progress
  • Assignee changed from Nicolas CHARLES to Elaad FURREEDAN
  • Priority changed from 91 to 90

I'm taking over this issue!

Actions #19

Updated by Vincent MEMBRÉ 6 months ago

  • Target version changed from 7.3.10 to 7.3.11
  • Priority changed from 90 to 89
Actions #20

Updated by Clark ANDRIANASOLO 5 months ago

  • Assignee changed from Elaad FURREEDAN to Nicolas CHARLES
  • Priority changed from 89 to 87
Actions #21

Updated by Clark ANDRIANASOLO 5 months ago

  • Assignee changed from Nicolas CHARLES to Elaad FURREEDAN
Actions #22

Updated by Clark ANDRIANASOLO 5 months ago

  • Priority changed from 87 to 86
The rules page was rewritten from Lift in Scala to Elm + Scala API and we lost the "skipped" information on the API side.
We need to :
  • add the information to the current API endpoint whenever the directive is skipped (ideally also all other API endpoints for compliance)
  • display the skipped badge from this information
Actions #23

Updated by Clark ANDRIANASOLO 5 months ago

Actions #24

Updated by Clark ANDRIANASOLO 5 months ago

(Note : in 7.3 the node compliance tree shows the "skipped" badge, but it has also been rewritten in Elm since 8.0 in the issue https://issues.rudder.io/issues/23656 where it disappears)

Actions #25

Updated by Clark ANDRIANASOLO 5 months ago

  • Assignee changed from Elaad FURREEDAN to Clark ANDRIANASOLO
Actions #26

Updated by Vincent MEMBRÉ 5 months ago

  • Target version changed from 7.3.11 to 7.3.12
Actions #27

Updated by Vincent MEMBRÉ 4 months ago

  • Target version changed from 7.3.12 to 7.3.13
  • Priority changed from 86 to 85
Actions #28

Updated by Vincent MEMBRÉ 3 months ago

  • Target version changed from 7.3.13 to 7.3.14
  • Priority changed from 85 to 84
Actions #32

Updated by Clark ANDRIANASOLO 2 months ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from Clark ANDRIANASOLO to François ARMAND
  • Pull Request changed from https://github.com/Normation/rudder/pull/5132 to https://github.com/Normation/rudder/pull/5600
Actions #33

Updated by Clark ANDRIANASOLO 2 months ago

  • Status changed from Pending technical review to In progress
Actions #34

Updated by Clark ANDRIANASOLO 2 months ago

  • Status changed from In progress to Pending technical review
Actions #35

Updated by Clark ANDRIANASOLO 2 months ago

  • Assignee changed from François ARMAND to Elaad FURREEDAN
Actions #36

Updated by Clark ANDRIANASOLO 2 months ago

  • Status changed from Pending technical review to Pending release
Actions #37

Updated by François ARMAND about 2 months ago

  • Related to Bug #24097: Skipped directive does not show in node compliance tree added
Actions #38

Updated by François ARMAND about 2 months ago

  • Fix check changed from To do to Checked

Works well for rules, it still needs to be corrected for nodes (see #24097)

Actions #39

Updated by Vincent MEMBRÉ about 2 months ago

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 7.3.14, 8.0.8 and 8.1.1 which were released today.

Actions

Also available in: Atom PDF