ClusterCockpit/cc-backend
1
0
Fork 0
mirror of https://github.com/ClusterCockpit/cc-backend synced 2026-06-19 09:47:29 +02:00
Code Issues Packages Releases Activity

feat(config): warn on unrecognized top-level config sections

Browse Source 
A setting nested at the wrong level (e.g. "resampling" placed next to
"main" instead of inside it) was silently ignored. Emit a startup warning
for any top-level config section not consumed by the backend so such
misconfigurations surface instead of failing silently.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Entire-Checkpoint: 69d1b85c8db3
This commit is contained in:
Jan Eitzinger
2026-06-18 10:28:59 +02:00
parent ffbe171327
commit d89f526eb2
1 changed files with 13 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
cmd/cc-backend/main.go
View File

@@ -91,6 +91,19 @@ func initGops() error {
func initConfiguration() error { func initConfiguration() error {
ccconf.Init(flagConfigFile) ccconf.Init(flagConfigFile)
// Warn about unrecognized top-level config sections. A common mistake is
// nesting a setting at the wrong level (e.g. placing "resampling" next to
// "main" instead of inside it), which is otherwise silently ignored.
knownSections := map[string]bool{
"main": true, "auth": true, "nats": true, "archive": true,
"metric-store": true, "metric-store-external": true, "cron": true, "ui": true,
}
for _, k := range ccconf.GetKeys() {
if !knownSections[k] {
cclog.Warnf("ignoring unrecognized top-level config section %q (check nesting in config file)", k)
}
}
cfg := ccconf.GetPackageConfig("main") cfg := ccconf.GetPackageConfig("main")
if cfg == nil { if cfg == nil {
return fmt.Errorf("main configuration must be present") return fmt.Errorf("main configuration must be present")