-
Notifications
You must be signed in to change notification settings - Fork 6
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
Output helm values diff #35
Comments
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 (or whatever the tool is) messages. There are ways to solve that problem of course. I've noticed that both Slack and Google Chat have a concept of threaded messages. A quick Google tells me that Microsoft Teams has this feature too. We could put the main notification in the channel and then add supplementary data in a thread on the message. What are your thoughts on how this might work? |
Threads work fine. For our use cases, the NOTES of a helm chart are interesting when a chart is first installed, but not so helpful when a chart is updated. What people really want to see is how the helm values change (and the helm chart version itself). Usually helm values change very little for our case |
To clarify, I'm talking about |
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 in your Slack channel. |
We follow a best practice of never putting sensitive values in helm charts. They are usually secrets that are created with either https://github.com/godaddy/kubernetes-external-secrets or https://github.com/bitnami-labs/sealed-secrets |
Hey @cep21. I made some progress on this. Here's how it looks in Slack. Let me know what you think please. BTW, would you mind emailing me at [email protected] please? |
Looks awesome! I'm guessing it's configurable somehow? |
I was going to put it behind a flag like What other configuration do you have in mind? The example I have shown uses the https://github.com/pmezard/go-difflib library to render the diff. Unfortunately that package is not maintained and hasn't been updated in 4 years. I couldn't find anything else which had that nice |
A parameter with default false sounds reasonable. I'm unsure if this is too much configuration, but this would be perfect for us except I just realized that the datadog helm chart requires you to embed the postgres secret into the configuration, so I would want some way to disable it for specific charts, or enable it for others. Two ways would be some parameter of chart names to not show diffs for, or maybe a k8s metadata I can attach to the chart somehow. Another option would be a no-op helm value of "kubewise-meta: supress-diff" that is seen by kubewise. I wonder if this works as a general way to configure kubewise inside helm charts. |
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: There is no ability to blacklist packages yet. That can be added in a separate PR on this same issue. Implementation wise, I'm leaning towards a blacklist rather than a whitelist. There are k8s users with hundreds of applications running in their clusters, a whitelist would be difficult to maintain and the feature would end up not being used. With a blacklist, the likelihood of accidentally leaking an API key or something goes up but
Open to further thoughts if you have any? |
blacklist sounds good, but realistically, secrets shouldn't be in values (not saying that we comply with that yet) ;) |
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. |
We often modify charts just by changing values. It would be useful to show a
diff
of the values from the current release to the previous one. For example, here it is for my kubewise helm chart from the CLI.The text was updated successfully, but these errors were encountered: