Skip to content

Commit

Permalink
we didn't reset the main configuration, causing some config vars stil…
Browse files Browse the repository at this point in the history
…l to be set
  • Loading branch information
cedricve committed Sep 14, 2023
1 parent 910bb3c commit 24136f8
Showing 1 changed file with 6 additions and 1 deletion.
7 changes: 6 additions & 1 deletion machinery/src/config/main.go
Original file line number Diff line number Diff line change
Expand Up @@ -141,8 +141,13 @@ func OpenConfig(configDirectory string, configuration *models.Configuration) {
},
)

// Merge Config toplevel
// Reset main configuration Config.
configuration.Config = models.Config{}

// Merge the global settings in the main config
conjungo.Merge(&configuration.Config, configuration.GlobalConfig, opts)

// Now we might override some settings with the custom config
conjungo.Merge(&configuration.Config, configuration.CustomConfig, opts)

// Merge Kerberos Vault settings
Expand Down

0 comments on commit 24136f8

Please sign in to comment.