Skip to content

Commit 4a56d13

Browse files
ArangoGutierrezelezar
authored andcommitted
[no-relnote] Rename Config to TopLevelConfigPath
This change renames the Config struct member to TopLevelConfigPath. This aids readability and better distinguishes between the top-level config file and a drop-in config file. Signed-off-by: Carlos Eduardo Arango Gutierrez <eduardoa@nvidia.com>
1 parent 96050da commit 4a56d13

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)