Past node name to labeller via env var

Author: y2kenny

cmd/k8s-node-labeller/main.go

@@ -293,11 +293,7 @@ func main() {
 	}
 
 	// laballer only respond to event about the node it is on by matching hostname
-	b, err := ioutil.ReadFile("/labeller/hostname")
-	if err != nil {
-		entryLog.Error(err, "Cannot read hostname")
-	}
-	hostname := strings.TrimSpace(string(b))
+	hostname := os.Getenv("DS_NODE_NAME")
 
 	pred := predicate.Funcs{
 		// Create returns true if the Create event should be processed

k8s-ds-amdgpu-labeller.yaml

@@ -49,9 +49,15 @@ spec:
       containers:
       - image: rocm/k8s-device-plugin:labeller-latest
         name: amdgpu-lr-cntr
+        imagePullPolicy: Always
         workingDir: /root
         command: ["./k8s-node-labeller"]
         args: ["-vram", "-cu-count", "-simd-count", "-device-id", "-family"]
+        env:
+          - name: DS_NODE_NAME
+            valueFrom:
+              fieldRef:
+                fieldPath: spec.nodeName
         securityContext:
           privileged: true #Needed for /dev
           capabilities:
@@ -61,15 +67,10 @@ spec:
             mountPath: /sys
           - name: dev
             mountPath: /dev
-          - name: etc
-            mountPath: /labeller
       volumes:
         - name: sys
           hostPath:
             path: /sys
         - name: dev
           hostPath:
             path: /dev
-        - name: etc
-          hostPath:
-            path: /etc