Static access to SmallRyeHealthReporter or HealthRegistry
slaskawi opened this issue · comments
Feature Request
Currently, it is pretty hard to wire new HealthCheck
implementation into the SmallRyeHealthReporter
in a non-CDI environment. Moreover, assembling SmallRyeHealthReporter
by hand (without CDI) seems to be non-trivial. I would like to request more support for accessing those classes from the static context.
More context
When working on Observability and Healthcheck API concepts for Keycloak, I found it very convenient that MetricRegistries
class has static accessors (`MetricRegistries#get1). This feature makes wiring into the application code much simpler. It would be great to have something similar in this project.
@antoinesd @xstefank @jmartisk @radcortez anyone free to take a look at this?
I'm already looking at it @kenfinnigan
thanks @antoinesd
Thank you!
Any progress on this?
actually today I started working on the 3.2.0 release so hopefully should be coming up soon. Sorry for the delays.
@jamesnetherton @slaskawi the PR is up, can you check please? The reporter will be usable like this - https://github.com/xstefank/learning-tests/blob/master/sr-health-plain/src/main/java/io/xstefank/App.java
@jamesnetherton @slaskawi the PR is up, can you check please? The reporter will be usable like this - https://github.com/xstefank/learning-tests/blob/master/sr-health-plain/src/main/java/io/xstefank/App.java
LGTM 👍