Skip to content

ci: send Slack notifications on CI failures#10729

Merged
anakin87 merged 17 commits intomainfrom
slack-notifications
Mar 6, 2026
Merged

ci: send Slack notifications on CI failures#10729
anakin87 merged 17 commits intomainfrom
slack-notifications

Conversation

@anakin87
Copy link
Member

@anakin87 anakin87 commented Mar 4, 2026

Related Issues

Proposed Changes:

Instead of sending DD events for CI failures and then Slack notifications from DD monitors, skip DD

  • create and publish a composite action (reusable across workflows and repos), based on slackapi/slack-github-action -> https://github.com/deepset-ai/notify-slack-action
  • adopt the composite action in all CI-related workflows
  • (skip for now the release workflow, to be handled in another PR)

To make this work, before opening this PR, I completed the setup described in the Slack documentation.

How did you test it?

In this PR, I temporarily made license_compliance workflow fail and got the expected notification.

image

Notes for the reviewer

From the safety point of view, we should be OK

Checklist

  • I have read the contributors guidelines and the code of conduct.
  • I have updated the related issue with new insights and changes.
  • I have added unit tests and updated the docstrings.
  • I've used one of the conventional commit types for my PR title: fix:, feat:, build:, chore:, ci:, docs:, style:, refactor:, perf:, test: and added ! in case the PR includes breaking changes.
  • I have documented my code.
  • I have added a release note file, following the contributors guidelines.
  • I have run pre-commit hooks and fixed any issue.

@vercel
Copy link

vercel bot commented Mar 4, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
haystack-docs Ignored Ignored Preview Mar 6, 2026 6:56am

Request Review

@anakin87 anakin87 marked this pull request as ready for review March 4, 2026 16:50
@anakin87 anakin87 requested a review from a team as a code owner March 4, 2026 16:50
@anakin87 anakin87 requested review from bogdankostic and removed request for a team March 4, 2026 16:50
@anakin87 anakin87 changed the title try to notify slack on CI failures ci: send Slack notifications on CI failures Mar 4, 2026
@anakin87 anakin87 self-assigned this Mar 4, 2026
@github-actions github-actions bot added topic:core type:documentation Improvements on the docs labels Mar 6, 2026
@coveralls
Copy link
Collaborator

Pull Request Test Coverage Report for Build 22752365539

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 28 unchanged lines in 1 file lost coverage.
  • Overall coverage remained the same at 92.768%

Files with Coverage Reduction New Missed Lines %
core/pipeline/async_pipeline.py 28 71.18%
Totals Coverage Status
Change from base Build 22752247302: 0.0%
Covered Lines: 15610
Relevant Lines: 16827

💛 - Coveralls

@anakin87 anakin87 merged commit 22ef01a into main Mar 6, 2026
13 checks passed
@anakin87 anakin87 deleted the slack-notifications branch March 6, 2026 10:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants