Skip to content

Commit

Permalink
Merge pull request #703 from elezar/fix-no-persistenced-flag
Browse files Browse the repository at this point in the history
[no-relnote] Move --no-persistenced flag to after configure
  • Loading branch information
elezar authored Sep 18, 2024
2 parents b061446 + 16fdef5 commit c02b144
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions cmd/nvidia-container-runtime-hook/main.go
Original file line number Diff line number Diff line change
Expand Up @@ -90,11 +90,6 @@ func doPrestart() {

args := []string{getCLIPath(cli)}

// Only include the nvidia-persistenced socket if it is explicitly enabled.
if !hook.Features.IncludePersistencedSocket.IsEnabled() {
args = append(args, "--no-persistenced")
}

if cli.Root != "" {
args = append(args, fmt.Sprintf("--root=%s", cli.Root))
}
Expand All @@ -117,6 +112,11 @@ func doPrestart() {
}
args = append(args, "configure")

// Only include the nvidia-persistenced socket if it is explicitly enabled.
if !hook.Features.IncludePersistencedSocket.IsEnabled() {
args = append(args, "--no-persistenced")
}

if ldconfigPath := cli.NormalizeLDConfigPath(); ldconfigPath != "" {
args = append(args, fmt.Sprintf("--ldconfig=%s", ldconfigPath))
}
Expand Down

0 comments on commit c02b144

Please sign in to comment.