Skip to content

Notify on merge queue failures#576

Draft
bschwedler wants to merge 2 commits into
mainfrom
feat/merge-queue-slack-notifications
Draft

Notify on merge queue failures#576
bschwedler wants to merge 2 commits into
mainfrom
feat/merge-queue-slack-notifications

Conversation

@bschwedler

Copy link
Copy Markdown
Contributor

The Slack notification action previously skipped silently when no prior
run history was found. Merge queue runs on ephemeral refs never appear
in a main-branch run query, so that guard was swallowing all merge queue
failures with no notification.

Now: no history + passing → still skip. No history + failing → notify.

The action previously skipped silently when no previous workflow
runs were found on main. This prevented merge queue failures from
triggering notifications, since merge queue runs land on ephemeral
refs and never appear in a main-branch run query.

Now: no history + failed → fire failure notification; no history +
passed → skip. State-transition logic is unchanged for builds with
history.
@github-actions

github-actions Bot commented Jun 8, 2026

Copy link
Copy Markdown

Test Results

1 598 tests  +2   1 598 ✅ +2   8m 16s ⏱️ -1s
    1 suites ±0       0 💤 ±0 
    1 files   ±0       0 ❌ ±0 

Results for commit 080f188. ± Comparison against base commit b3da13e.

♻️ This comment has been updated with latest results.

Extend `bakery ci matrix` with a `--latest` flag that mirrors the flag
already on `bakery build`. For regular versions it keeps only those
with `latest: true`; for matrix images it keeps only the
highest-version combination (the row where `ImageVersion.latest` is
set by `_matches_latest`).

Add a `latest-only` boolean input to `bakery-build-pr.yml` that
passes `--latest` to the matrix generation steps, so calling
workflows can opt individual event types into a faster subset run.

Add a `multiversion` test context and BDD scenarios that verify the
flag filters non-latest versions from the output.
bschwedler added a commit to posit-dev/images-workbench that referenced this pull request Jun 8, 2026
latest-only is not yet available in bakery-build-pr.yml@main.
Revert this commit and squash with the previous one once
posit-dev/images-shared#576 is merged.
bschwedler added a commit to posit-dev/images-workbench that referenced this pull request Jun 8, 2026
Points bakery-build-pr.yml at feat/merge-queue-slack-notifications
so latest-only can be tested before images-shared#576 merges.

Once posit-dev/images-shared#576 is merged,
drop this commit and restore @main refs in the previous commit.
bschwedler added a commit to posit-dev/images-workbench that referenced this pull request Jun 8, 2026
Points bakery-build-pr.yml and setup-bakery (via version:) at
feat/merge-queue-slack-notifications so latest-only can be tested
before images-shared#576 merges.

Once posit-dev/images-shared#576 is merged,
drop this commit and restore @main refs in the previous commit.
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