ci: build-test PRs with existing pipeline in main branch - #77
Merged
Christopher Obbard (obbardc) merged 1 commit intoAug 28, 2026
Merged
Conversation
Christopher Obbard (obbardc)
force-pushed
the
wip/obbardc/fix-pr-workflow
branch
from
August 27, 2026 19:27
07961f9 to
8ed08a1
Compare
Contributor
|
All we need is a premerge yml in qcom/debian/latest (example) that calls to daily yml on pull_request, and daily needs to have workflow_call inputs to be used as reusuable workflow. Daily yml then will automatically parse all matrix variants specifications, and it will delegate to build-kernel-deb, which then delegates to build-kernel-debusine (debian family) AND build-kernel-ubuntu (ubuntu family). To initiate the whole chain, all is needed is a call to daily. |
Christopher Obbard (obbardc)
force-pushed
the
wip/obbardc/fix-pr-workflow
branch
from
August 27, 2026 20:10
8ed08a1 to
71e8232
Compare
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
Christopher Obbard (obbardc)
force-pushed
the
wip/obbardc/fix-pr-workflow
branch
from
August 27, 2026 20:34
c7dc24b to
7df95c4
Compare
Christopher Obbard (obbardc)
changed the title
Christopher Obbard (obbardc)
temporarily deployed
to
Production
August 27, 2026 21:13 — with 
GitHub Actions
Inactive
GitHub Actions
Inactive
Christopher Obbard (obbardc)
temporarily deployed
to
Production
August 27, 2026 21:13 — with
GitHub Actions
Inactive
GitHub Actions
Inactive
Christopher Obbard (obbardc)
temporarily deployed
to
Production
August 27, 2026 21:13 — with
GitHub Actions
Inactive
GitHub Actions
Inactive
Christopher Obbard (obbardc)
deployed
to
Production
August 27, 2026 21:14 — with
GitHub Actions
Active
GitHub Actions
Active
Christopher Obbard (obbardc)
force-pushed
the
wip/obbardc/fix-pr-workflow
branch
2 times, most recently
from
August 27, 2026 23:12
82ea74d to
e9acde3
Compare
Christopher Obbard (obbardc)
requested a review
from Bjordis Collaku (bjordiscollaku)
August 27, 2026 23:13
Christopher Obbard (obbardc)
changed the title
Christopher Obbard (obbardc)
temporarily deployed
to
Staging
August 27, 2026 23:18 — with
GitHub Actions
Inactive
GitHub Actions
Inactive
Christopher Obbard (obbardc)
temporarily deployed
to
Staging
August 27, 2026 23:19 — with
GitHub Actions
Inactive
GitHub Actions
Inactive
Christopher Obbard (obbardc)
temporarily deployed
to
Staging
August 27, 2026 23:19 — with
GitHub Actions
Inactive
GitHub Actions
Inactive
Christopher Obbard (obbardc)
temporarily deployed
to
Staging
August 27, 2026 23:20 — with
GitHub Actions
Inactive
GitHub Actions
Inactive
Contributor
|
No concurrency group on this one, so every push to a PR fires another 5 kernel builds and the in-flight ones keep running on the self-hosted pool. The example from the premerge on pkg-linux-qcom-canonical guards it with: concurrency:
group: pr-build-${{ github.event.pull_request.number }}
cancel-in-progress: trueWorth adding here too? |
pkg-pr-hook.yml delegated to pkg-build-reusable-workflow.yml, which runs generate-source-package immediately after checkout. That needs a real debian/changelog, but this branch ships changelog.in and only "debian/rules prepare" generates it, inside a kernel tree, since it reads the version from the kernel Makefile. So the build never started. The main branch already has a workflow that does all of this: daily.yml already runs it nightly. Reuse that workflow for PRs. Closes: #75 Signed-off-by: Christopher Obbard <[email protected]>
Christopher Obbard (obbardc)
force-pushed
the
wip/obbardc/fix-pr-workflow
branch
from
August 28, 2026 03:30
e9acde3 to
a701a25
Compare
Christopher Obbard (obbardc)
temporarily deployed
to
Staging
August 28, 2026 03:36 — with
GitHub Actions
Inactive
GitHub Actions
Inactive
Christopher Obbard (obbardc)
temporarily deployed
to
Staging
August 28, 2026 03:36 — with
GitHub Actions
Inactive
GitHub Actions
Inactive
Christopher Obbard (obbardc)
temporarily deployed
to
Staging
August 28, 2026 03:37 — with
GitHub Actions
Inactive
GitHub Actions
Inactive
Christopher Obbard (obbardc)
temporarily deployed
to
Staging
August 28, 2026 03:37 — with
GitHub Actions
Inactive
GitHub Actions
Inactive
Christopher Obbard (obbardc)
marked this pull request as ready for review
August 28, 2026 07:34
Bjordis Collaku (bjordiscollaku)
approved these changes
Aug 28, 2026
Bjordis Collaku (bjordiscollaku)
left a comment
Bjordis Collaku (bjordiscollaku)
left a comment
Contributor
There was a problem hiding this comment.
Looks good.
Christopher Obbard (obbardc)
merged commit Aug 28, 2026
8cd25bb
into
qcom/debian/latest
32 checks passed
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
3 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.
Closes: #75