Skip to content

Conversation

ejona86
Copy link
Member

@ejona86 ejona86 commented Jul 29, 2025

Notably, protobuf to 3.25.8, opentelemetry to 1.52.0. Protobuf in Bazel only has 25.5 in the BCR, and it seems better to align the WORKSPACE with that version.

@ejona86 ejona86 requested a review from kannanjgithub July 29, 2025 20:57
@ejona86
Copy link
Member Author

ejona86 commented Aug 5, 2025

Ugh. The version bump conflicts with. every. single. example. file. Which also means back-porting will be prohibitive.

ejona86 added 3 commits August 5, 2025 10:38
Notably, protobuf to 3.25.8, opentelemetry to 1.52.0. Protobuf in Bazel
only has 25.5 in the BCR, and it seems better to align the WORKSPACE
with that version.
@ejona86 ejona86 added the TODO:backport PR needs to be backported. Removed after backport complete label Aug 5, 2025
@ejona86 ejona86 merged commit f30964a into grpc:master Aug 6, 2025
16 checks passed
@ejona86 ejona86 deleted the deps-bump branch August 6, 2025 18:01
ejona86 added a commit to ejona86/grpc-java that referenced this pull request Aug 6, 2025
Notably, protobuf to 3.25.8, opentelemetry to 1.52.0. Protobuf in Bazel
has 25.5 in the BCR and it seems better to align the WORKSPACE
with that version. But we can't actually use 25.5 in BCR because it is
incompatible with Bazel 7.
ejona86 added a commit that referenced this pull request Aug 8, 2025
Notably, protobuf to 3.25.8, opentelemetry to 1.52.0. Protobuf in Bazel
has 25.5 in the BCR and it seems better to align the WORKSPACE
with that version. But we can't actually use 25.5 in BCR because it is
incompatible with Bazel 7.
@ejona86 ejona86 removed the TODO:backport PR needs to be backported. Removed after backport complete label Aug 18, 2025
AgraVator pushed a commit to AgraVator/grpc-java that referenced this pull request Sep 26, 2025
Notably, protobuf to 3.25.8, opentelemetry to 1.52.0. Protobuf in Bazel
has 25.5 in the BCR and it seems better to align the WORKSPACE
with that version. But we can't actually use 25.5 in BCR because it is
incompatible with Bazel 7.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants