Skip to content

feat(bonedigger): add clanker-queue migration workflow - #33

Open
castrojo wants to merge 1 commit into
mainfrom
feat/clanker-queue-rollout
Open

feat(bonedigger): add clanker-queue migration workflow#33
castrojo wants to merge 1 commit into
mainfrom
feat/clanker-queue-rollout

Conversation

@castrojo

Copy link
Copy Markdown
Contributor

Adds the clanker-queue migration namespace to bonedigger:

  • new canonical labels.json with clanker-queue/* labels
  • new sync-labels.yml to propagate labels.json to the factory repos
  • dual-write legacy labels and clanker-queue labels in the reusable lifecycle workflow during the soak
  • updated repo docs and skills to describe the new ownership model

This is the rollout branch that the downstream bonedigger callers pin to.

- add a canonical labels.json with the new clanker-queue/* namespace
- add sync-labels.yml to propagate labels.json to the factory repos
- dual-write legacy labels and clanker-queue labels in the reusable lifecycle workflow during the soak
- update the repo docs and skills to describe the new ownership model

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

@hanthor hanthor left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Two rollout invariants need fixing. First, labels.json defines clanker-queue/p0 and p1 as replacements for cycle-wide hive priorities, while lifecycle.yml applies them at the repo-scoped priority/p0 and priority/p1 confirmation thresholds. That collapses two distinct severities without a consistent mapping. Second, the legacy and clanker labels are added in one gh issue edit guarded by a single or-true; before sync-labels has created the new labels downstream, the missing clanker label can make the whole edit fail and suppress the legacy update too. Please define one consistent priority mapping and make dual writes independent or otherwise guarantee the new labels exist before any pinned caller can execute. The workflow PR also needs the required actionlint and pre-commit verification evidence.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4-review A pull request is awaiting review.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants