Fix

Signed-off-by: Pavol Loffay <[email protected]>

Author: pavolloffay

internal/fips/fipscheck.go

@@ -32,11 +32,7 @@ type fipsCheck struct {
 }
 
 // NewFipsCheck creates new FipsCheck.
-func NewFipsCheck(FIPSEnabled bool, receivers, exporters, processors, extensions []string) FIPSCheck {
-	if !FIPSEnabled {
-		return nil
-	}
-
+func NewFipsCheck(receivers, exporters, processors, extensions []string) FIPSCheck {
 	return &fipsCheck{
 		receivers:  listToMap(receivers),
 		exporters:  listToMap(exporters),

internal/fips/fipscheck_test.go

@@ -21,7 +21,7 @@ import (
 )
 
 func TestFipsCheck(t *testing.T) {
-	fipsCheck := NewFipsCheck(true, []string{"rec1", "rec2"}, []string{"exp1"}, []string{"processor"}, []string{"ext1"})
+	fipsCheck := NewFipsCheck([]string{"rec1", "rec2"}, []string{"exp1"}, []string{"processor"}, []string{"ext1"})
 	blocked := fipsCheck.DisabledComponents(
 		map[string]interface{}{"otlp": true, "rec1/my": true},
 		map[string]interface{}{"exp1": true},

main.go

@@ -441,9 +441,12 @@ func main() {
 			return warnings
 		}
 
-		receivers, exporters, processors, extensions := parseFipsFlag(fipsDisabledComponents)
-		logger.Info("Fips disabled components", "receivers", receivers, "exporters", exporters, "processors", processors, "extensions", extensions)
-		fipsCheck := fips.NewFipsCheck(ad.FIPSEnabled(ctx), receivers, exporters, processors, extensions)
+		var fipsCheck fips.FIPSCheck
+		if ad.FIPSEnabled(ctx) {
+			receivers, exporters, processors, extensions := parseFipsFlag(fipsDisabledComponents)
+			logger.Info("Fips disabled components", "receivers", receivers, "exporters", exporters, "processors", processors, "extensions", extensions)
+			fipsCheck = fips.NewFipsCheck(receivers, exporters, processors, extensions)
+		}
 		if err = otelv1beta1.SetupCollectorWebhook(mgr, cfg, reviewer, crdMetrics, bv, fipsCheck); err != nil {
 			setupLog.Error(err, "unable to create webhook", "webhook", "OpenTelemetryCollector")
 			os.Exit(1)