David Tuite

Results 35 comments of David Tuite

Merged one pull request for this to get a basic integration test working. Still more to do. - [ ] Assert that the output which is sent to Slack is...

I agree this would be good. It should work for Slack too. There is related discussion in https://github.com/RoadieHQ/kubewise/issues/45. If you're not using `--wait` with Helm, then the second "upgraded" message...

If #45 was done, and assuming you are not using `--wait`, I think this issue (#49) would be a no-op. If upgrading a chart is just a single message then...

> @dtuite How about global way to disable wait behaviour to reduce noice? Yes. This is another option. I presume that most teams will have scripted their helm deployments and...

Interesting. Slightly worried it might get very long, especially if the diff covered multiple files. There's obviously a small cost to having to scroll through too many pages of Slack...

Is sensitive data in the values a concern? There would very likely be Slack API tokens in there for example. You might not want that broadcasted to anyone who was...

Hey @cep21. I made some progress on this. Here's how it looks in Slack. ![Screenshot 2020-04-09 at 22 05 30](https://user-images.githubusercontent.com/562403/78940815-5e1bcd80-7aae-11ea-95f7-1b105a5f3b1d.png) Let me know what you think please. BTW, would you...

I was going to put it behind a flag like `--set showValuesDiff=true` and default it to `false` because I do want to continue to support teams who are not confident...

The ability to output a diff of the values on upgrade or rollback is now released in helm chart version 0.11.1. It is disabled by default. To enable it: `--set...

Apologies for going quiet on this. I've been busy with other things and COVID isn't helping. I'll get back to it but it will take some time.