Skip to content

cluster-zone: consider own zone connected if there's only one endpoint #8595

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jan 17, 2023
Merged

cluster-zone: consider own zone connected if there's only one endpoint #8595

merged 1 commit into from
Jan 17, 2023

Conversation

Al2Klimov
Copy link
Member

... because in this case only the checking node can be (not) connected to itself.

fixes #8570

@Al2Klimov
cluster-zone: consider own zone connected if there's only one endpoint
b90b77b 
... because in this case only the checking node can be (not) connected to itself.

refs #8570
@icinga-probot icinga-probot bot added this to the 2.13.0 milestone Jan 14, 2021
@icinga-probot icinga-probot bot added area/distributed Distributed monitoring (master, satellites, clients) area/itl Template Library CheckCommands bug Something isn't working labels Jan 14, 2021
@Al2Klimov
Copy link
Member Author

Before

                    "output": "Zone 'master' is not connected. Log lag: less than 1 millisecond",

After

                    "output": "Zone 'master' is connected. Log lag: less than 1 millisecond",

With additional endpoint

                    "output": "Zone 'master' is not connected. Log lag: less than 1 millisecond",

@Al2Klimov Al2Klimov modified the milestones: 2.13.0, 2.14.0 Jun 2, 2021
@Al2Klimov
Copy link
Member Author

@cla-bot check

@cla-bot cla-bot bot added the cla/signed label Aug 4, 2021
@julianbrost julianbrost self-requested a review January 12, 2023 12:23
@julianbrost julianbrost merged commit a259650 into master Jan 17, 2023
@icinga-probot icinga-probot bot deleted the bugfix/cluster-zone-own-zone-8570 branch January 17, 2023 16:26
@Al2Klimov Al2Klimov mentioned this pull request Jul 4, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@julianbrost julianbrost julianbrost approved these changes

Assignees
No one assigned
Labels
area/distributed Distributed monitoring (master, satellites, clients) area/itl Template Library CheckCommands bug Something isn't working cla/signed
Projects
None yet
Milestone
2.14.0
Development

Successfully merging this pull request may close these issues.

Issue with icinga health check
2 participants
@Al2Klimov @julianbrost