update: code review and go fmt

Signed-off-by: Wen Zhou <wenzhou@redhat.com>

Author: zdtsw

internal/config/loader_test.go

@@ -7,8 +7,6 @@ import (
 	"time"
 
 	flag "github.com/spf13/pflag"
-
-
 )
 
 // writeTestConfigFile writes a YAML config file to a temp directory and returns its path.

internal/config/saturation_scaling.go

@@ -67,7 +67,7 @@ type AnalyzerScoreConfig struct {
 	Name              string   `yaml:"name"`
 	Enabled           *bool    `yaml:"enabled,omitempty"`           // default true
 	Score             float64  `yaml:"score,omitempty"`             // default 1.0
-	ScaleUpThreshold  *float64 `yaml:"scaleUpThreshold,omitempty"` // overrides global
+	ScaleUpThreshold  *float64 `yaml:"scaleUpThreshold,omitempty"`  // overrides global
 	ScaleDownBoundary *float64 `yaml:"scaleDownBoundary,omitempty"` // overrides global
 }
 

internal/controller/configmap_bootstrap.go

@@ -64,7 +64,6 @@ func (r *ConfigMapReconciler) BootstrapInitialConfigMaps(ctx context.Context) er
 							continue // Skip excluded namespaces. Only for all-namespaces mode.
 						}
 					}
-
 				}
 				if ns.Labels != nil {
 					if value, ok := ns.Labels[constants.NamespaceConfigEnabledLabelKey]; ok {

internal/engines/pipeline/enforcer.go

@@ -185,4 +185,3 @@ func updateDecisionAction(d *interfaces.VariantDecision, optimizerName string) {
 	}
 	d.Reason = fmt.Sprintf("V2 %s (optimizer: %s, enforced)", d.Action, optimizerName)
 }
-

internal/engines/pipeline/greedy_score_optimizer_test.go

@@ -967,10 +967,10 @@ var _ = Describe("GreedyByScoreOptimizer", func() {
 					ModelID:   "model-1",
 					Namespace: "default",
 					Result: &interfaces.AnalyzerResult{
-						ModelID:          "model-1",
-						Namespace:        "default",
-						AnalyzedAt:       time.Now(),
-						SpareCapacity:    50000,
+						ModelID:       "model-1",
+						Namespace:     "default",
+						AnalyzedAt:    time.Now(),
+						SpareCapacity: 50000,
 						VariantCapacities: []interfaces.VariantCapacity{
 							{VariantName: "expensive", AcceleratorName: "A100", Cost: 15.0, ReplicaCount: 3, PerReplicaCapacity: 20000},
 							{VariantName: "cheap", AcceleratorName: "A100", Cost: 5.0, ReplicaCount: 3, PerReplicaCapacity: 10000},
@@ -998,10 +998,10 @@ var _ = Describe("GreedyByScoreOptimizer", func() {
 					ModelID:   "model-1",
 					Namespace: "default",
 					Result: &interfaces.AnalyzerResult{
-						ModelID:          "model-1",
-						Namespace:        "default",
-						AnalyzedAt:       time.Now(),
-						SpareCapacity:    80000, // enough to remove all
+						ModelID:       "model-1",
+						Namespace:     "default",
+						AnalyzedAt:    time.Now(),
+						SpareCapacity: 80000, // enough to remove all
 						VariantCapacities: []interfaces.VariantCapacity{
 							{VariantName: "keep-alive", AcceleratorName: "A100", Cost: 15.0, ReplicaCount: 2, PerReplicaCapacity: 20000},
 							{VariantName: "expendable", AcceleratorName: "A100", Cost: 5.0, ReplicaCount: 3, PerReplicaCapacity: 10000},

internal/engines/saturation/engine_v2.go

@@ -114,7 +114,7 @@ func (e *Engine) runAnalyzersAndScore(
 		switch aw.Name {
 		case "saturation":
 			totalWeighted += baseResult.RequiredCapacity * aw.Score
-		// future: case "throughput", "slo"
+			// future: case "throughput", "slo"
 		}
 	}