Skip to content

ci: add Bazel RBE lane#38

Merged
haasonsaas merged 1 commit into
mainfrom
codex/bazel-rbe-orbit-agent-20260516
May 16, 2026
Merged

ci: add Bazel RBE lane#38
haasonsaas merged 1 commit into
mainfrom
codex/bazel-rbe-orbit-agent-20260516

Conversation

@haasonsaas
Copy link
Copy Markdown
Contributor

Summary

  • add Bazel 9/Bzlmod/rules_python wiring for the Orbit Agent package and pytest suite
  • reuse the existing requirements-dev.lock so Bazel resolves the same DSPy/Flask/Twilio dependency graph as CI
  • add a repo-scoped Bazel RBE workflow gated by BAZEL_RBE_ENABLED and the evalops-orbit-agent-rbe runner label

Test plan

  • actionlint .github/workflows/ci.yml .github/workflows/bazel-rbe.yml
  • python3 -m pytest -q
  • make bazel-check

@cursor
Copy link
Copy Markdown

cursor Bot commented May 16, 2026

PR Summary

Medium Risk
Medium risk because it introduces new Bazel/Bzlmod build configuration and a self-hosted GitHub Actions lane that can affect CI reliability and remote execution behavior.

Overview
Adds Bazel 9 + Bzlmod support for this repo, including MODULE.bazel/lockfile, .bazelrc (with a remote-gcp-dev RBE config), .bazelignore, and a root BUILD.bazel that defines a py_library for orbit_agent plus a Bazel-driven py_test target.

Introduces a new GitHub Actions workflow (bazel-rbe.yml) gated by BAZEL_RBE_ENABLED that runs make bazel-check and an RBE smoke test on a labeled self-hosted runner, plus local tooling glue: Makefile Bazel targets, a gcloud IAP tunnel helper (run-bazel-rbe.sh), a Bazel platform definition, and a Bazel-friendly tests/pytest_main.py entrypoint; also ignores /bazel-* outputs in .gitignore.

Reviewed by Cursor Bugbot for commit 0b405b4. Bugbot is set up for automated code reviews on this repo. Configure here.

@socket-security
Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Addedmkdocs-material@​9.7.674100100100100
Addedmypy@​2.1.076100100100100
Addedmkdocs@​1.6.181100100100100
Addedpytest@​9.0.387100100100100
Addedpre-commit@​4.6.093100100100100
Addedruff@​0.15.12100100100100100
Addedpytest-cov@​7.1.0100100100100100
Addedpytest-asyncio@​1.3.0100100100100100
Addedtypes-pyyaml@​6.0.12.20260510100100100100100
Addedtypes-requests@​2.33.0.20260508100100100100100

View full report

@haasonsaas haasonsaas merged commit b1c216e into main May 16, 2026
6 checks passed
@haasonsaas haasonsaas deleted the codex/bazel-rbe-orbit-agent-20260516 branch May 16, 2026 05:33
Copy link
Copy Markdown

@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 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.

Comment thread scripts/run-bazel-rbe.sh
kill "${tunnel_pid}" >/dev/null 2>&1 || true
fi
}
trap cleanup EXIT
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 0b405b4. Configure here.

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.

1 participant