Skip to content

[Build] Upgrade Bazel from 6.5.0 to 7.5.0#61601

Open
andrew-anyscale wants to merge 1 commit intomasterfrom
andrew/revup/master/bazel-7-5-upgrade
Open

[Build] Upgrade Bazel from 6.5.0 to 7.5.0#61601
andrew-anyscale wants to merge 1 commit intomasterfrom
andrew/revup/master/bazel-7-5-upgrade

Conversation

@andrew-anyscale
Copy link
Contributor

@andrew-anyscale andrew-anyscale commented Mar 9, 2026

Bump the pinned Bazel version to 7.5.0 and add --noenable_bzlmod to keep the existing WORKSPACE-based dependency setup working.

Topic: bazel-7-5-upgrade
Signed-off-by: andrew andrew@anyscale.com

@andrew-anyscale
Copy link
Contributor Author

andrew-anyscale commented Mar 9, 2026

Reviews in this chain:
#61601 [Build] Upgrade Bazel from 6.5.0 to 7.5.0
 └#61725 [Build] Upgrade protobuf 28.2, absl 20240722.0, grpc 1.68.2, boringssl

@andrew-anyscale
Copy link
Contributor Author

andrew-anyscale commented Mar 9, 2026

# head base diff date summary
0 b0853dc6 55227f94 diff Mar 9 14:15 PM 10 files changed, 50 insertions(+), 11 deletions(-)
1 99ce3fe7 55227f94 diff Mar 9 19:52 PM 4 files changed, 22 insertions(+), 3 deletions(-)
2 72d1c2a7 55227f94 diff Mar 9 20:07 PM 1 file changed, 1 insertion(+), 1 deletion(-)
3 a3c7b007 55227f94 diff Mar 9 20:56 PM 1 file changed, 1 deletion(-)
4 3deab42e 55227f94 diff Mar 10 6:06 AM 4 files changed, 16 insertions(+), 5 deletions(-)
5 de576e28 55227f94 diff Mar 10 9:28 AM 3 files changed, 223 insertions(+)
6 b13ed975 55227f94 diff Mar 10 11:36 AM 1 file changed, 19 insertions(+), 14 deletions(-)
7 c84e7784 55227f94 diff Mar 10 18:39 PM 2 files changed, 11 insertions(+)
8 6394f91f 55227f94 diff Mar 11 7:02 AM 1 file changed, 7 insertions(+)
9 5126efae 55227f94 diff Mar 11 7:32 AM 1 file changed, 4 insertions(+)
10 35898927 55227f94 diff Mar 11 7:56 AM 1 file changed, 6 insertions(+)
11 ce7d8029 a6fec9ca diff Mar 11 16:07 PM 5 files changed, 5 insertions(+), 27 deletions(-)
12 52abc9ad d2029f77 rebase Mar 12 11:49 AM 0 files changed
13 bb290893 d2029f77 diff Mar 12 12:31 PM 2 files changed, 2 insertions(+), 2 deletions(-)
14 5a50d48f f63c28fd rebase Mar 12 12:31 PM 0 files changed
15 62db8ee1 a8f15f83 diff Mar 12 12:49 PM 1 file changed, 2 insertions(+), 1 deletion(-)
16 500f21ee 899eb2e7 diff Mar 12 12:55 PM 0 files changed
17 1d0d801f abcb022d rebase Mar 12 13:08 PM 0 files changed
18 9915e816 e74b48dc rebase Mar 12 20:31 PM 0 files changed
19 c4abe00b 5114f871 rebase Mar 13 11:13 AM 0 files changed
20 8e29c072 4b753789 rebase Mar 13 14:43 PM 0 files changed
21 2d3be3e1 bc015e04 rebase Mar 13 17:26 PM 0 files changed
22 faea87ee 8883f3c3 rebase Mar 13 17:27 PM 0 files changed
23 a0f915de 23fb31c7 rebase Mar 16 7:15 AM 0 files changed
24 cf89a0a2 2a5a4465 diff Mar 16 15:59 PM 0 files changed

Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request upgrades Bazel from version 6.5.0 to 7.5.0. The changes are comprehensive, updating the .bazelversion file, build configurations in .bazelrc and WORKSPACE, and relevant documentation and test files. To maintain compatibility with the existing WORKSPACE-based dependency setup, Bzlmod is disabled. The pull request also includes necessary patches and dependency overrides for rules_apple, apple_support, and protobuf to address breaking changes in Bazel 7. The changes appear correct and well-justified for the upgrade. I have one suggestion to relax the Bazel version pinning in the WORKSPACE file to allow for patch releases, which could improve the developer experience.

@andrew-anyscale andrew-anyscale added the go add ONLY when ready to merge, run all tests label Mar 9, 2026
@andrew-anyscale andrew-anyscale force-pushed the andrew/revup/master/bazel-7-5-upgrade branch from 3589892 to ce7d802 Compare March 11, 2026 23:07
@andrew-anyscale andrew-anyscale changed the base branch from master to andrew/revup/master/lzma-custom-build-file March 11, 2026 23:07
@andrew-anyscale andrew-anyscale force-pushed the andrew/revup/master/lzma-custom-build-file branch from a6fec9c to d2029f7 Compare March 12, 2026 18:49
@andrew-anyscale andrew-anyscale force-pushed the andrew/revup/master/bazel-7-5-upgrade branch 2 times, most recently from 52abc9a to bb29089 Compare March 12, 2026 19:31
@andrew-anyscale andrew-anyscale force-pushed the andrew/revup/master/lzma-custom-build-file branch from d2029f7 to f63c28f Compare March 12, 2026 19:31
@andrew-anyscale andrew-anyscale force-pushed the andrew/revup/master/bazel-7-5-upgrade branch from bb29089 to 5a50d48 Compare March 12, 2026 19:31
@andrew-anyscale andrew-anyscale force-pushed the andrew/revup/master/lzma-custom-build-file branch from f63c28f to c8e8439 Compare March 12, 2026 19:49
@andrew-anyscale andrew-anyscale force-pushed the andrew/revup/master/bazel-7-5-upgrade branch from 5a50d48 to 62db8ee Compare March 12, 2026 19:49
@andrew-anyscale andrew-anyscale changed the base branch from andrew/revup/master/lzma-custom-build-file to andrew/revup/master/macos-apple-toolchain-bazel7 March 12, 2026 19:49
@andrew-anyscale andrew-anyscale force-pushed the andrew/revup/master/bazel-7-5-upgrade branch from 62db8ee to 500f21e Compare March 12, 2026 19:55
@andrew-anyscale andrew-anyscale force-pushed the andrew/revup/master/macos-apple-toolchain-bazel7 branch 2 times, most recently from 899eb2e to abcb022 Compare March 12, 2026 20:08
@andrew-anyscale andrew-anyscale force-pushed the andrew/revup/master/bazel-7-5-upgrade branch from 500f21e to 1d0d801 Compare March 12, 2026 20:08
@andrew-anyscale andrew-anyscale force-pushed the andrew/revup/master/macos-apple-toolchain-bazel7 branch from abcb022 to e74b48d Compare March 13, 2026 03:31
@andrew-anyscale andrew-anyscale force-pushed the andrew/revup/master/bazel-7-5-upgrade branch 2 times, most recently from 9915e81 to c4abe00 Compare March 13, 2026 18:13
@andrew-anyscale andrew-anyscale force-pushed the andrew/revup/master/macos-apple-toolchain-bazel7 branch from e74b48d to 5114f87 Compare March 13, 2026 18:13
@andrew-anyscale andrew-anyscale force-pushed the andrew/revup/master/bazel-7-5-upgrade branch from c4abe00 to 8e29c07 Compare March 13, 2026 21:43
@andrew-anyscale andrew-anyscale force-pushed the andrew/revup/master/macos-apple-toolchain-bazel7 branch 2 times, most recently from 4b75378 to bc015e0 Compare March 14, 2026 00:27
@andrew-anyscale andrew-anyscale force-pushed the andrew/revup/master/bazel-7-5-upgrade branch 2 times, most recently from 2d3be3e to faea87e Compare March 14, 2026 00:27
@andrew-anyscale andrew-anyscale force-pushed the andrew/revup/master/macos-apple-toolchain-bazel7 branch from bc015e0 to 8883f3c Compare March 14, 2026 00:27
@andrew-anyscale andrew-anyscale force-pushed the andrew/revup/master/bazel-7-5-upgrade branch from faea87e to a0f915d Compare March 16, 2026 14:15
@andrew-anyscale andrew-anyscale force-pushed the andrew/revup/master/macos-apple-toolchain-bazel7 branch from 8883f3c to 23fb31c Compare March 16, 2026 14:15
Base automatically changed from andrew/revup/master/macos-apple-toolchain-bazel7 to master March 16, 2026 18:00
Bump the pinned Bazel version to 7.5.0 and add --noenable_bzlmod to keep the existing WORKSPACE-based dependency setup working.

Topic: bazel-7-5-upgrade
Signed-off-by: andrew <andrew@anyscale.com>
@andrew-anyscale andrew-anyscale marked this pull request as ready for review March 16, 2026 22:59
@andrew-anyscale andrew-anyscale force-pushed the andrew/revup/master/bazel-7-5-upgrade branch from a0f915d to cf89a0a Compare March 16, 2026 22:59
@andrew-anyscale andrew-anyscale requested review from a team as code owners March 16, 2026 22:59
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.

# Our generated CPP template does not work with bazel 7.x ,
# so pin the bazel version to 6
USE_BAZEL_VERSION=6.5.0 bash run.sh
USE_BAZEL_VERSION=7.5.0 bash run.sh
Copy link

Choose a reason for hiding this comment

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

Comment contradicts the updated Bazel version pin

Medium Severity

The comment on lines 10-11 explicitly states "Our generated CPP template does not work with bazel 7.x, so pin the bazel version to 6," but USE_BAZEL_VERSION was mechanically changed from 6.5.0 to 7.5.0. If the comment is still accurate, this will break the CPP sanity check. If the template now works with Bazel 7, the comment is stale and misleading. Either way, this looks like an unintentional find-and-replace without verifying the stated incompatibility.

Fix in Cursor Fix in Web

@ray-gardener ray-gardener bot added core Issues that should be addressed in Ray Core devprod labels Mar 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build core Issues that should be addressed in Ray Core devprod go add ONLY when ready to merge, run all tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant