-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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
[NET-6232] docs: Update consul-k8s Helm chart docs (1.2.x) #19698
Conversation
@zalimeni I didn't realize these merged right after approval so approved one of these PRs. These are docs for un-released functionality is that correct? If so, we should only merge upon release. |
Also typically it is a customary for the release engineer to run the make-docs make command to help create a PR against the consul core repo based on the Helm chart comments, not sure if that is a better way to handle things. |
@david-yu good point, that was my mistake 🤦🏻♂️ I hadn't considered that after #19577 was merged and misinterpreted your backport label addition there; I really should have only synced over the old content that was missing. I'll open a PR to revert the merged change, and update this one to just address the un-synced content, then allow the normal process to take care of the new docs. (TIL we had that in place already, though I should have guessed so)
Makes sense. I did use |
Thanks I appreciate the follow up here, sorry for leading you originally down the wrong path. |
@david-yu of course, thanks for catching and educating me! |
Sync changes not yet represented in docs and relevant before the next patch release.
e71e31c
to
25a58a1
Compare
Ok @david-yu , this one is ready for review. |
Sync docs for recent changes to the Helm chart from
consul-k8s
.Description
Updates docs for backport of hashicorp/consul-k8s#3180 following #19577.
Also includes another minor previous change that wasn't synced.
PR Checklist