Skip to content

ci: send Slack notification on nightly failures + modernize workflows#434

Merged
anakin87 merged 11 commits intomainfrom
notify-slack
Mar 6, 2026
Merged

ci: send Slack notification on nightly failures + modernize workflows#434
anakin87 merged 11 commits intomainfrom
notify-slack

Conversation

@anakin87
Copy link
Member

@anakin87 anakin87 commented Mar 6, 2026

I initially worked on this as part of https://github.com/deepset-ai/haystack-private/issues/256: Instead of sending DD events for CI failures and then Slack notifications from DD monitors, skip DD.

For more details, see the similar PR deepset-ai/haystack#10729.

Then I realized that these workflows can be modernized and improved, so I did the following:

  • merge nightly and run_tutorials that were doing the same (running tutorials), but the first on scheduled nightly runs and the second on PRs
  • use Ubuntu slim (cheaper) for simple jobs (computing test matrices)
  • use newer versions of GitHub actions
  • use uv for installations and stop using containers (uv pip install haystack-ai is fast)
  • stop installing system dependencies: they are no longer needed; they probably were in the past, partly based on v1 tutorials, partly because Haystack container comes with limited system libraries compared to Ubuntu GitHub runners

I tested this on this PR, by triggering temporary changes. Works correctly.

In addition, installation times are impressively reduced. An example for tutorial 27: 32s vs 140s!

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@anakin87 anakin87 changed the title ci: send Slack notification on nightly failures + modernize workflow ci: send Slack notification on nightly failures + modernize workflows Mar 6, 2026
@anakin87 anakin87 marked this pull request as ready for review March 6, 2026 11:23
@anakin87 anakin87 requested a review from a team as a code owner March 6, 2026 11:23
@anakin87 anakin87 self-assigned this Mar 6, 2026
@anakin87 anakin87 requested a review from kacperlukawski March 6, 2026 11:23
Copy link
Member

@kacperlukawski kacperlukawski left a comment

Choose a reason for hiding this comment

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

Nice work! Left just a minor comment, but it's good to merge. Btw, does that mean we are also going to save some money due to deduplication?

name: Run Tutorials

on:
workflow_dispatch: # Activate this workflow manually
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
workflow_dispatch: # Activate this workflow manually
workflow_dispatch:

Isn't the comment outdated?

Copy link
Member Author

Choose a reason for hiding this comment

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

this is a just a comment that informs that using this event, you can activate the workflow manually
More info: https://docs.github.com/en/actions/reference/workflows-and-actions/events-that-trigger-workflows#workflow_dispatch

Currently, in https://github.com/deepset-ai/haystack-tutorials/actions/workflows/nightly.yml there is this button
image

@anakin87 anakin87 merged commit fba3e04 into main Mar 6, 2026
@anakin87 anakin87 deleted the notify-slack branch March 6, 2026 14:27
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.

2 participants