Skip to content

Commit

Permalink
install/0000_00_cluster-version-operator_03_deployment: Fix Recreate …
Browse files Browse the repository at this point in the history
…strategy

I'd added this in 078686d
(install/0000_00_cluster-version-operator_03_deployment: Set
'strategy: Recreate', 2019-03-20, openshift#140), but apparently got the wrong
place [1]:

  $ curl -s https://storage.googleapis.com/origin-ci-test/logs/release-openshift-origin-installer-e2e-aws-4.0/6412/artifacts/e2e-aws/deployments.json.gz | gunzip | jq '.items[] | select(.metadata.name == "cluster-version-operator").spec.strategy'
  {
    "rollingUpdate": {
      "maxSurge": "25%",
      "maxUnavailable": "25%"
    },
    "type": "RollingUpdate"
  }

I'm not actually sure how I arrived at the previous position.  Anyway,
hopefully this commit puts the setting in the right place.

[1]: https://bugzilla.redhat.com/show_bug.cgi?id=1670727#c14
  • Loading branch information
wking committed Apr 3, 2019
1 parent b72cf63 commit 5d8a527
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion install/0000_00_cluster-version-operator_03_deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,8 @@ spec:
selector:
matchLabels:
k8s-app: cluster-version-operator
strategy:
type: Recreate
template:
metadata:
name: cluster-version-operator
Expand Down Expand Up @@ -46,7 +48,6 @@ spec:
nodeSelector:
node-role.kubernetes.io/master: ""
priorityClassName: "system-cluster-critical"
strategy: Recreate
tolerations:
- operator: Exists
volumes:
Expand Down

0 comments on commit 5d8a527

Please sign in to comment.