Skip to content

Commit c3f5ed2

Browse files
authored
Merge pull request #1301 from ArangoGutierrez/dropins_ctk_installer_defaults
[no-relnote] Rename Config to TopLevelConfigPath
2 parents 96050da + 4a56d13 commit c3f5ed2

File tree

7 files changed

+215
-215
lines changed

7 files changed

+215
-215
lines changed

cmd/nvidia-ctk-installer/container/container.go

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -36,9 +36,9 @@ const (
3636
// Options defines the shared options for the CLIs to configure containers runtimes.
3737
type Options struct {
3838
DropInConfig string
39-
// TODO: Rename to TopLevelConfig
40-
Config string
41-
Socket string
39+
// TopLevelConfigPath stores the path to the top-level config for the runtime.
40+
TopLevelConfigPath string
41+
Socket string
4242
// ExecutablePath specifies the path to the container runtime executable.
4343
// This is used to extract the current config, for example.
4444
// If a HostRootMount is specified, this path is relative to the host root
@@ -75,7 +75,7 @@ func (o Options) Unconfigure(cfg engine.Interface) error {
7575
func (o Options) flush(cfg engine.Interface) error {
7676
filepath := o.DropInConfig
7777
if filepath == "" {
78-
filepath = o.Config
78+
filepath = o.TopLevelConfigPath
7979
}
8080
logrus.Infof("Flushing config to %v", filepath)
8181
n, err := cfg.Save(filepath)

0 commit comments

Comments
 (0)