consul-k8s icon indicating copy to clipboard operation
consul-k8s copied to clipboard

Expose Server Statefulset updateStrategy configuration

Open MeijerM1 opened this issue 3 years ago • 2 comments

Community Note

  • Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request. Searching for pre-existing feature requests helps us consolidate datapoints for identical requirements into a single place, thank you!
  • Please do not leave "+1" or other comments that do not add relevant new information or questions, they generate extra noise for issue followers and do not help prioritize the request.
  • If you are interested in working on this issue or have submitted a pull request, please leave a comment.

Is your feature request related to a problem? Please describe.

The Helm chart allows the configuration of the Consul Client updateStrategy. However the Server does not expose this configuration. The official Consul K8s upgrade guide recommends updating the server pods one by one by decreasing the updatePartition amount. This process van be simplified by using the OnDelete strategy which reduces the need for multiple updates to the Helm chart.

Feature Description

Allow the server pod updateStrategy to be configured via the Helm chart.

Use Case(s)

Reduce the required effort when upgrading Consul.

Contributions

MeijerM1 avatar May 23 '22 12:05 MeijerM1

Hi @MeijerM1,

Thank you for opening this issue. I think this makes a lot of sense and would be a great enhancement. I will share this with our PMs, @nrichu-hcp and @david-yu.

t-eckert avatar May 31 '22 16:05 t-eckert

Hi @MeijerM1 This does seem like a valid improvement and we are open to PRs to improve this workflow. At a high level we would like to improve our overall upgrade workflow and provide some sort of automation to improve there overall UX for upgrades on Consul K8s sometime later this year.

david-yu avatar Jun 02 '22 16:06 david-yu

No longer needed as we no longer need to utilize updatePartition due to this change: https://github.com/hashicorp/consul-k8s/pull/3000

david-yu avatar Feb 13 '24 08:02 david-yu