From c22ac01a361ab3f1c48f86ad6eeed4d491995bd5 Mon Sep 17 00:00:00 2001 From: Steve Kuznetsov Date: Mon, 18 Dec 2023 08:15:18 -0700 Subject: [PATCH] deploy: use Recreate strategy for Deployments Our controllers must not share runtime with controllers of previous versions, as we expect that the set of actors on the cluster to be coherent with respect to what they're trying to do. RollingUpdate strategies do not guarantee this and lead to cases where previous versions of the operators issue spurious mutating calls that the current operator needs to un-do. Signed-off-by: Steve Kuznetsov --- .../chart/templates/0000_50_olm_07-olm-operator.deployment.yaml | 2 +- .../templates/0000_50_olm_08-catalog-operator.deployment.yaml | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/deploy/chart/templates/0000_50_olm_07-olm-operator.deployment.yaml b/deploy/chart/templates/0000_50_olm_07-olm-operator.deployment.yaml index 76f7ca1ee0..f2c7bd8ab2 100644 --- a/deploy/chart/templates/0000_50_olm_07-olm-operator.deployment.yaml +++ b/deploy/chart/templates/0000_50_olm_07-olm-operator.deployment.yaml @@ -7,7 +7,7 @@ metadata: app: olm-operator spec: strategy: - type: RollingUpdate + type: Recreate replicas: {{ .Values.olm.replicaCount }} selector: matchLabels: diff --git a/deploy/chart/templates/0000_50_olm_08-catalog-operator.deployment.yaml b/deploy/chart/templates/0000_50_olm_08-catalog-operator.deployment.yaml index 2b2d4534d3..eea8046cea 100644 --- a/deploy/chart/templates/0000_50_olm_08-catalog-operator.deployment.yaml +++ b/deploy/chart/templates/0000_50_olm_08-catalog-operator.deployment.yaml @@ -7,7 +7,7 @@ metadata: app: catalog-operator spec: strategy: - type: RollingUpdate + type: Recreate replicas: {{ .Values.catalog.replicaCount }} selector: matchLabels: