Skip to content

Fix - use enabled instead of enable-all for java 17 runtime telemetry#446

Merged
cleverchuk merged 1 commit intomainfrom
cc/NH-132006
Mar 18, 2026
Merged

Fix - use enabled instead of enable-all for java 17 runtime telemetry#446
cleverchuk merged 1 commit intomainfrom
cc/NH-132006

Conversation

@cleverchuk
Copy link
Contributor

Summary

Switches the Java 17 runtime telemetry configuration from enable-all to enabled, so that only additional JFR-sourced metrics are collected instead of duplicating metrics already available via JMX.

Details

The SolarwindsPropertiesSupplier class supplies default OpenTelemetry configuration properties when the agent is enabled. One of these properties controls the Java 17 runtime telemetry instrumentation.

The previous property key otel.instrumentation.runtime-telemetry-java17.enable-all activates all JFR-based runtime metrics, including those that overlap with metrics already collected through JMX. This results in redundant data collection for metrics that JMX already provides.

The new property key otel.instrumentation.runtime-telemetry-java17.enabled activates only the supplemental JFR metrics that are not available via JMX, avoiding the duplication.

One trade-off is that jvm.system.cpu.utilization will no longer be available, as it is only exposed through the enable-all configuration. However, reverting to enable-all just for that metric would re-introduce the full set of duplicate JMX metrics, which is not what we want. Hence, we should remove it from the dashboard.

Test services data

  1. e-1712644058766987264
  2. e-1712643928659124224
  3. e-1742334541200846848
  4. e-1777406072376840192

@cleverchuk cleverchuk requested review from a team as code owners March 17, 2026 14:09
Copy link

@tammy-baylis-swi tammy-baylis-swi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sounds good

@cleverchuk cleverchuk merged commit 1f0b512 into main Mar 18, 2026
30 checks passed
@cleverchuk cleverchuk deleted the cc/NH-132006 branch March 18, 2026 14:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants