-
Notifications
You must be signed in to change notification settings - Fork 202
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Add a new concept to containers.conf called "modules". A "module" is a containers.conf file located at a specific directory. More than one modules can be loaded in the specified order, following existing override semantics. There are three directories to load modules from: - $CONFIG_HOME/containers/containers.conf.modules - /etc/containers/containers.conf.modules - /usr/share/containers/containers.conf.modules With CONFIG_HOME pointing to $HOME/.config or, if set, $XDG_CONFIG_HOME. Absolute paths will be loaded as is, relative paths will be resolved relative to the three directories above allowing for admin configs (/etc/) to override system configs (/usr/share/) and user configs ($CONFIG_HOME) to override admin configs. Also move some functions from config.go for locality. Signed-off-by: Valentin Rothberg <[email protected]>
- Loading branch information
Showing
21 changed files
with
445 additions
and
125 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,88 @@ | ||
| package config | ||
|
|
||
| import ( | ||
| "fmt" | ||
| "os" | ||
| "path/filepath" | ||
|
|
||
| "github.com/containers/storage/pkg/homedir" | ||
| "github.com/containers/storage/pkg/unshare" | ||
| "github.com/hashicorp/go-multierror" | ||
| ) | ||
|
|
||
| // The subdirectory for looking up containers.conf modules. | ||
| const moduleSubdir = "containers/containers.conf.modules" | ||
|
|
||
| // Moving the base paths into variables allows for overriding them in units | ||
| // tests. | ||
| var ( | ||
| moduleBaseEtc = "/etc/" | ||
| moduleBaseUsr = "/usr/share" | ||
| ) | ||
|
|
||
| // Find the specified modules in the options. Return an error if a specific | ||
| // module cannot be located on the host. | ||
| func (o *Options) modules() ([]string, error) { | ||
| if len(o.Modules) == 0 { | ||
| return nil, nil | ||
| } | ||
|
|
||
| dirs, err := moduleDirectories() | ||
| if err != nil { | ||
| return nil, err | ||
| } | ||
|
|
||
| configs := make([]string, 0, len(o.Modules)) | ||
| for _, path := range o.Modules { | ||
| resolved, err := resolveModule(path, dirs) | ||
| if err != nil { | ||
| return nil, fmt.Errorf("could not resolve module %q: %w", path, err) | ||
| } | ||
| configs = append(configs, resolved) | ||
| } | ||
|
|
||
| return configs, nil | ||
| } | ||
|
|
||
| // Return the directories to load modules from: | ||
| // 1) XDG_CONFIG_HOME/HOME if rootless | ||
| // 2) /etc/ | ||
| // 3) /usr/share | ||
| func moduleDirectories() ([]string, error) { | ||
| modules := []string{ | ||
| filepath.Join(moduleBaseEtc, moduleSubdir), | ||
| filepath.Join(moduleBaseUsr, moduleSubdir), | ||
| } | ||
|
|
||
| if !unshare.IsRootless() { | ||
| return modules, nil | ||
| } | ||
|
|
||
| // Prepend the user modules dir. | ||
| configHome, err := homedir.GetConfigHome() | ||
| if err != nil { | ||
| return nil, err | ||
| } | ||
| return append([]string{filepath.Join(configHome, moduleSubdir)}, modules...), nil | ||
| } | ||
|
|
||
| // Resolve the specified path to a module. | ||
| func resolveModule(path string, dirs []string) (string, error) { | ||
| if filepath.IsAbs(path) { | ||
| _, err := os.Stat(path) | ||
| return path, err | ||
| } | ||
|
|
||
| // Collect all errors to avoid suppressing important errors (e.g., | ||
| // permission errors). | ||
| var multiErr error | ||
| for _, d := range dirs { | ||
| candidate := filepath.Join(d, path) | ||
| _, err := os.Stat(candidate) | ||
| if err == nil { | ||
| return candidate, nil | ||
| } | ||
| multiErr = multierror.Append(multiErr, err) | ||
| } | ||
| return "", multiErr | ||
| } |
Oops, something went wrong.