Not able to change updateStrategy for masters deployment to recreate

[tarioch]

Describe the Bug

At the moment there is an updateStrategy config but I can't change it to recreate as there are additional rollingUpdate settings which are forbidden with recreate.

Expected Behavior

I would expect to be able to specify something like

puppetserver:
  masters:
    updateStrategy:
      type: Recreate

in my values file and it to work.

But because of

      rollingUpdate:
        maxSurge: 20%
        maxUnavailable: 0%

in the default values file, it will fail.

My suggestion would be to remove the maxSurge/maxUnavailable settings from the default values file. Or alternatively only add them if the updateStrategy is rollingUpdate.

[davidphay]

Hello,

you can use the follow syntax, it should works:

puppetserver:
  masters:
    updateStrategy:
      type: Recreate
      rollingUpdate:

[tarioch]

thank you very much, that worked, I had tried different other syntax but wasn't aware that this is the way to do it.