Skip to content

ci: add notify-team step for integration test workflow #295

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
May 7, 2025

Conversation

VeraChristina
Copy link
Collaborator

@VeraChristina VeraChristina commented May 1, 2025

Description

Adds a step to the nightly integration test workflow that adds a comment to #294 and tags anemoisecurity for broader visibility of failed nightly runs. Currently, only the person that created the workflow gets notified.

The first comment on the issue (which I edited to avoid confusion) was created by this test run, which simulated a failure: https://github.com/ecmwf/anemoi-core/actions/runs/14778582874

@VeraChristina VeraChristina requested a review from a team as a code owner May 1, 2025 15:58
@github-actions github-actions bot added the CI/CD label May 1, 2025
@VeraChristina VeraChristina added the ATS Approval Not Needed No approval needed by ATS label May 2, 2025
theissenhelen
theissenhelen previously approved these changes May 7, 2025
@VeraChristina VeraChristina merged commit 256949e into main May 7, 2025
20 checks passed
@VeraChristina VeraChristina deleted the ci/add-notification-for-nightly-failures branch May 7, 2025 16:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ATS Approval Not Needed No approval needed by ATS CI/CD
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants