-
Notifications
You must be signed in to change notification settings - Fork 122
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
Demosntrate declarative config 0.3 with otel java agent #492
base: main
Are you sure you want to change the base?
Conversation
@@ -24,7 +24,7 @@ dependencies { | |||
//spring modules | |||
implementation("org.springframework.boot:spring-boot-starter-web") | |||
|
|||
agent("io.opentelemetry.javaagent:opentelemetry-javaagent:2.8.0") | |||
agent("io.opentelemetry.javaagent:opentelemetry-javaagent:2.9.0-SNAPSHOT") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't merge until next release of opentelemetry-java-instrumentation
and the -SNAPSHOT
is reverted.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it that important tho for examples? As long as renovate knows to bump it.....I dunno.
dropwizard-metrics: | ||
enabled: true |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This instrumentation is not enbabled by default....do we want it enabled here? This also applies to:
- jdbc-datasource
- micrometer
- mybatis
- spring-batch
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it should reflect the default values I think
Demonstrates how declarative config is integrated with the otel java agent and can be used to configure instrumentations.
Related to: open-telemetry/opentelemetry-java-instrumentation#12265