update owner ref

shivakunv · shivakunv · commit 946c3d279f7f · 2025-10-20T14:29:06.000+05:30
diff --git a/assets/gpu-feature-discovery/0200_role.yaml b/assets/gpu-feature-discovery/0200_role.yaml
@@ -12,3 +12,9 @@ rules:
   - use
   resourceNames:
   - privileged
+- apiGroups:
+  - ""
+  resources:
+  - pods
+  verbs:
+  - get
diff --git a/assets/gpu-feature-discovery/0210_clusterrole.yaml b/assets/gpu-feature-discovery/0210_clusterrole.yaml
@@ -11,3 +11,9 @@ rules:
   - get
   - list
   - watch
+- apiGroups:
+  - ""
+  resources:
+  - pods
+  verbs:
+  - get
diff --git a/config/rbac/auth_proxy_client_clusterrole.yaml b/config/rbac/auth_proxy_client_clusterrole.yaml
@@ -5,3 +5,9 @@ metadata:
 rules:
 - nonResourceURLs: ["/metrics"]
   verbs: ["get"]
+- apiGroups:
+  - ""
+  resources:
+  - pods
+  verbs:
+  - get
diff --git a/config/rbac/auth_proxy_role.yaml b/config/rbac/auth_proxy_role.yaml
@@ -11,3 +11,9 @@ rules:
   resources:
   - subjectaccessreviews
   verbs: ["create"]
+- apiGroups:
+  - ""
+  resources:
+  - pods
+  verbs:
+  - get
diff --git a/config/rbac/clusterpolicy_editor_role.yaml b/config/rbac/clusterpolicy_editor_role.yaml
@@ -22,3 +22,9 @@ rules:
   - clusterpolicies/status
   verbs:
   - get
+- apiGroups:
+  - ""
+  resources:
+  - pods
+  verbs:
+  - get
diff --git a/config/rbac/clusterpolicy_viewer_role.yaml b/config/rbac/clusterpolicy_viewer_role.yaml
@@ -18,3 +18,9 @@ rules:
   - clusterpolicies/status
   verbs:
   - get
+- apiGroups:
+  - ""
+  resources:
+  - pods
+  verbs:
+  - get
diff --git a/config/rbac/leader_election_role.yaml b/config/rbac/leader_election_role.yaml
@@ -25,3 +25,9 @@ rules:
   verbs:
   - create
   - patch
+- apiGroups:
+  - ""
+  resources:
+  - pods
+  verbs:
+  - get
diff --git a/config/rbac/nvidiadriver_editor_role.yaml b/config/rbac/nvidiadriver_editor_role.yaml
@@ -22,3 +22,9 @@ rules:
   - nvidiadrivers/status
   verbs:
   - get
+- apiGroups:
+  - ""
+  resources:
+  - pods
+  verbs:
+  - get
diff --git a/config/rbac/nvidiadriver_viewer_role.yaml b/config/rbac/nvidiadriver_viewer_role.yaml
@@ -18,3 +18,9 @@ rules:
   - nvidiadrivers/status
   verbs:
   - get
+- apiGroups:
+  - ""
+  resources:
+  - pods
+  verbs:
+  - get
diff --git a/config/rbac/role.yaml b/config/rbac/role.yaml
@@ -204,3 +204,9 @@ rules:
   - securitycontextconstraints
   verbs:
   - use
+- apiGroups:
+  - ""
+  resources:
+  - pods
+  verbs:
+  - get
diff --git a/controllers/clusterpolicy_controller.go b/controllers/clusterpolicy_controller.go
@@ -302,7 +302,8 @@ func addWatchNewGPUNode(r *ClusterPolicyReconciler, c controller.Controller, mgr
 
 			ownerRefs := e.ObjectNew.GetOwnerReferences()
 			if isOwnedByDaemonSet(ownerRefs) {
-				return false
+				r.Log.Info("SHIVAAAAAAAA DaemonSet owner found", "ownerRefs", ownerRefs)
+				// return false
 			}
 			r.Log.Info("SHIVAAAAAAAA Calling labels: UpdateFunc", "ownerRefs", ownerRefs)
 
diff --git a/controllers/object_controls.go b/controllers/object_controls.go
@@ -3783,6 +3783,16 @@ func isDaemonSetReady(name string, n ClusterPolicyController) gpuv1.State {
 				}
 			}
 		}
+		// Create owner ref
+		ownerReference := []metav1.OwnerReference{}
+		// Get pod owner reference
+		ownerReference = append(ownerReference, metav1.OwnerReference{
+			APIVersion: corev1.SchemeGroupVersion.String(),
+			Kind:       "Pod",
+			Name:       pod.Name,
+			UID:        pod.UID,
+		})
+		pod.OwnerReferences = ownerReference
 	}
 
 	// All containers are ready
diff --git a/deployments/gpu-operator/charts/node-feature-discovery/templates/clusterrole.yaml b/deployments/gpu-operator/charts/node-feature-discovery/templates/clusterrole.yaml
@@ -137,4 +137,10 @@ rules:
   verbs:
   - delete
   - list
+- apiGroups:
+  - ""
+  resources:
+  - pods
+  verbs:
+  - get
 {{- end }}
