Skip to content

Conversation

@diegomarquezp
Copy link
Contributor

@diegomarquezp diegomarquezp commented Nov 5, 2025

With the recent upgrade of grpc, the transitive dependency on opentelemetry-api was bumped to 1.52.0 as per grpc/grpc-java@f30964a.

In order to prevent upper bound dependency warnings (as in googleapis/java-storage#3381), we update opentelemetry.version to 1.52.0

@diegomarquezp
Copy link
Contributor Author

diegomarquezp commented Nov 5, 2025

java-datastore:

Error:  Errors: 
Error:    EnabledTraceUtilTest.globalOpenTelemetryRegistersGrpcChannelConfigurator:101 » NoClassDefFound io/opentelemetry/sdk/internal/ExceptionAttributeResolver
Error:    EnabledTraceUtilTest.openTelemetryInstanceRegistersGrpcChannelConfigurator:85 » NoClassDefFound io/opentelemetry/sdk/internal/ExceptionAttributeResolver
Error:    EnabledTraceUtilTest.usesGlobalOpenTelemetryIfOpenTelemetryInstanceNotProvided:68 » NoClassDefFound io/opentelemetry/sdk/internal/ExceptionAttributeResolver
Error:    EnabledTraceUtilTest.usesOpenTelemetryFromOptions:53 » NoClassDefFound io/opentelemetry/sdk/internal/ExceptionAttributeResolver

We may need to look further into this repository.

@sonarqubecloud
Copy link

sonarqubecloud bot commented Nov 5, 2025

@sonarqubecloud
Copy link

sonarqubecloud bot commented Nov 5, 2025

Quality Gate Passed Quality Gate passed for 'java_showcase_integration_tests'

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@diegomarquezp
Copy link
Contributor Author

The downstream check uses the latest release, not HEAD.

git clone "https://github.com/googleapis/$repo.git" --depth=1 --branch "v$last_release"
update_all_poms_dependency "$repo" google-cloud-shared-dependencies "$SHARED_DEPS_VERSION"

This means that we will expect the downstream check of java-datastore to fail even after merging googleapis/java-datastore#1995 (PR to adapt to opentelemetry bump).

@diegomarquezp diegomarquezp merged commit 764778c into main Nov 7, 2025
72 of 76 checks passed
@diegomarquezp diegomarquezp deleted the deps/bump-opentelemetry branch November 7, 2025 15:08
diegomarquezp pushed a commit that referenced this pull request Nov 7, 2025
🤖 I have created a release *beep* *boop*
---


<details><summary>2.64.1</summary>

##
[2.64.1](v2.64.0...v2.64.1)
(2025-11-07)


### Dependencies

* bump opentelemetry.version to 1.52.0
([#3979](#3979))
([764778c](764778c))
</details>

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size: xs Pull request size is extra small.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants