Skip to content
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

Improve haproxy and ceph dashboard integration #2185

Merged
merged 1 commit into from
May 24, 2024
Merged

Improve haproxy and ceph dashboard integration #2185

merged 1 commit into from
May 24, 2024

Conversation

janhorstmann
Copy link
Contributor

Passive ceph mgr are configured to emit an error with status code 404 for ceph dashboard.
Haproxy is configured to treat status 404 as server state "NOLB" instead of marking servers as "DOWN".
The clear separation between a passive server with state "NOLB" and a malfunctioning server with state "DOWN" will allow more differentiated alerting on haproxy server states.

Part of osism/issues#1013

Passive ceph mgr are configured to emit an error with status code 404
for ceph dashboard.
Haproxy is configured to treat status 404 as server state "NOLB" instead
of marking servers as "DOWN".
The clear separation between a passive server with state "NOLB" and a
malfunctioning server with state "DOWN" will allow more differentiated
alerting on haproxy server states.

Part of osism/issues#1013

Signed-off-by: Jan Horstmann <horstmann@osism.tech>
@janhorstmann janhorstmann requested a review from berendt May 7, 2024 10:14
@berendt
Copy link
Member

berendt commented May 8, 2024

We should also add this to the osism/cfg-cookiecutter repository as default.

@janhorstmann
Copy link
Contributor Author

We should also add this to the osism/cfg-cookiecutter repository as default.

Done in osism/cfg-cookiecutter#610

But it still needs testing

@berendt berendt marked this pull request as ready for review May 24, 2024 17:19
@berendt berendt merged commit 954c9c8 into main May 24, 2024
2 checks passed
@berendt berendt deleted the gh1013 branch May 24, 2024 17:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants