[release-0.2 ] drop DaemonSet unsupported OnFailure restartPolicy

The DaemonSet controller currently only supports the 'Always' restart policy, which is set by default. This commit addresses the issue of incorrectly set 'OnFailure' restart policy in DaemonSet objects of plugins, ensuring that only the 'Always' restart policy is used as intended. Signed-off-by: Feruzjon Muyassarov <[email protected]>
containers · Oct 17, 2023 · ff43d1f · ff43d1f
1 parent 0288494
commit ff43d1f
Show file tree

Hide file tree

Showing 4 changed files with 0 additions and 4 deletions.
diff --git a/deployment/helm/balloons/templates/daemonset.yaml b/deployment/helm/balloons/templates/daemonset.yaml
@@ -14,7 +14,6 @@ spec:
       labels:
       {{- include "balloons-plugin.labels" . | nindent 8 }}
     spec:
-      restartPolicy: OnFailure
       serviceAccount: nri-resource-policy-balloons
       nodeSelector:
         kubernetes.io/os: "linux"

diff --git a/deployment/helm/memory-qos/templates/daemonset.yaml b/deployment/helm/memory-qos/templates/daemonset.yaml
@@ -14,7 +14,6 @@ spec:
       labels:
       {{- include "memory-qos.labels" . | nindent 8 }}
     spec:
-      restartPolicy: OnFailure
       nodeSelector:
         kubernetes.io/os: "linux"
       {{- if .Values.nri.patchRuntimeConfig }}

diff --git a/deployment/helm/memtierd/templates/daemonset.yaml b/deployment/helm/memtierd/templates/daemonset.yaml
@@ -14,7 +14,6 @@ spec:
       labels:
       {{- include "memtierd.labels" . | nindent 8 }}
     spec:
-      restartPolicy: OnFailure
       nodeSelector:
         kubernetes.io/os: "linux"
       hostPID: true

diff --git a/deployment/helm/topology-aware/templates/daemonset.yaml b/deployment/helm/topology-aware/templates/daemonset.yaml
@@ -14,7 +14,6 @@ spec:
       labels:
       {{- include "topology-aware-plugin.labels" . | nindent 8 }}
     spec:
-      restartPolicy: OnFailure
       serviceAccount: nri-resource-policy-topology-aware
       nodeSelector:
         kubernetes.io/os: "linux"