ci: add Bazel RBE lane#38
Conversation
PR SummaryMedium Risk Overview Introduces a new GitHub Actions workflow ( Reviewed by Cursor Bugbot for commit 0b405b4. 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 0b405b4. Configure here.
| kill "${tunnel_pid}" >/dev/null 2>&1 || true | ||
| fi | ||
| } | ||
| trap cleanup EXIT |
There was a problem hiding this comment.
Tunnel cleanup trap registered after resource creation
Low Severity
The trap cleanup EXIT is registered after the IAP tunnel process is started and sleep 2 runs. If the script is interrupted (e.g., CI job cancellation sending SIGTERM) during the sleep 2 window, the tunnel background process is orphaned because the EXIT trap hasn't been registered yet. On the self-hosted runner used by this workflow, orphaned gcloud tunnel processes persist across jobs. Moving the cleanup function definition and trap cleanup EXIT to before the tunnel-start block would close this gap.
Reviewed by Cursor Bugbot for commit 0b405b4. Configure here.


Summary
requirements-dev.lockso Bazel resolves the same DSPy/Flask/Twilio dependency graph as CIBAZEL_RBE_ENABLEDand theevalops-orbit-agent-rberunner labelTest plan
actionlint .github/workflows/ci.yml .github/workflows/bazel-rbe.ymlpython3 -m pytest -qmake bazel-check