validate agent config (#567)

real-danm · web-flow · commit 78235decda49 · 2025-05-06T13:32:22.000-06:00
diff --git a/pkg/agentfs/config-file.go b/pkg/agentfs/config-file.go
@@ -21,6 +21,7 @@ import (
 	"path/filepath"
 
 	"github.com/BurntSushi/toml"
+
 	"github.com/livekit/protocol/logger"
 )
 
@@ -73,5 +74,12 @@ func LoadTomlFile(dir string, tomlFileName string) (*AgentTOML, bool, error) {
 		}
 	}
 
+	if configExists {
+		// validate config
+		if agentConfig.Replicas > agentConfig.MaxReplicas {
+			return nil, configExists, fmt.Errorf("replicas cannot be greater than max_replicas")
+		}
+	}
+
 	return &agentConfig, configExists, err
 }

Original file line number	Diff line number	Diff line change
`@@ -21,6 +21,7 @@ import (`
`21`	`21`	`"path/filepath"`
`22`	`22`
`23`	`23`	`"github.com/BurntSushi/toml"`
	`24`	`+`
`24`	`25`	`"github.com/livekit/protocol/logger"`
`25`	`26`	`)`
`26`	`27`
`@@ -73,5 +74,12 @@ func LoadTomlFile(dir string, tomlFileName string) (*AgentTOML, bool, error) {`
`73`	`74`	`}`
`74`	`75`	`}`
`75`	`76`
	`77`	`+ if configExists {`
	`78`	`+ // validate config`
	`79`	`+ if agentConfig.Replicas > agentConfig.MaxReplicas {`
	`80`	`+ return nil, configExists, fmt.Errorf("replicas cannot be greater than max_replicas")`
	`81`	`+ }`
	`82`	`+ }`
	`83`	`+`
`76`	`84`	`return &agentConfig, configExists, err`
`77`	`85`	`}`