Skip to content

Filter ci matrix by dev-spec channel#624

Merged
bschwedler merged 1 commit into
mainfrom
dev-spec-channel-filter
Jun 18, 2026
Merged

Filter ci matrix by dev-spec channel#624
bschwedler merged 1 commit into
mainfrom
dev-spec-channel-filter

Conversation

@bschwedler

Copy link
Copy Markdown
Contributor

A dispatched dev build puts its channel in the --dev-spec JSON, and the shared workflow then stops passing --dev-channel. bakery ci matrix only filtered by --dev-channel, so a single-channel dispatch still built every channel's dev version. Only the dispatched channel got its version pinned, and the rest resolved to their channel head.

ci matrix now uses the dev-spec channel as the filter when --dev-channel is absent, so the filter and the pin agree.

A dispatched dev build folds its channel into the --dev-spec JSON, and
the shared CI workflow then stops passing --dev-channel. bakery ci matrix
filtered only by the explicit --dev-channel option, so a single-channel
dispatch still emitted every channel's dev version. Only the matching
channel got its version pinned. The others resolved to their channel
head, producing surprise extra builds.

Derive the matrix channel filter from the dev-spec channel when
--dev-channel is absent. This matches the channel that _apply_dev_spec
already uses to pin the version, so the filter and the pin agree.
@bschwedler bschwedler requested a review from ianpittwood as a code owner June 18, 2026 14:15
@github-actions

github-actions Bot commented Jun 18, 2026

Copy link
Copy Markdown

Test Results

1 896 tests  +2   1 896 ✅ +2   6m 57s ⏱️ - 4m 36s
    1 suites ±0       0 💤 ±0 
    1 files   ±0       0 ❌ ±0 

Results for commit 7b7c3bb. ± Comparison against base commit eca9265.

♻️ This comment has been updated with latest results.

@bschwedler bschwedler merged commit c461a2c into main Jun 18, 2026
34 of 42 checks passed
@bschwedler bschwedler deleted the dev-spec-channel-filter branch June 18, 2026 15:28
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.

1 participant