Repository navigation
ci: connect CI to Aspect Cloud - #216
Merged
Merged
Conversation
Point the aspect-cli jobs at Aspect Cloud's remote cache and BES via --remote, with endpoints from a committed .aspect/config.json. Point the vanilla-bazel smoke job at it via --config=aspect-cloud from a committed .aspect/bazelrc, which every workspace try-imports. Fork PRs, which get no secrets, skip both. Bump the Aspect CLI to 2026.38.24 for deployment support, and move from the deprecated --task-key to --task:name. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Aspect Workflows Tasks📅 Tue Sep 22 18:36:49 UTC 2026 Task Results
⏱ Last updated Tue Sep 22 18:39:12 UTC 2026 · 📊 GitHub API quota 337/7,700 (4% used, resets in 56m) |
The Aspect CLI, which the aspect-cloud config uses as its credential helper and workspace status command, does not support Windows. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
acozzette
force-pushed
the
aspect-cloud-windows
branch
from
September 22, 2026 17:55
8f0c4ee to
b013406
Compare
test.sh shares the output base of the aspect test run before it, whose remote cache hits were never downloaded. Without the remote cache, Bazel 7 fails with "Missing digest" when an action needs one of those outputs, and Bazel 8+ rebuilds them locally. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
That config sets --execution_log_compact_file, which conflicts with the --execution_log_json_file this script reads. Clear it for that build, and forward the script's arguments right after the bazel command so that each invocation's own flags win. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
acozzette
marked this pull request as ready for review
September 22, 2026 18:50
jbedard
approved these changes
Sep 22, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This change points both our aspect-cli and vanilla bazel jobs at Aspect Cloud's remote cache and BES. Windows jobs are excluded because aspect-cli does not support Windows. Fork PRs also do not use Aspect Cloud since they do not have access to the necessary organization secrets.
I also wired up the vanilla bazel invocations inside of
test.shscripts. This works around a bug in Bazel 7: bazelbuild/bazel#26657 Otherwise we would getMissing digesterrors.Changes are visible to end-users: no
Test plan