Skip to content

Commit

Permalink
config explicitly merge defaults once when using a config directory
Browse files Browse the repository at this point in the history
  • Loading branch information
langmartin committed Jun 11, 2019
1 parent 5a3a47c commit 006a9a1
Showing 1 changed file with 8 additions and 2 deletions.
10 changes: 8 additions & 2 deletions command/agent/config.go
Original file line number Diff line number Diff line change
Expand Up @@ -1461,16 +1461,22 @@ func LoadConfig(path string) (*Config, error) {
return nil, err
}

defaults := ParseConfigDefault()

if fi.IsDir() {
return LoadConfigDir(path)
config, err := LoadConfigDir(path)
if err != nil {
return nil, err
}
return defaults.Merge(config), nil
}

cleaned := filepath.Clean(path)
config, err := ParseConfigFile(cleaned)
if err != nil {
return nil, fmt.Errorf("Error loading %s: %s", cleaned, err)
}

config = defaults.Merge(config)
config.Files = append(config.Files, cleaned)
return config, nil
}
Expand Down

0 comments on commit 006a9a1

Please sign in to comment.