Skip to content

Commit e1d6928

Browse files
committed
When changing volume names, controller needs to be changed too
Change volume name to a more generic one before the NFD host mount directory itself needs to be changed: kubernetes-sigs/node-feature-discovery#856 Signed-off-by: Eero Tamminen <[email protected]>
1 parent efc4e48 commit e1d6928

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

pkg/controllers/gpu/controller.go

+4-4
Original file line numberDiff line numberDiff line change
@@ -142,7 +142,7 @@ func (c *controller) NewDaemonSet(rawObj client.Object) *apps.DaemonSet {
142142

143143
if devicePlugin.Spec.InitImage == "" {
144144
daemonSet.Spec.Template.Spec.InitContainers = nil
145-
daemonSet.Spec.Template.Spec.Volumes = removeVolume(daemonSet.Spec.Template.Spec.Volumes, "nfd-source-hooks")
145+
daemonSet.Spec.Template.Spec.Volumes = removeVolume(daemonSet.Spec.Template.Spec.Volumes, "nfd-features")
146146
} else {
147147
setInitContainer(&daemonSet.Spec.Template.Spec, devicePlugin.Spec.InitImage)
148148
}
@@ -201,11 +201,11 @@ func setInitContainer(spec *v1.PodSpec, imageName string) {
201201
VolumeMounts: []v1.VolumeMount{
202202
{
203203
MountPath: "/etc/kubernetes/node-feature-discovery/source.d/",
204-
Name: "nfd-source-hooks",
204+
Name: "nfd-features",
205205
},
206206
},
207207
}}
208-
addVolumeIfMissing(spec, "nfd-source-hooks", "/etc/kubernetes/node-feature-discovery/source.d/", v1.HostPathDirectoryOrCreate)
208+
addVolumeIfMissing(spec, "nfd-features", "/etc/kubernetes/node-feature-discovery/source.d/", v1.HostPathDirectoryOrCreate)
209209
}
210210

211211
func removeVolume(volumes []v1.Volume, name string) []v1.Volume {
@@ -243,7 +243,7 @@ func (c *controller) UpdateDaemonSet(rawObj client.Object, ds *apps.DaemonSet) (
243243
if dp.Spec.InitImage == "" {
244244
if ds.Spec.Template.Spec.InitContainers != nil {
245245
ds.Spec.Template.Spec.InitContainers = nil
246-
ds.Spec.Template.Spec.Volumes = removeVolume(ds.Spec.Template.Spec.Volumes, "nfd-source-hooks")
246+
ds.Spec.Template.Spec.Volumes = removeVolume(ds.Spec.Template.Spec.Volumes, "nfd-features")
247247
updated = true
248248
}
249249
} else {

0 commit comments

Comments
 (0)