Project

General

Profile

Actions

Bug #18877

closed

"Skipped" Directives on Rules Screen

Added by Lars Koenen almost 4 years ago. Updated over 3 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:

Description

This is a bug report related to the following conversation in gitter.
https://gitter.im/normation/rudder?at=601835cc1ed88c58d81391ec

Rudder Version is 6.2.1

When i look at the details of a rule in the Rule Screen, several directives are shown as "skipped" under "Compliance by Directive". The problem is that these directives were not added to the rule.
This happens not only with one rule but with several.

For example, I have a rule that has only two directives "d1" and "d2". This rule is applied to the group of all VMs. When I then look at the "Compliance by Directive" section, Rudder shows me 31 other skipped directives "d3" - "d33" that I never added to the rule.

What I additionally do not understand. If I then hover with the mouse over the "skipped" tag, Rudder says:
"This directive is skipped, because it is overriden by an other one on that node"
"...on that Node" Which Node? I have only added groups.

Overall, I find it strange that at this place Directives are displayed as "skipped". Is this the way it should be?
I would only expect something like this in the rules overview of a single node.

EDIT: moreover, rules with skipped directives are duplicated in node compliance (from node screen).


Files

clipboard-202102032145-sjpea.png (28.7 KB) clipboard-202102032145-sjpea.png François ARMAND, 2021-02-03 21:45
clipboard-202102032148-bdqft.png (29.6 KB) clipboard-202102032148-bdqft.png François ARMAND, 2021-02-03 21:48
clipboard-202102032149-y0luh.png (38.6 KB) clipboard-202102032149-y0luh.png François ARMAND, 2021-02-03 21:49

Subtasks 1 (0 open1 closed)

Bug #18931: No skipped directives on Rule screen after parent ticketReleasedNicolas CHARLESActions
Actions #1

Updated by François ARMAND almost 4 years ago

Thanks for reporting. This is indeed very strange.

Actions #2

Updated by François ARMAND almost 4 years ago

  • Status changed from New to In progress
  • Assignee set to François ARMAND

Updated by François ARMAND almost 4 years ago

OK, I can reproduce.

I have a case where it makes sense, but the wording it bad:

Here, on that rule, we have one directive skipped because of one other directive on the same rule. It should not say "on that node", but "on that rule" and skippe the end part which is redondant.

I have also a case where it doesn't make any sense. On an other rule, I have only one directive:

But the compliance part of the rule screen still displays on directive from another rule to tell us it is skipped (but one directive on that same other rule, ie it's the previous example):

So: very likely, we are looking for all possible skipped directive on all nodes (because we compute compliance by node) but when we aggregate by rule for the compliance by rule, we are not filtering the skipped one for the given rule.

Actions #4

Updated by François ARMAND almost 4 years ago

  • Description updated (diff)
  • Target version changed from 6.2.3 to 6.1.10
Actions #5

Updated by François ARMAND almost 4 years 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/3499
Actions #6

Updated by François ARMAND almost 4 years ago

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

Updated by Nicolas CHARLES almost 4 years ago

  • Fix check changed from To do to Error - Blocking
Actions #8

Updated by Nicolas CHARLES almost 4 years ago

It seems it does not correctly show skipped directives in Rules screen - but in Nodes it is correct, opening a sub-task

Actions #9

Updated by Vincent MEMBRÉ over 3 years ago

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

Updated by Vincent MEMBRÉ over 3 years ago

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 6.1.10 and 6.2.3 which were released today.

Actions

Also available in: Atom PDF