Skip to content

Conversation

@leighmcculloch
Copy link
Member

What

Add a notification job to the CI workflow that sends Slack messages when builds fail on the main branch. Implement caching to prevent duplicate notifications for the same image configuration.

Why

Teams need immediate visibility into CI failures on the main branch to respond quickly to broken builds without being spammed by repeated notifications.

@github-project-automation github-project-automation bot moved this to Backlog (Not Ready) in DevX Oct 29, 2025
@leighmcculloch leighmcculloch marked this pull request as ready for review October 29, 2025 04:40
@leighmcculloch leighmcculloch enabled auto-merge (squash) October 29, 2025 04:40
@fnando fnando requested a review from Copilot October 29, 2025 21:38
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds a Slack notification job to the CI workflow that sends alerts when builds fail on the main branch. The notification system includes a caching mechanism to prevent duplicate notifications for the same failure state.

Key changes:

  • Added a new notify job that triggers on failure of any dependent job
  • Implemented notification caching to avoid duplicate Slack messages for the same configuration
  • Integrated Slack webhook notification with build status emojis

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@leighmcculloch leighmcculloch requested a review from a team October 29, 2025 21:53
@leighmcculloch leighmcculloch merged commit 1ddef56 into main Oct 29, 2025
95 checks passed
@leighmcculloch leighmcculloch deleted the add-slack-notification-on-failure branch October 29, 2025 21:56
@github-project-automation github-project-automation bot moved this from Backlog (Not Ready) to Done in DevX Oct 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants