Skip to content

Commit

Permalink
chore: remove driver-pod label (Mellanox#729)
Browse files Browse the repository at this point in the history
We already have a label which marks a pod as a driver pod which does not
rely on the version of the driver. use it instead
  • Loading branch information
e0ne authored Jan 3, 2024
2 parents 049c08b + 1d6fe33 commit 1244098
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 3 deletions.
3 changes: 1 addition & 2 deletions controllers/nicclusterpolicy_controller.go
Original file line number Diff line number Diff line change
Expand Up @@ -175,8 +175,7 @@ func (r *NicClusterPolicyReconciler) handleMOFEDWaitLabels(
return r.handleMOFEDWaitLabelsNoConfig(ctx)
}
pods := &corev1.PodList{}
podLabel := "mofed-" + cr.Spec.OFEDDriver.Version
_ = r.Client.List(ctx, pods, client.MatchingLabels{"driver-pod": podLabel})
_ = r.Client.List(ctx, pods, client.MatchingLabels{"nvidia.com/ofed-driver": ""})
for i := range pods.Items {
pod := pods.Items[i]
labelValue := "true"
Expand Down
1 change: 0 additions & 1 deletion manifests/state-ofed-driver/0050_ofed-driver-ds.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,6 @@ spec:
metadata:
labels:
app: mofed-{{ .RuntimeSpec.OSName }}{{ .RuntimeSpec.OSVer }}
driver-pod: mofed-{{ .CrSpec.Version }}
nvidia.com/ofed-driver: ""
spec:
priorityClassName: system-node-critical
Expand Down

0 comments on commit 1244098

Please sign in to comment.