Skip to content

Conversation

@renovate-bot
Copy link
Contributor

@renovate-bot renovate-bot commented Nov 3, 2025

This PR contains the following updates:

Package Change Age Adoption Passing Confidence
com.google.cloud:sdk-platform-java-config 3.53.0 -> 3.54.1 age adoption passing confidence

Configuration

📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about these updates again.


  • If you want to rebase/retry this PR, check this box

This PR was generated by Mend Renovate. View the repository job log.

@renovate-bot renovate-bot requested review from a team as code owners November 3, 2025 23:51
@product-auto-label product-auto-label bot added size: xs Pull request size is extra small. api: storage Issues related to the googleapis/java-storage API. labels Nov 3, 2025
@dpebot
Copy link
Collaborator

dpebot commented Nov 3, 2025

/gcbrun

@trusted-contributions-gcf trusted-contributions-gcf bot added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Nov 3, 2025
BenWhitehead
BenWhitehead previously approved these changes Nov 3, 2025
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Nov 3, 2025
@BenWhitehead
Copy link
Collaborator

Blocked on com.google.api.grpc:proto-google-common-protos:2.63.0 being available in maven central

@dpebot
Copy link
Collaborator

dpebot commented Nov 4, 2025

/gcbrun

@trusted-contributions-gcf trusted-contributions-gcf bot added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Nov 4, 2025
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Nov 4, 2025
@diegomarquezp
Copy link
Contributor

Deleting the cache of these CI jobs.

@dpebot
Copy link
Collaborator

dpebot commented Nov 4, 2025

/gcbrun

@trusted-contributions-gcf trusted-contributions-gcf bot added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Nov 4, 2025
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Nov 4, 2025
@diegomarquezp
Copy link
Contributor

Error: Failed to execute goal org.apache.maven.plugins:maven-enforcer-plugin:3.5.0:enforce (enforce) on project google-cloud-storage: Error: Rule 2: org.apache.maven.enforcer.rules.dependency.RequireUpperBoundDeps failed with message: Error: Failed while enforcing RequireUpperBoundDeps. The error(s) are [ Error: Require upper bound dependencies error for io.opentelemetry:opentelemetry-api:1.47.0 paths to dependency are: Error: +-com.google.cloud:google-cloud-storage:2.59.1-SNAPSHOT Error: +-io.opentelemetry:opentelemetry-api:1.47.0 Error: and Error: +-com.google.cloud:google-cloud-storage:2.59.1-SNAPSHOT Error: +-io.opentelemetry:opentelemetry-sdk:1.47.0 Error: +-io.opentelemetry:opentelemetry-api:1.47.0 (managed) <-- io.opentelemetry:opentelemetry-api:1.47.0 Error: and Error: +-com.google.cloud:google-cloud-storage:2.59.1-SNAPSHOT Error: +-io.grpc:grpc-opentelemetry:1.76.0 Error: +-io.opentelemetry:opentelemetry-api:1.47.0 [runtime] (managed) <-- io.opentelemetry:opentelemetry-api:1.52.0 [runtime] 

Thanks @BenWhitehead for looking into this. I have raised googleapis/sdk-platform-java#3979, which solves this upper bound problem by bumping opentelemetry.version to 1.52.0.

@BenWhitehead
Copy link
Collaborator

Thanks @diegomarquezp!

diegomarquezp added a commit to googleapis/sdk-platform-java that referenced this pull request Nov 7, 2025
With the [recent upgrade of grpc](#3942), 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
@renovate-bot renovate-bot changed the title deps: update dependency com.google.cloud:sdk-platform-java-config to v3.54.0 deps: update dependency com.google.cloud:sdk-platform-java-config to v3.54.1 Nov 7, 2025
@renovate-bot renovate-bot force-pushed the renovate/sdk-platform-java-dependencies branch from 5a191b9 to 7c62a92 Compare November 7, 2025 18:43
@dpebot
Copy link
Collaborator

dpebot commented Nov 7, 2025

/gcbrun

@trusted-contributions-gcf trusted-contributions-gcf bot added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Nov 7, 2025
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Nov 7, 2025
@product-auto-label product-auto-label bot added size: m Pull request size is medium. and removed size: xs Pull request size is extra small. labels Nov 7, 2025
@dpebot
Copy link
Collaborator

dpebot commented Nov 7, 2025

/gcbrun

@trusted-contributions-gcf trusted-contributions-gcf bot added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Nov 7, 2025
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Nov 7, 2025
{ "name": "io.grpc.xds.shaded.com.github.udpa.udpa.type.v1.TypedStruct$Builder", "queryAllDeclaredConstructors": true, "queryAllPublicConstructors": true, "queryAllDeclaredMethods": true, "allPublicMethods": true, "allDeclaredClasses": true, "allPublicClasses": true },
{ "name": "io.grpc.xds.shaded.com.github.xds.core.v3.Authority", "queryAllDeclaredConstructors": true, "queryAllPublicConstructors": true, "queryAllDeclaredMethods": true, "allPublicMethods": true, "allDeclaredClasses": true, "allPublicClasses": true },
{ "name": "io.grpc.xds.shaded.com.github.xds.core.v3.Authority$Builder", "queryAllDeclaredConstructors": true, "queryAllPublicConstructors": true, "queryAllDeclaredMethods": true, "allPublicMethods": true, "allDeclaredClasses": true, "allPublicClasses": true },
{ "name": "io.grpc.xds.shaded.com.github.xds.core.v3.CidrRange", "queryAllDeclaredConstructors": true, "queryAllPublicConstructors": true, "queryAllDeclaredMethods": true, "allPublicMethods": true, "allDeclaredClasses": true, "allPublicClasses": true },
Copy link
Contributor

Choose a reason for hiding this comment

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

I guess this codepath is triggered in storage only but not in gax?

Copy link
Collaborator

Choose a reason for hiding this comment

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

It's triggered by directpath being available for gcs.

Copy link
Contributor

Choose a reason for hiding this comment

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

I've had to move this kind of configurations upstream in recent GraalVM migrations.

"methods":[{"name":"<init>","parameterTypes":[] }]
},
{ "name": "com.google.protobuf.Message$Builder", "queryAllDeclaredConstructors": true, "queryAllPublicConstructors": true, "queryAllDeclaredMethods": true, "allPublicMethods": true, "allDeclaredClasses": true, "allPublicClasses": true },
{ "name": "com.google.protobuf.GeneratedMessageV3", "queryAllDeclaredConstructors": true, "queryAllPublicConstructors": true, "queryAllDeclaredMethods": true, "allPublicMethods": true, "allDeclaredClasses": true, "allPublicClasses": true },
Copy link
Contributor

Choose a reason for hiding this comment

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

This is somewhat alarming, are the protobuf changes required? I don't see GeneratedMessageV3 being used explicitly in grpc-java, and usually we should use the interface directly for stability.

Copy link
Collaborator

Choose a reason for hiding this comment

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

This is one of the classes found by classgraph when analyzing the grpc classes for protobuf classes. we're not depending on this class, simply including it as one that native-image needs to load.

@forking-renovate
Copy link

Edited/Blocked Notification

Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.

You can manually request rebase by checking the rebase/retry box above.

⚠️ Warning: custom changes will be lost.

@diegomarquezp diegomarquezp self-requested a review November 7, 2025 21:59
@BenWhitehead BenWhitehead merged commit e3d3700 into googleapis:main Nov 7, 2025
39 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: storage Issues related to the googleapis/java-storage API. size: m Pull request size is medium.

7 participants