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

Conditional health checks #495

Open
HerrDerb opened this issue Sep 11, 2023 · 1 comment
Open

Conditional health checks #495

HerrDerb opened this issue Sep 11, 2023 · 1 comment

Comments

@HerrDerb
Copy link
Contributor

HerrDerb commented Sep 11, 2023

We have a situation where we would like to execute only a set of specific health checks depending on the incoming REST call. This is because we run multiple tenants on the same instance and would like to show only "tenant A"-relevant health checks if the request runs in the context of "tenant A." On the other hand, we would like to see all health checks if there is no tenant-specific context.

Registering/unregistering health checks inside of a Vertx router filter is not an option, as it seems hacky and mainly beacuse this changes the configuration of the system rather than only being effective for the current request.

SUGGESTION: A new interface ConditionalHealthCheck which extends HealthCheck that holds an additional method, e.g., boolean runCheck(). The Health Registry can then check if a health check needs to be executed or not.

@HerrDerb
Copy link
Contributor Author

HerrDerb commented Dec 23, 2024

@xstefank we probably can close this, as no traction. What is your opinion?

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

No branches or pull requests

1 participant