github-actions: Upload kselftests/LTP tarballs; add no_pr dispatch input#1283
Draft
kemotaha wants to merge 3 commits into
Draft
github-actions: Upload kselftests/LTP tarballs; add no_pr dispatch input#1283kemotaha wants to merge 3 commits into
kemotaha wants to merge 3 commits into
Conversation
Three changes to the multi-arch workflow:
1. Two new upload-artifact steps publish the kselftests and LTP install
tree tarballs produced by ctrliq/kernel-container-build's Steps 3.5
and 4.5 (see the paired PR there):
- kselftests-binaries-<arch>: output/kselftests-*.tar.xz
- ltp-binaries-<arch>: output/ltp-*.tar.xz
Both honor the existing skip_kselftests / skip_ltp metadata and use
if-no-files-found: warn so a missing tarball is non-fatal.
2. A new workflow_dispatch input no_pr (bool, default false) lets
manual reruns suppress the PR creation/update step. Useful for
integration testing without producing a spurious "[BASE_BRANCH]
..." PR labeled created-by-kernelci.
3. The create-pr job's if-condition now includes
`(github.event_name != 'workflow_dispatch' || !inputs.no_pr)` so
normal workflow_run events are unaffected and manual dispatches
honor the new opt-out.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
DO NOT MERGE — revert before opening for review. Pins the workflow's kernel-container-build checkout to the extract-test-suite-tarballs branch so workflow_dispatch can exercise the new kselftests/LTP tarball extraction steps end-to-end before ctrliq/kernel-container-build#33 lands on main. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
There was a problem hiding this comment.
Pull request overview
This PR updates the multi-arch kernel build/test GitHub Actions workflow to publish additional test-suite tarball artifacts and allow manual dispatch runs to skip PR creation.
Changes:
- Adds
kselftests-binaries-<arch>andltp-binaries-<arch>artifact uploads. - Adds a
workflow_dispatchno_prinput. - Updates the
create-prjob condition to honorno_prfor manual runs.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| with: | ||
| repository: ctrliq/kernel-container-build | ||
| ref: main | ||
| ref: extract-test-suite-tarballs |
| with: | ||
| repository: ctrliq/kernel-container-build | ||
| ref: main | ||
| ref: extract-test-suite-tarballs |
| with: | ||
| repository: ctrliq/kernel-container-build | ||
| ref: main | ||
| ref: extract-test-suite-tarballs |
| with: | ||
| repository: ctrliq/kernel-container-build | ||
| ref: main | ||
| ref: extract-test-suite-tarballs |
| with: | ||
| repository: ctrliq/kernel-container-build | ||
| ref: main | ||
| ref: extract-test-suite-tarballs |
…-tarballs" This reverts commit 832854d.
Comment on lines
1208
to
+1212
| if: | | ||
| always() && | ||
| needs.build.result == 'success' && | ||
| needs.boot.result == 'success' | ||
| needs.boot.result == 'success' && | ||
| (github.event_name != 'workflow_dispatch' || !inputs.no_pr) |
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
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.
Summary
Follow-on to #1225 (kernel binaries tarball). Three changes to the multi-arch workflow:
upload-artifactsteps publish the kselftests and LTP install-tree tarballs produced by ctrliq/kernel-container-build#33:kselftests-binaries-<arch>—output/kselftests-*.tar.xzltp-binaries-<arch>—output/ltp-*.tar.xzBoth honor existing
skip_kselftests/skip_ltpmetadata; both useif-no-files-found: warnso a missing tarball is non-fatal.workflow_dispatchinputno_pr(bool, defaultfalse) lets manual reruns suppress thecreate-prjob — useful for integration testing without producing a spuriouscreated-by-kernelciPR.workflow_runevents (normal CI) are unaffected.create-prif-condition to honorno_prwhen set on aworkflow_dispatchevent.Order of operations
Land ctrliq/kernel-container-build#33 first. Until that's merged, the upload step will find no file in
output/and log a warning (non-fatal). Once kernel-container-build is producing the tarballs, this PR can land.Cleanup before merge
The
TEST ONLYcommit on this branch (832854d7) pins the workflow'skernel-container-buildcheckout to theextract-test-suite-tarballsbranch so we can exercise the change viaworkflow_dispatchbefore either side merges. Revert that commit before flipping out of draft — or use "Squash and merge" and hand-edit the diff to drop it.Wiki note
The Kernel CI Workflow Confluence page should be updated post-merge to reflect:
kernel-binaries-<arch>,kselftests-binaries-<arch>,ltp-binaries-<arch>)workflow_dispatchusage incl.run_idand the newno_prinputmaininstead ofautomated-testing-v1Test plan
workflow_dispatchagainst this branch withno_pr=trueand a real PR'srun_id: verifykselftests-binaries-<arch>andltp-binaries-<arch>appear in the Artifacts section, and that no PR is created/edited🤖 Generated with Claude Code