Loader changes due to FeatureGates no longer being a pointer

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

Author: shmuelk

pkg/epp/config/loader/configloader.go

@@ -125,15 +125,15 @@ func loadSchedulerConfig(configProfiles []configapi.SchedulingProfile, handle pl
 	return scheduling.NewSchedulerConfig(profileHandler, profiles), nil
 }
 
-func loadFeatureConfig(featureGates *configapi.FeatureGates) map[string]bool {
+func loadFeatureConfig(featureGates configapi.FeatureGates) map[string]bool {
 	featureConfig := map[string]bool{}
 
 	for gate := range registeredFeatureGates {
 		featureConfig[gate] = false
 	}
 
 	if featureGates != nil {
-		for gate, enabled := range *featureGates {
+		for gate, enabled := range featureGates {
 			featureConfig[gate] = enabled
 		}
 	}
@@ -226,12 +226,12 @@ func RegisterFeatureGate(gate string) {
 	registeredFeatureGates[gate] = struct{}{}
 }
 
-func validateFeatureGates(fg *configapi.FeatureGates) error {
+func validateFeatureGates(fg configapi.FeatureGates) error {
 	if fg == nil {
 		return nil
 	}
 
-	for gate := range *fg {
+	for gate := range fg {
 		if _, ok := registeredFeatureGates[gate]; !ok {
 			return errors.New(gate + " is an unregistered Feature Gate")
 		}

pkg/epp/config/loader/defaults.go

@@ -55,7 +55,7 @@ func setDefaultsPhaseOne(cfg *configapi.EndpointPickerConfig) {
 
 	// If no feature gates were specified, provide a default FeatureGates struct
 	if cfg.FeatureGates == nil {
-		cfg.FeatureGates = &configapi.FeatureGates{}
+		cfg.FeatureGates = configapi.FeatureGates{}
 	}
 
 	// If the SaturationDetector configuration wasn't specified setup a default one