ci: add Bazel RBE lane#12
Conversation
PR SummaryMedium Risk Overview Adds remote execution (RBE) support via Updates CI/tooling: a new gated GitHub Actions workflow Reviewed by Cursor Bugbot for commit 3c714e9. Bugbot is set up for automated code reviews on this repo. Configure here. |
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is ON, but it could not run because the branch was deleted or merged before autofix could start.
Reviewed by Cursor Bugbot for commit 3c714e9. Configure here.
| bazel-rbe-smoke: | ||
| scripts/run-bazel-rbe.sh test //:pytest | ||
|
|
||
| bazel-check: requirements-lock bazel-format bazel-mod-tidy bazel-test |
There was a problem hiding this comment.
CI "check" target modifies files instead of validating
Medium Severity
The bazel-check target depends on requirements-lock, bazel-format, and bazel-mod-tidy, all of which mutate files in-place (regenerating the lock file, reformatting BUILD files, tidying MODULE.bazel) before running bazel-test. When the CI workflow runs make bazel-check, it silently fixes stale or unformatted committed files and then tests against the modified working tree, so it will never catch a PR with a stale requirements_lock.txt or unformatted BUILD files. The committed files remain incorrect but CI passes.
Additional Locations (1)
Reviewed by Cursor Bugbot for commit 3c714e9. Configure here.


Summary
BAZEL_RBE_ENABLEDand theevalops-agent-harness-rberunner labelTest plan
actionlint .github/workflows/ci.yml .github/workflows/bazel-rbe.ymlpython3 -m pytest -qmake bazel-checkmake bazel-test