DEVPROD-4021: conditional runs for non-Buildkite affecting CI checks#1490
Conversation
|
|
ivotron
force-pushed
the
devprod-4021-conditional-ci-runs
branch
from
1be49b3 to
aadd47a
Compare
ivotron
requested review from
RafalKorepta,
andrewstucki,
chrisseto,
gene-redpanda and
hidalgopl
as code owners
|
@claude review once |
There was a problem hiding this comment.
Pull request overview
Adds a Buildkite entrypoint script to conditionally skip the heavy operator CI test suite on PRs that only touch documentation or GitHub workflows, while still reporting passing GitHub commit statuses for required checks.
Changes:
- Introduce
.buildkite/scripts/conditional_testsuite.shto detect PR-only doc/GitHub workflow changes and upload either a dummy “pass” pipeline or the full testsuite pipeline. - Update
.buildkite/pipeline.ymlci-entry-pointstep to run the new conditional script instead of directly uploading.buildkite/testsuite.yml.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
| .buildkite/scripts/conditional_testsuite.sh | New PR change-detection logic and dummy pipeline uploader for doc/workflow-only changes. |
| .buildkite/pipeline.yml | Switch CI entrypoint to use the conditional testsuite uploader script. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Adds a Buildkite entrypoint script that detects PRs which only touch docs, GitHub workflows, or root-level Markdown and skips the heavy operator test suite. A dummy pipeline is uploaded instead, which reports passing GitHub commit statuses for every required check by extracting the context list straight from `.buildkite/testsuite.yml` (scoped to `context:` keys under `github_commit_status:` so plugin contexts like junit-annotate are excluded). The dummy step pins the same agent queue as the entrypoint so it actually gets scheduled. Diff parsing is NUL-delimited and arguments are quoted to handle paths with whitespace and unusual branch names safely.
ivotron
force-pushed
the
devprod-4021-conditional-ci-runs
branch
from
155d4df to
95787b7
Compare
|
This PR is stale because it has been open 5 days with no activity. Remove stale label or comment or this will be closed in 5 days. |
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation and see the Github Action logs for details |
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation and see the Github Action logs for details |
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation and see the Github Action logs for details |
Summary
This introduces a script that checks what files have changed in a PR.
If only docs or github workflows changed, it skips the heavy test suite
and simply generates dummy passing statuses for all required checks,
saving CI resources.
Ref: https://redpandadata.atlassian.net/browse/DEVPROD-4021