Skip to content

.github: Report dispatch failures on Slack#410

Merged
joestringer merged 1 commit into
mainfrom
pr/joe/notify-on-dispatch-fail
May 6, 2026
Merged

.github: Report dispatch failures on Slack#410
joestringer merged 1 commit into
mainfrom
pr/joe/notify-on-dispatch-fail

Conversation

@joestringer
Copy link
Copy Markdown
Member

The credentials for dispatching a request to the team-management
repository in order to apply these settings to the organization
periodically expire. When they expire, this workflow starts failing, but
there is no corresponding notification.

These credentials have a different lifecycle compared to the Slack
notification, so probably it could be helpful to have a backup system
here where this workflow posts to Slack when there's a failure. Then
maintainers can monitor the same Slack channel for approval requests and
infrastructure failures (including credential expiry).

Add a new Slack app using the existing webhook URL to post a message
when the dispatch fails.

The credentials for dispatching a request to the team-management
repository in order to apply these settings to the organization
periodically expire. When they expire, this workflow starts failing, but
there is no corresponding notification.

These credentials have a different lifecycle compared to the Slack
notification, so probably it could be helpful to have a backup system
here where this workflow posts to Slack when there's a failure. Then
maintainers can monitor the same Slack channel for approval requests and
infrastructure failures (including credential expiry).

Add a new Slack app using the existing webhook URL to post a message
when the dispatch fails.

Signed-off-by: Joe Stringer <joe@cilium.io>
@joestringer joestringer requested a review from a team as a code owner May 6, 2026 22:00
@joestringer joestringer merged commit 96d548e into main May 6, 2026
1 check passed
@joestringer joestringer deleted the pr/joe/notify-on-dispatch-fail branch May 6, 2026 22:01
@joestringer
Copy link
Copy Markdown
Member Author

joestringer commented May 6, 2026

Task successfully failed and notified per the new logic: https://github.com/cilium/community/actions/runs/25463581617/job/74711497340 (+ expected notification on Slack)

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