Skip to content

refactor: remove JMX and JVM metrics receivers remnants#2332

Merged
rafaelwestphal merged 1 commit into
ops_agent_3.0from
westphalrafael/remove-jmx-jvm-remnants
Jun 4, 2026
Merged

refactor: remove JMX and JVM metrics receivers remnants#2332
rafaelwestphal merged 1 commit into
ops_agent_3.0from
westphalrafael/remove-jmx-jvm-remnants

Conversation

@rafaelwestphal
Copy link
Copy Markdown
Contributor

Description

Ops Agent 3.0 does not register JVM or JMX-based receivers (e.g. jvm, activemq, cassandra, tomcat), rendering JMX build steps and validation logic dead.
This PR removes:

  • The opentelemetry-java-contrib submodule.
  • JMX build steps from tasks.mak, builds/otel.sh, and Dockerfile.windows.
  • OpenJDK/Java installation and build stages from the Linux Dockerfile template (dockerfiles/template, dockerfiles/template-header) and generator (dockerfiles/compile.go).
  • JMX jar references from packaging and plugin build scripts (pkg/goo/google-cloud-ops-agent.goospec, pkg/plugin/build.ps1, pkg/plugin/build.sh).
  • Unused JMX/JVM structs and validation logic in confgenerator/config.go.
  • The mock for FindJarPath in confgenerator/confgenerator_test.go.

Related issue

How has this been tested?

Checklist:

  • Unit tests
    • Unit tests do not apply.
    • Unit tests have been added/modified and passed for this PR.
  • Integration tests
    • Integration tests do not apply.
    • Integration tests have been added/modified and passed for this PR.
  • Documentation
    • This PR introduces no user visible changes.
    • This PR introduces user visible changes and the corresponding documentation change has been made.
  • Minor version bump
    • This PR introduces no new features.
    • This PR introduces new features, and there is a separate PR to bump the minor version since the last release already.
    • This PR bumps the version.

@rafaelwestphal rafaelwestphal changed the base branch from master to ops_agent_3.0 June 4, 2026 02:01
Ops Agent 3.0 does not register JVM or JMX-based receivers.
This CL removes:
- opentelemetry-java-contrib submodule.
- JMX build steps from tasks.mak, builds/otel.sh, and Dockerfile.windows.
- JMX jar references from packaging and plugin build scripts (pkg/goo/google-cloud-ops-agent.goospec, pkg/plugin/build.ps1, pkg/plugin/build.sh).
- Unused JMX/JVM structs and validation logic in confgenerator/config.go.
- Mock for FindJarPath in confgenerator/confgenerator_test.go.
- OpenJDK installation and build steps from Linux Dockerfile template and generator.
@rafaelwestphal rafaelwestphal force-pushed the westphalrafael/remove-jmx-jvm-remnants branch from 65af40e to 6ab7711 Compare June 4, 2026 09:46
@rafaelwestphal rafaelwestphal marked this pull request as ready for review June 4, 2026 12:53
@rafaelwestphal rafaelwestphal merged commit 5d6f31b into ops_agent_3.0 Jun 4, 2026
70 of 74 checks passed
@rafaelwestphal rafaelwestphal deleted the westphalrafael/remove-jmx-jvm-remnants branch June 4, 2026 12:53
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.

1 participant