Project

General

Profile

Actions

Bug #18346

closed

Make the group option to include the Rudder server or not in the group clearer

Added by Félix DALLIDET over 3 years ago. Updated over 3 years ago.

Status:
Released
Priority:
N/A
Category:
Server components
Target version:
Severity:
UX impact:
User visibility:
Effort required:
Priority:
0
Name check:
Reviewed
Fix check:
Checked
Regression:

Description

It is currently displayed:

Include Rudder server components 

But it only excludes the Rudder "root" server and not the relays, so I find it confusing.
It should either state something like:

Include the Rudder server

Or change its behaviour to exclude the relay servers, which seems big breaking change.


Files

Actions #1

Updated by Félix DALLIDET over 3 years ago

I was wrong, it currently also exclude the relays from the group when unchecked. But they are not excluded from the search result when creating/modifying the group.
In definitive they are 2 things to edit:
  • Fix the search display to properly exclude the server and the relays if asked to
  • Change the text of the checkbox to "Include Rudder server and relays"
Actions #2

Updated by François ARMAND over 3 years ago

I double checked and actually, relay server are always included, only the root server is excluded.
It would make sense to exclude all policy servers (root+relays) but it would break existing groups. So I propose to:

- change text for now,
- add an option to exclude relays too, in 6.2.

Actions #4

Updated by François ARMAND over 3 years ago

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

Updated by François ARMAND over 3 years ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from François ARMAND to Félix DALLIDET
  • Pull Request set to https://github.com/Normation/rudder/pull/3272
Actions #6

Updated by François ARMAND over 3 years ago

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

Updated by Alexis Mousset over 3 years ago

  • Subject changed from Make the group option to include the Rudder server or not in the group more clear to Make the group option to include the Rudder server or not in the group clearer
Actions #8

Updated by François ARMAND over 3 years ago

  • Fix check changed from To do to Checked
Actions #9

Updated by Alexis Mousset over 3 years ago

  • Name check changed from To do to Reviewed
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.6 and 6.2.0~beta1 which were released today.

Actions

Also available in: Atom PDF