Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Automated workflow should not fail due to Slack outage #493

Closed
joverlee521 opened this issue Feb 27, 2025 · 1 comment · Fixed by #494
Closed

Automated workflow should not fail due to Slack outage #493

joverlee521 opened this issue Feb 27, 2025 · 1 comment · Fixed by #494

Comments

@joverlee521
Copy link
Contributor

Yesterday's GenBank fetch and ingest workflow failed during the Slack outage.

As said in nextstrain/ncov#1173 (comment)

Ideally the workflow would recovery gracefully from errors messaging Slack instead of crashing out. :/
@joverlee521
Copy link
Contributor Author

All Slack notification scripts use the vendored ingest/notify-slack since a82898b, so this should probably be a change in the centralized ingest/notify-slack.

I'm already modifying the script for nextstrain/ingest#46, so I might as well tack this on.

This was referenced Feb 27, 2025
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Update vendored/notify-slack nextstrain/ncov-ingest
1 participant
@joverlee521