Skip to content

Commit

Permalink
Merge branch 'fix-gds-transforms' into 'master'
Browse files Browse the repository at this point in the history
Fix bug in transformGDSContainer()

See merge request nvidia/kubernetes/gpu-operator!996
  • Loading branch information
cdesiniotis committed Dec 28, 2023
2 parents 76a454b + 18083ce commit ed89f38
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions controllers/object_controls.go
Original file line number Diff line number Diff line change
Expand Up @@ -2459,7 +2459,7 @@ func transformGDSContainer(obj *appsv1.DaemonSet, config *gpuv1.ClusterPolicySpe
return fmt.Errorf("GPUDirect Storage driver '%s' is only supported with NVIDIA OpenRM drivers. Please set 'driver.useOpenKernelModules=true' in ClusterPolicy to enable OpenRM mode", config.GPUDirectStorage.Version)
}

gdsContainer := obj.Spec.Template.Spec.Containers[i]
gdsContainer := &obj.Spec.Template.Spec.Containers[i]

// update nvidia-fs(sidecar) image and pull policy
gdsImage, err := resolveDriverTag(n, config.GPUDirectStorage)
Expand All @@ -2481,7 +2481,7 @@ func transformGDSContainer(obj *appsv1.DaemonSet, config *gpuv1.ClusterPolicySpe
// set/append environment variables for GDS container
if len(config.GPUDirectStorage.Env) > 0 {
for _, env := range config.GPUDirectStorage.Env {
setContainerEnv(&(gdsContainer), env.Name, env.Value)
setContainerEnv(gdsContainer, env.Name, env.Value)
}
}

Expand Down

0 comments on commit ed89f38

Please sign in to comment.