Skip to content

[ci] Upload Bazel build profile #9364

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
Gold856 wants to merge 3 commits into
base: main
Choose a base branch
from
+18 −1
Open

[ci] Upload Bazel build profile #9364

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
eb59986
[ci] Upload Bazel build profile
Gold856 Aug 19, 2026
0eb2350
Update .github/workflows/bazel.yml
Gold856 Aug 30, 2026
48b676a
Fix file extension
Gold856 Aug 30, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
19 changes: 18 additions & 1 deletion .github/workflows/bazel.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -96,9 +96,26 @@ jobs:
else
TARGETS=...
fi
bazel ${ACTION} ${TARGETS} --config=ci -c opt --repo_env=WPI_PUBLISH_CLASSIFIER_FILTER='${{ matrix.classifier }}'
bazel ${ACTION} ${TARGETS} --config=ci -c opt --repo_env=WPI_PUBLISH_CLASSIFIER_FILTER='${{ matrix.classifier }}' --execution_log_compact_file="${{ matrix.name }} Exec Log.binpb.zst"
shell: bash

- uses: actions/upload-artifact@v7
with:
archive: false
include-hidden-files: true
path: "/github/home/.bazel/command-*.profile.gz"
Comment thread
Gold856 marked this conversation as resolved.
if: runner.os == 'Linux'

@chatgpt-codex-connector chatgpt-codex-connector Bot Sep 9, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Upload Bazel profiles for the macOS matrix job

For the macOS matrix entry, this Linux predicate is false and the following profile-upload step's Windows predicate is also false, so that job uploads only its execution log and never its Bazel build profile. Add a macOS-specific profile path or make the profile upload path platform-dependent so performance data from the macOS build is retained too.

Useful? React with 👍 / 👎.

@chatgpt-codex-connector chatgpt-codex-connector Bot Sep 9, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Upload the macOS Bazel profile too

For the macOS matrix entry, both profile-upload steps are skipped because their conditions only accept Linux or Windows, while the remaining upload only captures the execution log. As a result, this change never preserves the Bazel build profile for the macOS build, so add a macOS-specific profile path/upload.

Useful? React with 👍 / 👎.


- uses: actions/upload-artifact@v7
with:
archive: false
path: "D:/bazelroot/**/command-*.profile.gz"
Comment thread
Gold856 marked this conversation as resolved.
if: runner.os == 'Windows'

- uses: actions/upload-artifact@v7
with:
archive: false
path: "${{ matrix.name }} Exec Log.binpb.zst"
Comment on lines +115 to +118

@chatgpt-codex-connector chatgpt-codex-connector Bot Sep 9, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Run diagnostic uploads after failed Bazel commands

When the Bazel build/test step fails, GitHub Actions' implicit success() condition skips this execution-log upload as well as the two preceding profile uploads. Those files are especially useful for investigating a failed CI build and Bazel still produces them for many action or test failures, so give the uploads an explicit non-cancellation or always() condition while retaining the platform checks.

Useful? React with 👍 / 👎.

- name: Check disk free space
if: always()
run: df -h
Expand Down
Loading