Skip to content

agents: add shell scripting rule and artifact staging/Starlark rules - #4308

Merged
rickeylev merged 4 commits into
bazel-contrib:mainfrom
rickeylev:optimize_rbe_linux_crossbuilding
Oct 22, 2026
Merged

rickeylev merged 4 commits into
bazel-contrib:mainfrom
rickeylev:optimize_rbe_linux_crossbuilding

Conversation

@rickeylev

Copy link
Copy Markdown
Collaborator

Capture review feedback from cross-building Linux release artifacts so
agents avoid repeating shell, workflow, and Starlark pitfalls.

Codify guidelines for flat artifact staging across Bazel split
transitions, safe manifest iteration and elif usage in shell scripts,
and dedicated providers, explicit for loops, and minimal BUILD
exports in Starlark.

Spinning up a separate GitHub Actions runner for each of the six Linux
platforms is unnecessary now that Linux release builds run on BuildBuddy
RBE.

Replace the six Linux matrix entries with a single `linux` job that
builds `//dev/release_artifacts:cross_compiled_artifacts`, and include
`x86_64-unknown-linux-gnu` in that target along with a manifest file so
one runner builds and verifies all Linux binaries in a single Bazel
invocation.

Work towards bazel-contrib#4216
Cross-compiled Linux outputs live in separate bazel-out/<cfg> directories, causing actions/upload-artifact to preserve those subdirectories when uploading raw manifest paths. Copy the manifest files into release-artifacts/ so they are uploaded flat.
Add .agents/rules/shell.md for bash control flow and safe manifest reading, document flat artifact staging across Bazel split transitions in github_actions_workflows.md, and document provider/comprehension/BUILD export rules in starlark.md.
@rickeylev
rickeylev requested a review from aignas as a code owner October 22, 2026 07:19
@rickeylev
rickeylev merged commit 0774506 into bazel-contrib:main Oct 22, 2026
13 checks passed
@rickeylev
rickeylev deleted the optimize_rbe_linux_crossbuilding branch October 22, 2026 08:03
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