Martin Stefanko

Results 45 comments of Martin Stefanko

Hi @snavlani, this is not the correct place for this issue since the executing thread is specific to Quarkus. But I'll respond here. In SR health (Quarkus integration) `HealthCheck` is...

Sure, on top of my head, this should be doable. I'll look.

@sklalegg, can you please let me know if this interface - https://github.com/smallrye/smallrye-health/blob/main/api/src/main/java/io/smallrye/health/api/HealthContentFilter.java, sorry it's still not document since I didn't have time to do it. But you can then do...

Hi @AleksanderBrzozowski, I guess this is because QOSDK didn't release Quarkus 3.13.0 in time. This is already fixed in main. So if you can, just update the QOSDK dependency to...

@metacosm There isn't much we can do about this. We can either: - Really make these warnings always debug level. Also when they are different by more than a minor...

I did a little perf testing and the difference in caching is there but IMO it's not notable in the context of health: | | default | with context-propagation flag...

Thanks @jponge so we are going forward with this :) Time to pop some champagne, @Ladicek :D

@jmesnil please also take a look. I think this is the correct approach to per deployment overrides with what I had in mind when we declared this in the MP...

I was thinking about providing Jackson API in SR Health, but we need to be aligned with JSON-P for spec. But I personally don't see what's so wrong with a...

> For the existing discrepancies, I would suggest the following. > > Extensions using a slightly different naming scheme: align naming? > > * [`quarkus.health.extensions.enabled`](https://quarkus.io/guides/all-config#quarkus-smallrye-health_quarkus-health-extensions-enabled) => rename to `quarkus.health.enabled`? This...