Env var renaming

Author: MaciejKaras

main.go

@@ -270,11 +270,11 @@ func main() {
 		log.Info("Not running telemetry component!")
 	}
 
-	pprofEnabledString := env.ReadOrDefault(util.PprofEnabledEnv, "")
+	pprofEnabledString := env.ReadOrDefault(util.OperatorPprofEnabledEnv, "")
 	if pprofEnabled, err := pprof.IsPprofEnabled(pprofEnabledString, getOperatorEnv()); err != nil {
 		log.Errorf("Unable to check if pprof is enabled: %s", err)
 	} else if pprofEnabled {
-		port := env.ReadIntOrDefault(util.PprofPortEnv, util.PprofDefaultPort)
+		port := env.ReadIntOrDefault(util.OperatorPprofPortEnv, util.OperatorPprofDefaultPort)
 		if err := mgr.Add(pprof.NewRunnable(port, log)); err != nil {
 			log.Errorf("Unable to start pprof server: %s", err)
 		}

pkg/pprof/pprof.go

@@ -63,16 +63,16 @@ func (p *Runnable) Start(ctx context.Context) error {
 	return nil
 }
 
-// IsPprofEnabled checks if pprof is enabled based on the PPROF_ENABLED
+// IsPprofEnabled checks if pprof is enabled based on the MDB_OPERATOR_PPROF_ENABLED
 // and OPERATOR_ENV environment variables. It returns true if:
-// - PPROF_ENABLED is set to true
-// - OPERATOR_ENV is set to dev or local and PPROF_ENABLED is not set
+// - MDB_OPERATOR_PPROF_ENABLED is set to true
+// - OPERATOR_ENV is set to dev or local and MDB_OPERATOR_PPROF_ENABLED is not set
 // Otherwise, it returns false.
 func IsPprofEnabled(pprofEnabledString string, operatorEnv util.OperatorEnvironment) (bool, error) {
 	if pprofEnabledString != "" {
 		pprofEnabled, err := strconv.ParseBool(pprofEnabledString)
 		if err != nil {
-			return false, fmt.Errorf("unable to parse %s environment variable: %w", util.PprofEnabledEnv, err)
+			return false, fmt.Errorf("unable to parse %s environment variable: %w", util.OperatorPprofEnabledEnv, err)
 		}
 
 		return pprofEnabled, nil

pkg/util/constants.go

@@ -152,9 +152,9 @@ const (
 	MinimumScramSha256MdbVersion      = "4.0.0"
 
 	// pprof variables
-	PprofEnabledEnv  = "PPROF_ENABLED"
-	PprofPortEnv     = "PPROF_PORT"
-	PprofDefaultPort = 10081
+	OperatorPprofEnabledEnv  = "MDB_OPERATOR_PPROF_ENABLED"
+	OperatorPprofPortEnv     = "MDB_OPERATOR_PPROF_PORT"
+	OperatorPprofDefaultPort = 10081
 
 	// these were historically used and constituted a security issue—if set they should be changed
 	InvalidKeyFileContents         = "DUMMYFILE"