Skip to content
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

Update java agent instrumentation disable list #4875

Merged

Conversation

jaydeluca
Copy link
Member

@jaydeluca jaydeluca commented Jul 24, 2024

Updating Java agent instrumentation disable docs to account for newer instrumentation:

  • Clickhouse Client, clickhouse, added in 2.6.0
  • Javalin, javalin, added in 2.6.0
  • InfluxDB, influxdb, added in 2.4.0
  • Apache ShenYu, apache-shenyu, added in 2.4.0
  • XXL-JOB, xxl-job, added in 2.2.0
  • MyBatis, mybatis, added in 2.1.0
  • Finagle, finagle-http, added in 2.1.0
  • Vert.x Redis Client, vertx-redis-client, added in 2.0.0
  • Hibernate Reactive, hibernate-reactive, added in 1.30.0
  • Ktor, ktor, added in 1.29.0
  • Quarkus, quarkus, added in 1.27.0
  • Reactor Kafka, reactor-kafka, added in 1.27.0
  • Vert.x Sql Client, vertx-sql-client, added in 1.26.0

@jaydeluca jaydeluca requested a review from a team July 24, 2024 12:36
@jaydeluca jaydeluca force-pushed the update-java-instrumentation-disable branch from 99b0c49 to 02e8679 Compare July 24, 2024 12:55
@svrnm svrnm requested review from a team July 24, 2024 15:32
@zeitlinger
Copy link
Member

would be great to also add which modules are disabled by default

I don't think it's documented anywhere right now

@laurit do you know?

Copy link
Member

@trask trask left a comment

Choose a reason for hiding this comment

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

❤️

@trask
Copy link
Member

trask commented Jul 24, 2024

would be great to also add which modules are disabled by default

I don't think it's documented anywhere right now

@laurit do you know?

can you open a separate issue for this?

@svrnm svrnm merged commit 2a06b13 into open-telemetry:main Jul 25, 2024
16 of 17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants