Address reviewer requested changes

Signed-off-by: Carlos Eduardo Arango Gutierrez <eduardoa@nvidia.com>
Mellanox · Jun 26, 2023 · d559268 · d559268
1 parent d1c2b99
commit d559268
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 5 deletions.
diff --git a/deployment/network-operator/templates/upgrade-crd.yaml b/deployment/network-operator/templates/upgrade-crd.yaml
@@ -77,7 +77,7 @@ spec:
               kubectl apply -f /crds/sriov-network-operator;
         {{- end }}
         {{- if .Values.nfd.enabled }}
-              if ! kubectl get crd | grep -E "nodefeaturerules\.nfd\.k8s-sigs\.io|nodefeatures\.nfd\.k8s-sigs\.io" ; then kubectl apply -f /crds/node-feature-discovery; fi
+              kubectl apply -f /crds/node-feature-discovery;
         {{- end }}
       restartPolicy: OnFailure
 {{- end }}
diff --git a/deployment/network-operator/values.yaml b/deployment/network-operator/values.yaml
@@ -41,8 +41,10 @@ node-feature-discovery:
       create: false
     tolerations:
     - key: "node-role.kubernetes.io/master"
-      operator: "Equal"
-      value: ""
+      operator: "Exists"
+      effect: "NoSchedule"
+    - key: "node-role.kubernetes.io/control-plane"
+      operator: "Exists"
       effect: "NoSchedule"
     - key: nvidia.com/gpu
       operator: Exists
@@ -59,6 +61,7 @@ node-feature-discovery:
           deviceLabelFields:
           - vendor
   master:
+    instance: "nvidia.networking"
     serviceAccount:
       name: node-feature-discovery
       create: true

diff --git a/hack/templates/values/values.template b/hack/templates/values/values.template
@@ -41,8 +41,10 @@ node-feature-discovery:
       create: false
     tolerations:
     - key: "node-role.kubernetes.io/master"
-      operator: "Equal"
-      value: ""
+      operator: "Exists"
+      effect: "NoSchedule"
+    - key: "node-role.kubernetes.io/control-plane"
+      operator: "Exists"
       effect: "NoSchedule"
     - key: nvidia.com/gpu
       operator: Exists
@@ -59,6 +61,7 @@ node-feature-discovery:
           deviceLabelFields:
           - vendor
   master:
+    instance: "nvidia.networking"
     serviceAccount:
       name: node-feature-discovery
       create: true