Merge pull request #1091 from AhmedGrati/feat-helm-enable-taints

feat: add enableTaints to helm chart
kubernetes-sigs · Mar 21, 2023 · 821e042 · 821e042
2 parents b0a45cd + 02b3b7c
commit 821e042
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 0 deletions.
diff --git a/deployment/helm/node-feature-discovery/templates/master.yaml b/deployment/helm/node-feature-discovery/templates/master.yaml
@@ -92,6 +92,9 @@ spec:
             {{- if .Values.master.resourceLabels | empty | not }}
             - "-resource-labels={{- join "," .Values.master.resourceLabels }}"
             {{- end }}
+            {{- if .Values.master.enableTaints }}
+            - "-enable-taints"
+            {{- end }}
             {{- if .Values.master.crdController | kindIs "invalid" | not }}
             - "-crd-controller={{ .Values.master.crdController }}"
             {{- else }}

diff --git a/deployment/helm/node-feature-discovery/values.yaml b/deployment/helm/node-feature-discovery/values.yaml
@@ -20,6 +20,7 @@ master:
   denyLabelNs: []
   extraLabelNs: []
   resourceLabels: []
+  enableTaints: false
   crdController: null
   featureRulesController: null
   deploymentAnnotations: {}

diff --git a/docs/deployment/helm.md b/docs/deployment/helm.md
@@ -112,6 +112,7 @@ We have introduced the following Chart parameters.
 | `master.instance`           | string  |                                         | Instance name. Used to separate annotation namespaces for multiple parallel deployments                                                  |
 | `master.extraLabelNs`       | array   | []                                      | List of allowed extra label namespaces                                                                                                   |
 | `master.resourceLabels`     | array   | []                                      | List of labels to be registered as extended resources                                                                                          |
+| `master.enableTaints`       | bool    | false                                   | Specifies whether to enable or disable node tainting                                                                                           |
 | `master.crdController`      | bool    | null                                    | Specifies whether the NFD CRD API controller is enabled. If not set, controller will be enabled if `master.instance` is empty. |
 | `master.featureRulesController` | bool | null                                   | DEPRECATED: use `master.crdController` instead |
 | `master.replicaCount`       | integer | 1                                       | Number of desired pods. This is a pointer to distinguish between explicit zero and not specified                                         |