Skip to content

Commit ccdbc56

Browse files
committed
Prefer Micrometer Metrics on Fault Tolerance
1 parent 64d8d8c commit ccdbc56

File tree

1 file changed

+11
-8
lines changed

1 file changed

+11
-8
lines changed

extensions/smallrye-fault-tolerance/deployment/src/main/java/io/quarkus/smallrye/faulttolerance/deployment/SmallRyeFaultToleranceProcessor.java

Lines changed: 11 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -179,16 +179,19 @@ public void transform(TransformationContext context) {
179179
Enablement.class);
180180

181181
int metricsProviders = 0;
182-
if (metricsCapability.isPresent() && metricsCapability.get().metricsSupported(MetricsFactory.MP_METRICS)) {
183-
builder.addBeanClass("io.smallrye.faulttolerance.metrics.MicroProfileMetricsProvider");
184-
metricsProviders++;
185-
} else if (metricsCapability.isPresent() && metricsCapability.get().metricsSupported(MetricsFactory.MICROMETER)) {
182+
if (metricsCapability.isPresent() && metricsCapability.get().metricsSupported(MetricsFactory.MICROMETER)) {
183+
// Priority to Micrometer. We should avoid having OTel + Micrometer on.
186184
builder.addBeanClass("io.smallrye.faulttolerance.metrics.MicrometerProvider");
187185
metricsProviders++;
188-
}
189-
if (openTelemetrySdk.map(OpenTelemetrySdkBuildItem::isMetricsBuildTimeEnabled).orElse(false)) {
190-
builder.addBeanClass("io.smallrye.faulttolerance.metrics.OpenTelemetryProvider");
191-
metricsProviders++;
186+
} else {
187+
if (metricsCapability.isPresent() && metricsCapability.get().metricsSupported(MetricsFactory.MP_METRICS)) {
188+
builder.addBeanClass("io.smallrye.faulttolerance.metrics.MicroProfileMetricsProvider");
189+
metricsProviders++;
190+
}
191+
if (openTelemetrySdk.map(OpenTelemetrySdkBuildItem::isMetricsBuildTimeEnabled).orElse(false)) {
192+
builder.addBeanClass("io.smallrye.faulttolerance.metrics.OpenTelemetryProvider");
193+
metricsProviders++;
194+
}
192195
}
193196

194197
if (metricsProviders == 0) {

0 commit comments

Comments
 (0)