From 64bc7cb1c9da6ae400cb6f362abdb8b47446de24 Mon Sep 17 00:00:00 2001 From: Pawan Date: Tue, 20 Oct 2020 13:55:34 +0530 Subject: [PATCH] feat(upgrade): support parallel/faster upgrades for node daemonset For ZFSPV, all the node daemonset pods can go into the terminating state at the same time since it does not need any minimum availability of those pods. Changing maxUnavailable to 100% so that K8s can upgrade all the daemonset pods parallelly. Signed-off-by: Pawan --- changelogs/unreleased/230-pawanpraka1 | 1 + deploy/yamls/zfs-driver.yaml | 16 ++++++++++++++-- deploy/zfs-operator.yaml | 16 ++++++++++++++-- 3 files changed, 29 insertions(+), 4 deletions(-) create mode 100644 changelogs/unreleased/230-pawanpraka1 diff --git a/changelogs/unreleased/230-pawanpraka1 b/changelogs/unreleased/230-pawanpraka1 new file mode 100644 index 0000000..0f25af8 --- /dev/null +++ b/changelogs/unreleased/230-pawanpraka1 @@ -0,0 +1 @@ +support parallel/faster upgrades for node daemonset diff --git a/deploy/yamls/zfs-driver.yaml b/deploy/yamls/zfs-driver.yaml index c31729c..2cad9ee 100644 --- a/deploy/yamls/zfs-driver.yaml +++ b/deploy/yamls/zfs-driver.yaml @@ -501,6 +501,9 @@ apiVersion: apps/v1 metadata: name: openebs-zfs-controller namespace: kube-system + labels: + openebs.io/component-name: openebs-zfs-controller + openebs.io/version: ci spec: selector: matchLabels: @@ -513,6 +516,8 @@ spec: labels: app: openebs-zfs-controller role: openebs-zfs + openebs.io/component-name: openebs-zfs-controller + openebs.io/version: ci spec: affinity: podAntiAffinity: @@ -724,15 +729,24 @@ apiVersion: apps/v1 metadata: name: openebs-zfs-node namespace: kube-system + labels: + openebs.io/component-name: openebs-zfs-node + openebs.io/version: ci spec: selector: matchLabels: app: openebs-zfs-node + updateStrategy: + rollingUpdate: + maxUnavailable: 100% + type: RollingUpdate template: metadata: labels: app: openebs-zfs-node role: openebs-zfs + openebs.io/component-name: openebs-zfs-node + openebs.io/version: ci spec: priorityClassName: system-node-critical serviceAccount: openebs-zfs-node-sa @@ -768,8 +782,6 @@ spec: - name: openebs-zfs-plugin securityContext: privileged: true - capabilities: - add: ["CAP_MKNOD", "CAP_SYS_ADMIN", "SYS_ADMIN"] allowPrivilegeEscalation: true image: quay.io/openebs/zfs-driver:ci imagePullPolicy: IfNotPresent diff --git a/deploy/zfs-operator.yaml b/deploy/zfs-operator.yaml index 4e49ea3..1aae473 100644 --- a/deploy/zfs-operator.yaml +++ b/deploy/zfs-operator.yaml @@ -1540,6 +1540,9 @@ apiVersion: apps/v1 metadata: name: openebs-zfs-controller namespace: kube-system + labels: + openebs.io/component-name: openebs-zfs-controller + openebs.io/version: ci spec: selector: matchLabels: @@ -1552,6 +1555,8 @@ spec: labels: app: openebs-zfs-controller role: openebs-zfs + openebs.io/component-name: openebs-zfs-controller + openebs.io/version: ci spec: affinity: podAntiAffinity: @@ -1763,15 +1768,24 @@ apiVersion: apps/v1 metadata: name: openebs-zfs-node namespace: kube-system + labels: + openebs.io/component-name: openebs-zfs-node + openebs.io/version: ci spec: selector: matchLabels: app: openebs-zfs-node + updateStrategy: + rollingUpdate: + maxUnavailable: 100% + type: RollingUpdate template: metadata: labels: app: openebs-zfs-node role: openebs-zfs + openebs.io/component-name: openebs-zfs-node + openebs.io/version: ci spec: priorityClassName: system-node-critical serviceAccount: openebs-zfs-node-sa @@ -1807,8 +1821,6 @@ spec: - name: openebs-zfs-plugin securityContext: privileged: true - capabilities: - add: ["CAP_MKNOD", "CAP_SYS_ADMIN", "SYS_ADMIN"] allowPrivilegeEscalation: true image: quay.io/openebs/zfs-driver:ci imagePullPolicy: IfNotPresent