Repository navigation
chore(deps): Bump the opentelemetry group across 1 directory with 5 updates - #28547
dependabot[bot] wants to merge 1 commit into
Conversation
|
🌟 Thank you for your contribution to the Apache Camel project! 🌟 🐫 Apache Camel Committers, please review the following items:
|
|
🧪 CI tested the following changed modules:
✅ POM dependency changes: targeted tests included Changed properties: opentelemetry-alpha-version,opentelemetry-version Modules affected by dependency changes (5)
🔬 Scalpel shadow comparison — Scalpel: 17 of 712 tested, 3 compile-only — current: 0 all testedMaveniverse Scalpel detected 17 affected modules (current approach: 0). Changed properties: opentelemetry-alpha-version, opentelemetry-version Changed managed dependencies: io.opentelemetry:opentelemetry-common, io.opentelemetry:opentelemetry-context, io.opentelemetry:opentelemetry-opentracing-shim, io.opentelemetry:opentelemetry-api, io.opentelemetry:opentelemetry-exporter-common, io.opentelemetry:opentelemetry-exporter-logging, io.opentelemetry:opentelemetry-exporter-logging-otlp, io.opentelemetry:opentelemetry-extension-kotlin, io.opentelemetry:opentelemetry-extension-trace-propagators, io.opentelemetry:opentelemetry-sdk, io.opentelemetry:opentelemetry-sdk-common, io.opentelemetry:opentelemetry-sdk-logs, io.opentelemetry:opentelemetry-sdk-metrics, io.opentelemetry:opentelemetry-sdk-testing, io.opentelemetry:opentelemetry-sdk-trace, io.opentelemetry:opentelemetry-sdk-extension-autoconfigure, io.opentelemetry:opentelemetry-sdk-extension-autoconfigure-spi, io.opentelemetry:opentelemetry-sdk-extension-jaeger-remote-sampler, io.opentelemetry:opentelemetry-exporter-otlp, io.opentelemetry:opentelemetry-exporter-otlp-common, io.opentelemetry:opentelemetry-exporter-sender-grpc-managed-channel, io.opentelemetry:opentelemetry-exporter-sender-jdk, io.opentelemetry:opentelemetry-exporter-sender-okhttp, io.opentelemetry:opentelemetry-opencensus-shim, io.opentelemetry:opentelemetry-api-incubator, io.opentelemetry:opentelemetry-exporter-prometheus, io.opentelemetry:opentelemetry-sdk-profiles, io.opentelemetry:opentelemetry-sdk-extension-declarative-config, io.opentelemetry:opentelemetry-sdk-extension-incubator, io.opentelemetry:opentelemetry-exporter-otlp-profiles Skip-tests mode would test 17 modules (7 direct + 0 downstream), skip tests for 3 (generated code, meta-modules)
|
| Module | Duration | Status |
|---|---|---|
| Camel :: Opentelemetry Metrics | 37.0s | SUCCESS |
| Camel :: OpenTelemetry (deprecated) | 25.9s | SUCCESS |
| Camel :: Micrometer :: Observation (deprecated) | 2.5s | FAILURE |
| Camel :: Parent | 0.9s | SUCCESS |
| Camel :: All Components Sync point | n/a | |
| Camel :: Assembly | n/a | |
| Camel :: Catalog :: Camel Catalog | n/a | |
| Camel :: Catalog :: Camel Report Maven Plugin | n/a | |
| Camel :: Catalog :: Camel Route Parser | n/a | |
| Camel :: Catalog :: Console | n/a | |
| Camel :: Catalog :: Dummy Component | n/a | |
| Camel :: Catalog :: Maven | n/a | |
| Camel :: Catalog :: Suggest (deprecated) | n/a | |
| Camel :: Component DSL | n/a | |
| Camel :: Coverage | n/a | |
| Camel :: Docs | n/a | |
| Camel :: Endpoint DSL | n/a | |
| Camel :: Endpoint DSL :: Support | n/a | |
| Camel :: Integration Tests | n/a | |
| Camel :: JBang :: Core | n/a | |
| Camel :: JBang :: Export | n/a | |
| Camel :: JBang :: Integration tests | n/a | |
| Camel :: JBang :: MCP | n/a | |
| Camel :: JBang :: Main | n/a | |
| Camel :: JBang :: Plugin :: Edit | n/a | |
| Camel :: JBang :: Plugin :: Generate | n/a | |
| Camel :: JBang :: Plugin :: Kubernetes | n/a | |
| Camel :: JBang :: Plugin :: MCP | n/a | |
| Camel :: JBang :: Plugin :: TUI | n/a | |
| Camel :: JBang :: Plugin :: Testing | n/a | |
| Camel :: JBang :: Plugin :: Validate | n/a | |
| Camel :: JPA | n/a | |
| Camel :: Kamelet Main | n/a | |
| Camel :: Launcher | n/a | |
| Camel :: Launcher :: Container | n/a | |
| Camel :: Micrometer :: Observability 2 | n/a | |
| Camel :: Observability Services | n/a | |
| Camel :: Opentelemetry 2 | n/a | |
| Camel :: Test Infra :: All test services | n/a | |
| Camel :: Test Infra :: Jaeger | n/a | |
| Camel :: YAML DSL | n/a | |
| Camel :: YAML DSL :: Deserializers | n/a | |
| Camel :: YAML DSL :: Maven Plugins | n/a | |
| Camel :: YAML DSL :: Validator | n/a | |
| Camel :: YAML DSL :: Validator Maven Plugin | n/a |
Top 20 slowest modules:
Camel :: Opentelemetry Metrics(37.0s)Camel :: OpenTelemetry (deprecated)(25.9s)Camel :: Micrometer :: Observation (deprecated)(2.5s)Camel :: Parent(0.9s)
davsclaus
left a comment
There was a problem hiding this comment.
Thanks for keeping OpenTelemetry current. Most of 1.67.0 is safe for us: no removed public API, still Java 8 bytecode, and camel-opentelemetry, camel-opentelemetry2 and camel-opentelemetry-metrics all pass in CI. But build (21, false) fails because of this bump. Every test in camel-observation fails with NoClassDefFoundError: io/opentelemetry/api/internal/ConfigUtil.
Root cause. In 1.67.0, ConfigUtil moved out of opentelemetry-api (io.opentelemetry.api.internal.ConfigUtil was removed) into the new opentelemetry-common jar (io.opentelemetry.common.impl.ConfigUtil). camel-observation pins only opentelemetry-sdk-testing to ${opentelemetry-version} (components/camel-observation/pom.xml:78-82). It gets the rest of the SDK through micrometer-tracing-integration-test 1.7.2, which pins OpenTelemetry 1.62.0. So the test classpath now has opentelemetry-api 1.67.0 next to opentelemetry-sdk-common and sdk-trace 1.62.0. The 1.62.0 ExceptionAttributeResolver (called from SdkTracerProvider.builder(), CamelMicrometerObservationTestSupport.java:101) still links against the removed class. Up to 1.66.0, both versions had ConfigUtil in the api jar, which is why this mix used to work.
Suggested fix: let the OpenTelemetry BOM align the whole family in camel-observation's test classpath, as camel-opentelemetry already does:
<dependencyManagement>
<dependencies>
<dependency>
<groupId>io.opentelemetry</groupId>
<artifactId>opentelemetry-bom</artifactId>
<version>${opentelemetry-version}</version>
<type>pom</type>
<scope>import</scope>
</dependency>
</dependencies>
</dependencyManagement>Another option is to declare opentelemetry-sdk and opentelemetry-api at test scope, as camel-micrometer-observability does. camel-observation's main code depends only on micrometer-tracing, so this only affects tests and has no effect for users.
Not tested. Because of fail-fast, camel-micrometer-observability and camel-test-infra-jaeger were skipped. From the pom, camel-micrometer-observability should resolve the 1.67.0 SDK, because its explicit opentelemetry-sdk is declared before micrometer-tracing-bridge-otel. The 1.7.2 bridge does not reference ConfigUtil or the removed SemConvAttributes. Still, please check that module's result on the re-run.
[Nit] Deprecation, no action needed. 1.67.0 deprecates TextMapGetter.keys() and gives it a default implementation. Our overrides in OpenTelemetryTracer.java:209 (camel-opentelemetry2) and OpenTelemetryGetter.java:32 (camel-opentelemetry) now compile with a deprecation warning. Let's keep them for now: keys() is still abstract in the older OpenTelemetry API versions that users may have through the Spring Boot or Quarkus BOMs.
This needs a fix commit on this branch, since Dependabot rebases won't fix the pom.
Reviewed with Claude Code on behalf of davsclaus. This review was generated by an AI agent and may contain inaccuracies; please verify all suggestions before applying.
e17be44 to
309b315
Compare
…pdates Bumps the opentelemetry group with 5 updates in the / directory: | Package | From | To | | --- | --- | --- | | [io.opentelemetry:opentelemetry-bom](https://github.com/open-telemetry/opentelemetry-java) | `1.66.0` | `1.67.0` | | [io.opentelemetry:opentelemetry-sdk](https://github.com/open-telemetry/opentelemetry-java) | `1.66.0` | `1.67.0` | | [io.opentelemetry:opentelemetry-api](https://github.com/open-telemetry/opentelemetry-java) | `1.66.0` | `1.67.0` | | [io.opentelemetry:opentelemetry-sdk-testing](https://github.com/open-telemetry/opentelemetry-java) | `1.66.0` | `1.67.0` | | [io.opentelemetry:opentelemetry-bom-alpha](https://github.com/open-telemetry/opentelemetry-java) | `1.66.0-alpha` | `1.67.0-alpha` | | [io.opentelemetry:opentelemetry-sdk](https://github.com/open-telemetry/opentelemetry-java) | `1.66.0` | `1.67.0` | | [io.opentelemetry:opentelemetry-api](https://github.com/open-telemetry/opentelemetry-java) | `1.66.0` | `1.67.0` | | [io.opentelemetry:opentelemetry-sdk-testing](https://github.com/open-telemetry/opentelemetry-java) | `1.66.0` | `1.67.0` | Updates `io.opentelemetry:opentelemetry-bom` from 1.66.0 to 1.67.0 - [Release notes](https://github.com/open-telemetry/opentelemetry-java/releases) - [Changelog](https://github.com/open-telemetry/opentelemetry-java/blob/main/CHANGELOG.md) - [Commits](open-telemetry/opentelemetry-java@v1.66.0...v1.67.0) Updates `io.opentelemetry:opentelemetry-sdk` from 1.66.0 to 1.67.0 - [Release notes](https://github.com/open-telemetry/opentelemetry-java/releases) - [Changelog](https://github.com/open-telemetry/opentelemetry-java/blob/main/CHANGELOG.md) - [Commits](open-telemetry/opentelemetry-java@v1.66.0...v1.67.0) Updates `io.opentelemetry:opentelemetry-api` from 1.66.0 to 1.67.0 - [Release notes](https://github.com/open-telemetry/opentelemetry-java/releases) - [Changelog](https://github.com/open-telemetry/opentelemetry-java/blob/main/CHANGELOG.md) - [Commits](open-telemetry/opentelemetry-java@v1.66.0...v1.67.0) Updates `io.opentelemetry:opentelemetry-sdk-testing` from 1.66.0 to 1.67.0 - [Release notes](https://github.com/open-telemetry/opentelemetry-java/releases) - [Changelog](https://github.com/open-telemetry/opentelemetry-java/blob/main/CHANGELOG.md) - [Commits](open-telemetry/opentelemetry-java@v1.66.0...v1.67.0) Updates `io.opentelemetry:opentelemetry-bom-alpha` from 1.66.0-alpha to 1.67.0-alpha - [Release notes](https://github.com/open-telemetry/opentelemetry-java/releases) - [Changelog](https://github.com/open-telemetry/opentelemetry-java/blob/main/CHANGELOG.md) - [Commits](https://github.com/open-telemetry/opentelemetry-java/commits) Updates `io.opentelemetry:opentelemetry-sdk` from 1.66.0 to 1.67.0 - [Release notes](https://github.com/open-telemetry/opentelemetry-java/releases) - [Changelog](https://github.com/open-telemetry/opentelemetry-java/blob/main/CHANGELOG.md) - [Commits](open-telemetry/opentelemetry-java@v1.66.0...v1.67.0) Updates `io.opentelemetry:opentelemetry-api` from 1.66.0 to 1.67.0 - [Release notes](https://github.com/open-telemetry/opentelemetry-java/releases) - [Changelog](https://github.com/open-telemetry/opentelemetry-java/blob/main/CHANGELOG.md) - [Commits](open-telemetry/opentelemetry-java@v1.66.0...v1.67.0) Updates `io.opentelemetry:opentelemetry-sdk-testing` from 1.66.0 to 1.67.0 - [Release notes](https://github.com/open-telemetry/opentelemetry-java/releases) - [Changelog](https://github.com/open-telemetry/opentelemetry-java/blob/main/CHANGELOG.md) - [Commits](open-telemetry/opentelemetry-java@v1.66.0...v1.67.0) --- updated-dependencies: - dependency-name: io.opentelemetry:opentelemetry-api dependency-version: 1.67.0 dependency-type: direct:development update-type: version-update:semver-minor dependency-group: opentelemetry - dependency-name: io.opentelemetry:opentelemetry-api dependency-version: 1.67.0 dependency-type: direct:development update-type: version-update:semver-minor dependency-group: opentelemetry - dependency-name: io.opentelemetry:opentelemetry-bom dependency-version: 1.67.0 dependency-type: direct:production update-type: version-update:semver-minor dependency-group: opentelemetry - dependency-name: io.opentelemetry:opentelemetry-bom-alpha dependency-version: 1.67.0-alpha dependency-type: direct:production update-type: version-update:semver-minor dependency-group: opentelemetry - dependency-name: io.opentelemetry:opentelemetry-sdk dependency-version: 1.67.0 dependency-type: direct:production update-type: version-update:semver-minor dependency-group: opentelemetry - dependency-name: io.opentelemetry:opentelemetry-sdk dependency-version: 1.67.0 dependency-type: direct:production update-type: version-update:semver-minor dependency-group: opentelemetry - dependency-name: io.opentelemetry:opentelemetry-sdk-testing dependency-version: 1.67.0 dependency-type: direct:production update-type: version-update:semver-minor dependency-group: opentelemetry - dependency-name: io.opentelemetry:opentelemetry-sdk-testing dependency-version: 1.67.0 dependency-type: direct:production update-type: version-update:semver-minor dependency-group: opentelemetry ... Signed-off-by: dependabot[bot] <[email protected]>
309b315 to
6f64f85
Compare
Bumps the opentelemetry group with 5 updates in the / directory:
1.66.01.67.01.66.01.67.01.66.01.67.01.66.01.67.01.66.0-alpha1.67.0-alpha1.66.01.67.01.66.01.67.01.66.01.67.0Updates
io.opentelemetry:opentelemetry-bomfrom 1.66.0 to 1.67.0Release notes
Sourced from io.opentelemetry:opentelemetry-bom's releases.
... (truncated)
Changelog
Sourced from io.opentelemetry:opentelemetry-bom's changelog.
... (truncated)
Commits
b4b3a1b[release/v1.67.x] Prepare release 1.67.0 (#8924)66ff8f3Prepare for 1.67.0 release (#8918)49dd9f3fix: retry OTLP gRPC responses with trailer status (#8854)6e4eb27Add experimental SDK entity support and OTLP export (#8464)0d0b0caAvoid shared internal ConfigUtil (#6970) (#8855)766eb4dMatch array attributes element-wise in declarative config rule_based sampler ...aa88015Update dependency org.jetbrains.kotlin:kotlin-gradle-plugin to v2.4.21 (#8776)f235b4dUpdate all patch and minor versions (#8908)c99cb41Add type introspection to DeclarativeConfigProperties (#8883)cc4f5f2Avoid regex backtracking when parsing tracestate (#8914)Updates
io.opentelemetry:opentelemetry-sdkfrom 1.66.0 to 1.67.0Release notes
Sourced from io.opentelemetry:opentelemetry-sdk's releases.
... (truncated)
Changelog
Sourced from io.opentelemetry:opentelemetry-sdk's changelog.
... (truncated)
Commits
b4b3a1b[release/v1.67.x] Prepare release 1.67.0 (#8924)66ff8f3Prepare for 1.67.0 release (#8918)49dd9f3fix: retry OTLP gRPC responses with trailer status (#8854)6e4eb27Add experimental SDK entity support and OTLP export (#8464)0d0b0caAvoid shared internal ConfigUtil (#6970) (#8855)766eb4dMatch array attributes element-wise in declarative config rule_based sampler ...aa88015Update dependency org.jetbrains.kotlin:kotlin-gradle-plugin to v2.4.21 (#8776)f235b4dUpdate all patch and minor versions (#8908)c99cb41Add type introspection to DeclarativeConfigProperties (#8883)cc4f5f2Avoid regex backtracking when parsing tracestate (#8914)Updates
io.opentelemetry:opentelemetry-apifrom 1.66.0 to 1.67.0Release notes
Sourced from io.opentelemetry:opentelemetry-api's releases.
... (truncated)
Changelog
Sourced from io.opentelemetry:opentelemetry-api's changelog.
... (truncated)
Commits
b4b3a1b[release/v1.67.x] Prepare release 1.67.0 (#8924)66ff8f3Prepare for 1.67.0 release (#8918)49dd9f3fix: retry OTLP gRPC responses with trailer status (#8854)6e4eb27Add experimental SDK entity support and OTLP export (#8464)0d0b0caAvoid shared internal ConfigUtil (#6970) (#8855)766eb4dMatch array attributes element-wise in declarative config rule_based sampler ...aa88015Update dependency org.jetbrains.kotlin:kotlin-gradle-plugin to v2.4.21 (#8776)f235b4dUpdate all patch and minor versions (#8908)c99cb41Add type introspection to DeclarativeConfigProperties (#8883)cc4f5f2Avoid regex backtracking when parsing tracestate (#8914)Updates
io.opentelemetry:opentelemetry-sdk-testingfrom 1.66.0 to 1.67.0Release notes
Sourced from io.opentelemetry:opentelemetry-sdk-testing's releases.
... (truncated)
Changelog
Sourced from io.opentelemetry:opentelemetry-sdk-testing's changelog.
... (truncated)
Commits
b4b3a1b[release/v1.67.x] Prepare release 1.67.0 (#8924)66ff8f3Prepare for 1.67.0 release (#8918)49dd9f3fix: retry OTLP gRPC responses with trailer status (#8854)6e4eb27Add experimental SDK entity support and OTLP export (#8464)0d0b0caAvoid shared internal ConfigUtil (#6970) (#8855)766eb4dMatch array attributes element-wise in declarative config rule_based sampler ...aa88015Update dependency org.jetbrains.kotlin:kotlin-gradle-plugin to v2.4.21 (#8776)f235b4dUpdate all patch and minor versions (#8908)c99cb41Add type introspection to DeclarativeConfigProperties (#8883)cc4f5f2Avoid regex backtracking when parsing tracestate (#8914)Updates
io.opentelemetry:opentelemetry-bom-alphafrom 1.66.0-alpha to 1.67.0-alphaRelease notes
Sourced from io.opentelemetry:opentelemetry-bom-alpha's releases.
... (truncated)
Changelog
Sourced from io.opentelemetry:opentelemetry-bom-alpha's changelog.
... (truncated)
Commits
Updates
io.opentelemetry:opentelemetry-sdkfrom 1.66.0 to 1.67.0Release notes
Sourced from io.opentelemetry:opentelemetry-sdk's releases.
... (truncated)
Changelog
Sourced from io.opentelemetry:opentelemetry-sdk's changelog.