Skip to content

sync: fix packaging branch discovery in workflows sync#197

Open
Simon Beaudoin (simonbeaudoin0935) wants to merge 2 commits into
mainfrom
fix/workflows-sync-detect-packaging-branches
Open

sync: fix packaging branch discovery in workflows sync#197
Simon Beaudoin (simonbeaudoin0935) wants to merge 2 commits into
mainfrom
fix/workflows-sync-detect-packaging-branches

Conversation

@simonbeaudoin0935

Copy link
Copy Markdown
Contributor

Summary

  • Fix branch discovery in Workflows Sync so nested remote branches are
    included during sync passes.

Root cause

workflows_sync.yml used:

  • git for-each-ref ... refs/remotes/origin/*

That pattern only matches top-level refs and skips nested branch paths
such as:

  • debian/qcom-next
  • qcom/ubuntu/resolute

Because of that, the packaging-branch pkg-pr-hook.yml sync loop could be
empty, so no packaging-branch PRs were created.

Change

  • Use refs/remotes/origin (no trailing /*) in both branch collectors:
    • non-packaging cleanup branch list
    • packaging branch list
  • Keep existing filters; add HEAD exclusion in packaging list for
    consistency.

Validation

Validated the updated filters against pkg-urm-ext remote refs. The
packaging list now correctly includes:

  • debian/qcom-next
  • qcom/ubuntu/resolute
  • qcom/ubuntu/resolute-backup

This unblocks Workflows Sync from creating pkg-pr-hook sync PRs for the
packaging branches.

Workflows Sync currently enumerates refs with:
refs/remotes/origin/*

That only matches top-level refs and skips nested branches such as:
- debian/qcom-next
- qcom/ubuntu/resolute

As a result, the packaging-branch pkg-pr-hook sync loop is empty and
no PRs are created for packaging branches.

Use refs/remotes/origin instead so all origin remote branches are seen,
including nested branch paths.

Signed-off-by: Simon Beaudoin <sbeaudoi@qti.qualcomm.com>
Skip lfs/archive refs in the non-packaging cleanup pass.
Handle clone/checkout/push/PR failures per branch and continue.
Print a warning summary so the run still lists completed PRs.

Signed-off-by: Simon Beaudoin <sbeaudoi@qti.qualcomm.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants