nats-io / nats-streaming-operator

NATS Streaming Operator
Apache License 2.0
174 stars 44 forks source link

Helm chart deployment broken apps.DeploymentStrategy #64

Open rchenzheng opened 4 years ago

rchenzheng commented 4 years ago
helm install --name nats-steaming --namespace nats-io -f values.yaml .

Error:

Error: release nats-steaming-operator failed: Deployment.apps "nats-steaming-operator-nats-streaming-operator" is invalid: spec.strategy: Unsupported value: apps.DeploymentStrategy{Type:"OnDelete", RollingUpdate:(*apps.RollingUpdateDeployment)(nil)}: supported values: "Recreate", "RollingUpdate"
nick-phillips-dev commented 4 years ago

The values.yaml references the StatefulSet docs incorrectly. The operator is deployed as a Deployment which only allows "Recreate" or "RollingUpdate", not "OnDelete"

https://kubernetes.io/docs/concepts/workloads/controllers/deployment/#strategy

gnarea commented 4 years ago

For anyone coming across this, one quick workaround could be:

helm install --set updateStrategy=RollingUpdate nats-streaming .