ci: refactor pkg workflow release pipeline#174
Conversation
2278563 to
8a56e68
Compare
There was a problem hiding this comment.
Reduced the name length for better visibility in the UI.
The resolve-suite job is not necessary to happen here anymore, its all done in the pkg-build-reusable-workflow
There was a problem hiding this comment.
- Reduced the name length for better visibility in the UI.
- Suite is no longer an input parameter, its infered from branch name
There was a problem hiding this comment.
Reduced the name length for better visibility in the UI.
| set -euo pipefail | ||
|
|
||
| target_suite="$SUITE_INPUT" | ||
| normalize_ref() { |
There was a problem hiding this comment.
It will be once this lands. Immediately after I'll factorize into scripts, I am of the opinion too that its growing out of control, but wanna keep the refactor separate in this PR
There was a problem hiding this comment.
- Reduced the name length for better visibility in the UI.
- Deleted default branch name since since its error prone, a user can easily forget to change the default which is almost never the right choice anyway.
There was a problem hiding this comment.
- Reduced the name length for better visibility in the UI.
- Suite is no longer an input parameter, its infered from branch name
- Deleted default branch name since since its error prone, a user can easily forget to change the default which is almost never the right choice anyway.
- Added QSC_API_KEY secret passing here since we do the qartifactory upload from the pkg-release-reusable-workflow itself as opposed to before where we had a hop from this repo to qcom-distro-images
There was a problem hiding this comment.
No more default branch, need the debian-branch explicitely
There was a problem hiding this comment.
Adapt to sync from the .github/pkg-workflows/qli-ci instead of main folder
There was a problem hiding this comment.
Reformulated the huge promotion PR body text to be shorted and concised
| run: | ||
| shell: bash | ||
| steps: | ||
| - name: Resolve family and suite from branch |
There was a problem hiding this comment.
Duplicate code. a common script will help
c0938e6 to
177ffb5
Compare
Move pkg workflow caller templates from main/ to qli-ci/. Update template names, job labels, and sync path references. Signed-off-by: Simon Beaudoin <sbeaudoi@qti.qualcomm.com>
Derive family and suite from debian-ref branch naming. Remove finalize passthrough and gate Test on selected build. Signed-off-by: Simon Beaudoin <sbeaudoi@qti.qualcomm.com>
Run source prep before test build and gate family release jobs. Flatten Ubuntu upload path and unify post-release persistence. Signed-off-by: Simon Beaudoin <sbeaudoi@qti.qualcomm.com>
Require explicit debian-branch input across promote workflow calls. Polish and shorten autogenerated promotion PR messaging. Signed-off-by: Simon Beaudoin <sbeaudoi@qti.qualcomm.com>
Document the qcom/<family>/<suite> branch parsing model. Refresh workflow docs and agent guidance for new interfaces. Signed-off-by: Simon Beaudoin <sbeaudoi@qti.qualcomm.com>
Add shared branch family/suite resolver script. Reuse it in pkg-build and pkg-release workflows. Update docs for last-two-segments parsing rule. Signed-off-by: Simon Beaudoin <sbeaudoi@qti.qualcomm.com>
Remove srcpkg-artifact support from pkg-build-reusable-workflow. Keep build source flow branch-based via debian-ref only. Update pkg-release wiring and reusable workflow docs. Signed-off-by: Simon Beaudoin <sbeaudoi@qti.qualcomm.com>
Remove unused job-index plumbing from build reusable workflow. Set retention-days on upload-artifact steps and sync docs. Signed-off-by: Simon Beaudoin <sbeaudoi@qti.qualcomm.com>
951e833 to
f81f977
Compare
Run the Debian release job container as root so checkout can write runner command files under /__w/_temp without EACCES. Signed-off-by: Simon Beaudoin <sbeaudoi@qti.qualcomm.com>
Remove .github/TO_PASTE_IN_UPSTREAM_REPO during workflow sync for both default and managed packaging branches. Also stage .github/ changes so folder deletions are committed. Signed-off-by: Simon Beaudoin <sbeaudoi@qti.qualcomm.com>
Delete any workflow file containing pull_request_target during workflow sync on default, packaging, and other branches. Signed-off-by: Simon Beaudoin <sbeaudoi@qti.qualcomm.com>
Enumerate refs from refs/remotes/origin/* to avoid treating the parent remote ref as a branch named origin. Keep an explicit origin guard in both branch lists. Signed-off-by: Simon Beaudoin <sbeaudoi@qti.qualcomm.com>
Summary
.github/pkg-workflows/main/to.github/pkg-workflows/qli-ci/and update sync wiring. (since default branch is now qli-ci, not main)Build,Release,Promote,Promote Prebuilt,PR Build).suiteinputs in build/release flows by derivingfamilyandsuitefromdebian-ref(*/<family>/<suite>convention).pkg-build-reusable-workflowoutputs by removing the passthroughfinalizejob and exposing producer outputs directly.Testjob inpkg-build-reusable-workflowon selected build pathsuccess (Debian Debusine path vs Ubuntu/forced Docker path).
pkg-release-reusable-workflowto:debian-branchinputs andshortening autogenerated promotion PR text.
AGENTS.md,README.md,docs/reusable-workflows.md) for theupdated branch convention and workflow interfaces.
Commit Stack
b6abcedAlign caller templates to qli-ci7b0f683Refactor build reusable resolution5fea4f6Refactor release reusable workflowd416120Tighten promote workflow inputs8a56e68Update docs for branch conventionsValidation