fix: max-cpu-loras should be initialized from max-loras (#77)

* Don't initialize ma-cpu-loras. Take max-loras as default

Signed-off-by: Shmuel Kallner <[email protected]>

* Update config_test.go - fix test

Signed-off-by: Maya Barnea <[email protected]>

---------

Signed-off-by: Shmuel Kallner <[email protected]>
Signed-off-by: Maya Barnea <[email protected]>
Co-authored-by: Maya Barnea <[email protected]>

Author: shmuelk

pkg/llm-d-inference-sim/config.go

@@ -92,11 +92,10 @@ func (l *loraModulesValue) UnmarshalYAML(unmarshal func(interface{}) error) erro
 
 func newConfig() *configuration {
 	return &configuration{
-		Port:        vLLMDefaultPort,
-		MaxLoras:    1,
-		MaxCPULoras: 1,
-		MaxNumSeqs:  5,
-		Mode:        modeRandom,
+		Port:       vLLMDefaultPort,
+		MaxLoras:   1,
+		MaxNumSeqs: 5,
+		Mode:       modeRandom,
 	}
 }
 

pkg/llm-d-inference-sim/config_test.go

@@ -54,6 +54,7 @@ var _ = Describe("Simulator configuration", func() {
 	c := newConfig()
 	c.Model = model
 	c.ServedModelNames = []string{c.Model}
+	c.MaxCPULoras = 1
 	test := testCase{
 		name:           "simple",
 		args:           []string{"cmd", "--model", model, "--mode", modeRandom},