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

Run script on error #43

Open
ilovethensa opened this issue May 24, 2024 · 4 comments
Open

Run script on error #43

ilovethensa opened this issue May 24, 2024 · 4 comments
Labels
enhancement New feature or request

Comments

@ilovethensa
Copy link

It would allow comin to for example send a notification when the deployment fails

@nlewo
Copy link
Owner

nlewo commented May 25, 2024

There would also be two others ways to achieve this:

  • comin exposes Prometheus metrics: an alertmanager could send an alert
  • wrapping this switch-to-configuration script to detetch its failure and run an action on failure

@nlewo nlewo added the enhancement New feature or request label May 25, 2024
@charlot-shaw
Copy link

I'd also like to have it run something like this on success as well, so I can have it pop a notification or similar. It's very in the background otherwise.

@nlewo
Copy link
Owner

nlewo commented Jul 28, 2024

@charlot-shaw What kind of notifications do you have in mind?

@charlot-shaw
Copy link

Something as simple as popping something up through dunst or mako would be great. I've had some debugging sessions where knowing when comin had finished would have been really helpful.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants