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.

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

add alert details using slack block kit #1751

Merged

Conversation

russellbrooks
Copy link
Contributor

Adds more details to the slack webhook alerts sent when using --notify-on-error or --notify-on-success using Slack's block kit as part of the argo workflow template creation

Before the messages would simply be:

🚨 TestFlow/argo-testflow-z7mtw failed!

Now they include a link to the UI and conditionally add @project metadata when available, e.g. triggering an argo workflow created from the following:

from metaflow import FlowSpec, project, step


@project(name="test")
class TestFlow(FlowSpec):
    @step
    def start(self):
        self.next(self.end)

    @step
    def end(self):
        pass


if __name__ == "__main__":
    TestFlow()

CleanShot 2024-02-20 at 18 05 33

@savingoyal savingoyal merged commit 1bacc31 into Netflix:master Feb 21, 2024
30 of 34 checks passed
@russellbrooks russellbrooks deleted the russell/slack-message-details branch February 21, 2024 03:13
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