-
Notifications
You must be signed in to change notification settings - Fork 721
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
OpenJ9 support for OpenTelemetry metrics #18952
Labels
Comments
Felix Wong is doing some initial testing on this and will update here shortly... |
Using different Liberty Java runtime images with the following Maven dependency.
|
With
|
With
|
With
|
With
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
OpenTelemetry (OTel) has a table that shows the supported libraries and frameworks (at https://github.com/open-telemetry/opentelemetry-java-instrumentation/blob/main/docs/supported-libraries.md)
In that table, the Java Platform row shows:
opentelemetry-runtime-telemetry-java8,
opentelemetry-runtime-telemetry-java17,
opentelemetry-resources
OTel also publishes semantic conventions for metrics, which define standard names and labels to be used for metrics. For Java, there is Semantic Conventions for JVM Metrics.
OpenJ9 needs to provide a standalone library so that apps running on OpenJ9 that are using that library can emit metrics that are compliant with at least the metrics considered
stable
in the semantic conventions. This should be done for java8/11/17/21 (and future releases).The Java8 standalone library may already work for OpenJ9 -- but it would be good to test to make sure that works on all JVM releases. Also, even if the Java8 stanalone library works for all releases, the info at https://github.com/open-telemetry/opentelemetry-java-instrumentation/blob/main/docs/supported-libraries.md should be updated to make it clear that that can be used with different JVM releases (at least for OpenJ9).
The Java17 standalone library does not work with OpenJ9 as that library depends on JFR.
The text was updated successfully, but these errors were encountered: