Skip to content

Bazel v8 upgrade and migration to bzlmod#601

Merged
swufygoog merged 11 commits into
openconfig:mainfrom
swufygoog:main
Nov 10, 2025
Merged

Bazel v8 upgrade and migration to bzlmod#601
swufygoog merged 11 commits into
openconfig:mainfrom
swufygoog:main

Conversation

@swufygoog

@swufygoog swufygoog commented Oct 22, 2025

Copy link
Copy Markdown
Contributor

Bazel 8 disables WORKSPACE files by default, so leaving the WORKSPACE file as is allows for quick rollback without reverting the commit.

Protos regenerated for two reasons:

  • Minor version upgrade protoc-gen-go v1.36.3 ~> v1.36.5
  • grpc generated code is now in a separate file (Ex: packetio_grpc.pb.go) and generated with protoc-gen-go-grpc v1.5.1

tools/genproto.sh updated to add u+w bit to bazel-bin files before coping them over. This write bit is required to update the generated proto files later.

Rollback instructions without reverting commit:

  1. Change .bazelversion file to swap the two bazel version numbers.
  2. Add the flag --noenable_bzlmod to your bazel commands.
  3. Notify us of the breakage by filing an issue.

@swufygoog swufygoog requested a review from a team as a code owner October 22, 2025 16:53
@bstoll

bstoll commented Oct 22, 2025

Copy link
Copy Markdown
Contributor

/gcbrun

@bstoll

bstoll commented Oct 23, 2025

Copy link
Copy Markdown
Contributor

/gcbrun

@swufygoog

Copy link
Copy Markdown
Contributor Author

/gcbrun

@jasdeep-hundal jasdeep-hundal left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Relying on bstoll@'s review as the Bazel expert, approving as code owner.

@swufygoog swufygoog merged commit 8113535 into openconfig:main Nov 10, 2025
9 checks passed
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.

3 participants