darinspivey opened a new pull request, #668:
URL: https://github.com/apache/pulsar-helm-chart/pull/668

   Fixes: #667
   
   ### Motivation
   
   A user can control the `updateStrategy` for the pods of bookies and 
zookeeper. However, the values for brokers and proxies is hardcoded. Being able 
to control this value via the Helm chart is crucial to being able to do smooth 
chart upgrades to a fully-running cluster. For example, setting the strategy to 
`OnDelete` would allow a user to control which order the pods are restarted 
after an upgrade.
   
   ### Modifications
   
   For Broker and Proxy templates, set a default, but allow overrides from 
`.Values.xxx.updateStrategy`
   
   ### Verifying this change
   
   - [ ] Make sure that the change passes the CI checks.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to