unify pkg workflows: remove standalone debusine callers#188
unify pkg workflows: remove standalone debusine callers#188Simon Beaudoin (simonbeaudoin0935) wants to merge 5 commits into
Conversation
2ac6b2d to
f72dfd3
Compare
Introduce unified PR bridge templates and align Debian/Ubuntu build-release behavior for the reusable workflows. Keep Docker test path on pkg-builder while Debian routes through Debusine-aware resolution and release metadata. Signed-off-by: Simon Beaudoin <sbeaudoi@qti.qualcomm.com>
Restrict trigger wiring to PR Build Hook and keep hook/check names consistent with the unified PR Build context. Replace github-script usage with bash curl API calls for commit-status updates. Signed-off-by: Simon Beaudoin <sbeaudoi@qti.qualcomm.com>
Make workflows sync explicitly include pkg-pr-check.yml and delete standalone debusine-* workflow files. Apply the same cleanup on default and managed packaging branch sync passes. Signed-off-by: Simon Beaudoin <sbeaudoi@qti.qualcomm.com>
Rename promotion PR branches from debian/pr/* to promotion-pr/* in source and prebuilt promote flows. Update helper script, docs, and sync filtering to match the new promotion branch namespace. Signed-off-by: Simon Beaudoin <sbeaudoi@qti.qualcomm.com>
e39282b to
d9718cb
Compare
There was a problem hiding this comment.
This PR does not remove Debian Debusine build/release behavior.
It removes duplicate standalone caller workflow files and routes PR/build/release entry points through the unified pkg-* flow.
Sorry, I don't understand. I'm still actively maintaining these workflows and tech teams are currently actively using them. I agree with converging them. But what's the migration plan here?
|
|
Have you checked that everything implemented by the existing Debusine flows is correctly available in the flows that will replace them? How are you validating this? This is a hard prerequisite. When we discussed this the other day we said I'd do this, but I'm working on the priority ordered list that we agreed at the time, and we haven't got down this far yet. Repository configuration management was a higher priority. So I haven't looked at this yet. |
Problem
Some
pkg-*repos still carry standalonedebusine-*caller workflows from the earlier siloed integration, while current org-widepkg-*workflows already support the unified model:This leaves duplicate entry points and mixed conventions across repos.
Goal of this PR
Make the
pkg-*workflow path the single org-wide flow and remove legacydebusine-*caller workflow drift, while preserving the behavior needed for Debian (Debusine) and Ubuntu (Docker) paths.What This PR Changes
1) Unified PR bridge and build/release parity
.github/pkg-workflows/debian/pkg-pr-hook.yml.github/pkg-workflows/qli-ci/pkg-pr-check.ymlpkg-pr-checknow triggers only fromPR Build Hook.PR Build(no Debusine-only wording).actions/github-scriptstatus posting with bash + GitHub REST API (curl).latest -> forkyDebusine mapping and Docker fallback handling)source-refhandling for PR merge refsdebusine_target_workspace,package_version_identifier)2) Workflow sync migration behavior
.github/workflows/workflows_sync.ymlto:pkg-pr-check.ymlwith default-branch workflow templatesdebusine-*.yml/debusine-*.yamlduring sync3) Promotion PR branch namespace cleanup
debian/pr/<version>-1promotion-pr/<version>-1.github/workflows/pkg-promote-reusable-workflow.yml.github/workflows/pkg-promote-prebuilt-reusable-workflow.ymlscripts/create_promotion_pr.pyExplicit Parity Notes (for legacy Debusine workflow review)
pkg-*flow.Non-goals
Commit Structure (cleaned)
pkg: unify PR bridge and build-release paritypkg-pr: finalize hook/check status wiringsync: remove legacy debusine workflowspromote: use promotion-pr branch prefix