Skip to content

.bazelci/presubmit.yml: run BCR tests with multiple Bazel versions#4490

Merged
jayconrod merged 5 commits intobazel-contrib:masterfrom
jayconrod:presubmit-bcr-bazel-versions
Oct 24, 2025
Merged

.bazelci/presubmit.yml: run BCR tests with multiple Bazel versions#4490
jayconrod merged 5 commits intobazel-contrib:masterfrom
jayconrod:presubmit-bcr-bazel-versions

Conversation

@jayconrod
Copy link
Copy Markdown
Collaborator

What type of PR is this?

Test

What does this PR do? Why is it needed?

On presubmit, run the BCR test with multiple Bazel versions. This helps us
catch issues before a release.

Which issues(s) does this PR fix?

n/a

Other notes for review

@jayconrod jayconrod marked this pull request as ready for review October 24, 2025 18:00
@jayconrod
Copy link
Copy Markdown
Collaborator Author

Got CI to be green at last. Unsure of the syntax I tried a few different things. There's an imports feature, but it looks like we can't import .bcr/presubmit.yml because the BCR container image makes some assumptions about what directories exist.

matrix seems to work though. It only expands tasks if they're actually used, so setting a bazel list there only affects tasks with bazel: ${{ bazel }}.

Also added ubuntu2004_arm64 as one of the matrix platforms because the BCR tests cover that.

@fmeum Mind taking another look? I changed this after you reviewed.

@jayconrod jayconrod merged commit 3ea65d8 into bazel-contrib:master Oct 24, 2025
1 check passed
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.

2 participants