New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fixes #18346: Make the group option to include the Rudder server or not in the group more clear #3272
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is not only root server
@@ -18,7 +18,7 @@ | |||
<span class="ion ion-checkmark-round check-icon"></span> | |||
</label> | |||
<label for="typeQuery" class="form-control"> | |||
Include Rudder server components | |||
Include Rudder root server |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is not only root, but all node with a server component: so nodes having rudder-reports, rudder-db, rudder-webapp or rudder-server-root
But not the relays
Telling root is not correct - maybe something in the line
"Include Rudder server components (web interface, database)"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nobody knows what "server components" are (I didn't think it's documented anywhere). I really think it should say "root".
Include Rudder root server components (nodes with Rudder application or database)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"application" is maybe too broad, though
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Include Rudder root server components (nodes with Rudder web application or database)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR updated with a new commit |
This PR is not mergeable to upper versions. |
OK, squash merging this PR |
…ot in the group more clear
bbf447d
to
f600fef
Compare
https://issues.rudder.io/issues/18346