Add cli option to disable the default global zones#10028
Add cli option to disable the default global zones#10028julianbrost merged 5 commits intoIcinga:masterfrom
Conversation
|
Thank you for your pull request and welcome to our community. We could not parse the GitHub identity of the following contributors: Lorenz Kästle.
|
This comment was marked as duplicate.
This comment was marked as duplicate.
When setting up Icinga 2 agents, in most cases, the default global
zones are not needed, but have to be removed manually or automatically
whith tools outside of Icinga 2 from the configuration.
This seems like unnecessary work, since the node setup command
does everything else.
This commit introduces a new option for the node setup
command ("--no-default-global-zones") to exclude the default global zones.
19486b5 to
7afda4d
Compare
|
@Al2Klimov ok for me |
Al2Klimov
left a comment
There was a problem hiding this comment.
Please add that new switch to the node setup documentation.
|
I did add documentation in the table for the |
Co-authored-by: Alexander Aleksandrovič Klimov <al2klimov@gmail.com>
julianbrost
left a comment
There was a problem hiding this comment.
Change is fine for me now.
I'll try to push a rebased version (git rebase $(git merge-base origin/master HEAD), no changes, no conflicts) right afterwards to get rid of a merge commit within the PR. (If that doesn't introduce new changes, that should leave the approval intact, that's why I'm doing it that way around.)
7edf8a0 to
bf0ab6e
Compare
When setting up Icinga 2 agents, in most cases, the default global zones are not needed, but have to be removed manually or automatically whith tools outside of Icinga 2 from the configuration.
This seems like unnecessary work, since the node setup command does everything else.
This commit introduces a new option for the node setup command ("--no-default-global-zones") to exclude the default global zones.
fixes #10023