Skip to content

ci(.github/workflows): alert Slack when the dogfood image build fails on main - #31710

Draft
mafredri wants to merge 1 commit into
mainfrom
mafredri/dogfood-failure-slack
Draft

mafredri wants to merge 1 commit into
mainfrom
mafredri/dogfood-failure-slack

Conversation

@mafredri

@mafredri mafredri commented Nov 3, 2026

Copy link
Copy Markdown
Member

The dogfood image workflow had no failure alert. Its runs on main on 2026-11-02 and 2026-11-03 both failed on network errors and weren't re-run, so workspaces kept pulling the 2026-10-27 image until a manual re-run on 2026-11-03.

This adds the notify-slack-on-failure job from ci.yaml and nightly-gauntlet.yaml with its own header. It posts to the same CI_FAILURE_SLACK_WEBHOOK and mentions the same bot with BLINK_CI_FAILURE_PROMPT, as the nightly gauntlet already does. It runs only on main, the only ref that pushes the image.

Decision log
  • Reused prompt: I couldn't read BLINK_CI_FAILURE_PROMPT (the API returns 403), so I don't know its wording. The nightly gauntlet reuses it for a non-CI failure, so this follows that.
  • permissions: {}: the job only calls a webhook, so it gets no token permissions instead of inheriting the workflow's contents: read.
  • Checked locally: rendered the step's payload with curl stubbed, and it parses as JSON, including a prompt with quotes. make pre-commit-light (what the hook runs for this change, with actionlint) passes, and zizmor 1.11.0 with the Makefile's flags reports no findings (run in ubuntu:26.04, because the local binary needs a newer glibc).
  • Not checked: the job runs only after a failure on main, so this PR's CI doesn't exercise it.

🤖 This PR was created with the help of Coder Agents.

… on main

The dogfood image publish failed on main on 2026-11-02 and 2026-11-03
without anyone noticing, so workspaces kept pulling the 2026-10-27
image. Post the same failure alert as ci.yaml and nightly-gauntlet.yaml.

This branch has not been deployed

No deployments
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