Skip to content

jmx-scraper document activemq metrics from instrumentation#2666

Merged
jaydeluca merged 5 commits intoopen-telemetry:mainfrom
SylvainJuge:update-jmx-scraper-systems
Mar 6, 2026
Merged

jmx-scraper document activemq metrics from instrumentation#2666
jaydeluca merged 5 commits intoopen-telemetry:mainfrom
SylvainJuge:update-jmx-scraper-systems

Conversation

@SylvainJuge
Copy link
Copy Markdown
Contributor

  • activemq has been added in instrumentation 2.23.0 with #14996
  • the current version used by contrib 2.25.0

note: Apache Camel is not yet released, but will require a similar addition once it has been released with 2.26.0. (see #16088)

@SylvainJuge SylvainJuge requested a review from a team as a code owner March 2, 2026 09:32
Copilot AI review requested due to automatic review settings March 2, 2026 09:32
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the JMX Scraper documentation to reflect metrics definitions that are now available via the upstream opentelemetry-jmx-metrics instrumentation, specifically adding an instrumentation reference for ActiveMQ (and introducing a new Camel row).

Changes:

  • Add an instrumentation link for activemq metrics definitions.
  • Add a camel entry pointing to upstream instrumentation metrics definitions.
  • Reformat/update the supported otel.jmx.target.system table rows.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
| `kafka-producer` | Apache Kafka producer | [`kafka-producer.yaml`](src/main/resources/kafka-producer.yaml) | |
| `solr` | Apache Solr | [`solr.yaml`](src/main/resources/solr.yaml) | |
| `tomcat` | Apache Tomcat | [`tomcat.yaml`](src/main/resources/tomcat.yaml) | [`tomcat.yaml`](https://github.com/open-telemetry/opentelemetry-java-instrumentation/blob/main/instrumentation/jmx-metrics/library/src/main/resources/jmx/rules/tomcat.yaml) ([doc](https://github.com/open-telemetry/opentelemetry-java-instrumentation/blob/main/instrumentation/jmx-metrics/library/tomcat.md)) |
| `wildfly` | Wildfly | [`wildfly.yaml`](src/main/resources/wildfly.yaml) | [`wildfly.yaml`](https://github.com/open-telemetry/opentelemetry-java-instrumentation/blob/main/instrumentation/jmx-metrics/library/src/main/resources/jmx/rules/wildfly.yaml) ([doc](https://github.com/open-telemetry/opentelemetry-java-instrumentation/blob/main/instrumentation/jmx-metrics/library/wildfly.md)) |
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

[for reviewer] the link was wrong for wildfly, got caught by copilot review.

@jaydeluca jaydeluca added this pull request to the merge queue Mar 6, 2026
Merged via the queue into open-telemetry:main with commit 00674d8 Mar 6, 2026
21 checks passed
@SylvainJuge SylvainJuge self-assigned this Mar 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants