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
Docs: Add part on updating the Helm chart #23739
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! 👍 Please add the missing release note label.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi, and thanks!
It's nice to have the different steps covered, but to be honest I find that it's a lot of commands to run, when make -C install/kubernetes
runs them all at once (all: update-versions check-values-yaml docs lint
, update-versions: update-chart cilium/values.yaml
).
I don't mind keeping the details to explain what happens, but for users who aren't interested in those, can we at least mention somewhere at the top that these are all executed as part of the default Makefile target?
Please also find some minor comments below.
One last thing, can you please squash your two commits and force-push to your branch? |
This adds a part to the documentation on how to run the linting an document update for Helm chart. Signed-off-by: Maartje Eyskens <maartje.eyskens@isovalent.com>
4d87da2
to
7039635
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me, thanks for addressing all the feedback!
The Netlify preview build failed for some (apparently unrelated) reason, I'm trying to find someone who can re-trigger it.
Bill re-triggered the build (thanks!), and the preview looks all good now, even though Netlify didn't update the job status on this page. So we're ready to go! |
This adds a part to the documentation on how to run the linting an document update for Helm chart.
Signed-off-by: Maartje Eyskens maartje.eyskens@isovalent.com
I did this on the experience I had in #23719, am i missing something that should be added?