Getting rid of pkg/utils

pkg utils is used for solely for getting the WATCH_NAMESPACE env
variable . We don't need a separate public package just for
that functionality. Moving the function into the main. In addition,
since WATCH_NAMESPACEmust be set in the deployment, the operator will
fail in case the variable is not defined, instead of just logging a
warrning

Author: yevgeny-shnaidman

main.go

@@ -40,7 +40,6 @@ import (
 	"sigs.k8s.io/node-feature-discovery-operator/internal/deployment"
 	"sigs.k8s.io/node-feature-discovery-operator/internal/job"
 	"sigs.k8s.io/node-feature-discovery-operator/internal/status"
-	"sigs.k8s.io/node-feature-discovery-operator/pkg/utils"
 	"sigs.k8s.io/node-feature-discovery-operator/pkg/version"
 	// +kubebuilder:scaffold:imports
 )
@@ -52,7 +51,8 @@ var (
 
 const (
 	// ProgramName is the canonical name of this program
-	ProgramName = "nfd-operator"
+	ProgramName          = "nfd-operator"
+	watchNamespaceEnvVar = "WATCH_NAMESPACE"
 )
 
 // operatorArgs holds command line arguments
@@ -96,10 +96,10 @@ func main() {
 		os.Exit(0)
 	}
 
-	watchNamespace, envSet := utils.GetWatchNamespace()
-	if !envSet {
-		setupLogger.Info("unable to get WatchNamespace, " +
-			"the manager will watch and manage resources in all namespaces")
+	watchNamespace, err := getWatchNamespace()
+	if err != nil {
+		setupLogger.Error(err, "WatchNamespaceEnvVar is not set")
+		os.Exit(1)
 	}
 
 	// Create a new manager to manage the operator
@@ -184,3 +184,12 @@ func initFlags(flagset *flag.FlagSet) *operatorArgs {
 
 	return &args
 }
+
+// getWatchNamespace returns the Namespace the operator should be watching for changes
+func getWatchNamespace() (string, error) {
+	value, present := os.LookupEnv(watchNamespaceEnvVar)
+	if !present {
+		return "", fmt.Errorf("environment variable %s is not defined", watchNamespaceEnvVar)
+	}
+	return value, nil
+}