Add monitoring of container image deployments

wongnai / kube-slack

Kubernetes Slack Monitoring

MIT License

323 stars 59 forks source link

Add monitoring of container image deployments #13

Open nathancashmore opened 6 years ago

phillipj commented 6 years ago

@nathancashmore would you mind if I hi-jacked this PR and fixed the raised review issues? I'd really appreciate having the bot notify us about new deployments!

nathancashmore commented 6 years ago

@phillipj go for it.

phillipj commented 6 years ago

Alrighty, thanks. I'll for sure credit you as the author in the commit!

phillipj commented 6 years ago

Updates after having done some initial testing of this; we decided to send a Slack message from our deployment script with a cURL command instead. Looks something like this:

That in combination with the already existing monitors that will tell us about containers having trouble, covers our needs.

Bottom line, I was a little quick saying I'll hijack this PR with the goals of getting it merged. Sorry for those who waited in vain.

AshMartian commented 5 years ago

This may be a little unnecessary for some, this should be conditional with a configuration flag.