Skip to content
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

Use driver from gRPC core 1.60.0 #390

Merged
merged 3 commits into from
Mar 18, 2024
Merged

Conversation

lepistone
Copy link
Contributor

This is in preparation for using a driver that contains the fix grpc/grpc#35384.

There are no releases yet that contain the fix, but at least we can start by getting the latest and fixing the related problems.

There is in fact a pre-release with the fixed driver: v1.61.0-pre1.

Once this fix is merged, the fixed driver can be built by setting DRIVER_VERSION to v1.61.0-pre1 or master.

@ejona86 ejona86 requested a review from paulosjca March 12, 2024 17:04
@paulosjca paulosjca self-assigned this Mar 13, 2024
@paulosjca paulosjca added bug Something isn't working release notes: yes Indicates that PR needs to be in release notes labels Mar 13, 2024
Makefile Outdated Show resolved Hide resolved
This is the same image used in grpc#388. It upgrades bazel from 6.1.2 to
6.4.0, which is the default supported version starting with v1.60.0.

Without this change, building the driver image fails with:

/tmp/build_output/5e0c6d2c26e81c9358486406038ccc44/external/com_google_protobuf/upb_generator/BUILD:266:21:
@com_google_protobuf//upb_generator:protoc-gen-upb_toolchain: no such attribute 'output_files' in 'proto_lang_toolchain' rule

https://github.com/grpc/grpc/blob/master/.bazelversion
https://github.com/grpc/grpc/blob/master/bazel/supported_versions.txt
@paulosjca paulosjca merged commit e237efc into grpc:master Mar 18, 2024
1 of 2 checks passed
paulosjca pushed a commit to paulosjca/test-infra that referenced this pull request Mar 18, 2024
* Update bazel build image for the driver

This is the same image used in grpc#388. It upgrades bazel from 6.1.2 to
6.4.0, which is the default supported version starting with v1.60.0.

Without this change, building the driver image fails with:

/tmp/build_output/5e0c6d2c26e81c9358486406038ccc44/external/com_google_protobuf/upb_generator/BUILD:266:21:
@com_google_protobuf//upb_generator:protoc-gen-upb_toolchain: no such attribute 'output_files' in 'proto_lang_toolchain' rule

https://github.com/grpc/grpc/blob/master/.bazelversion
https://github.com/grpc/grpc/blob/master/bazel/supported_versions.txt

* Use the latest released driver, from grpc/grpc 1.62.1.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working release notes: yes Indicates that PR needs to be in release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants