chore: set replicated as empty string when not set (#311)

Signed-off-by: Prashant Shahi <[email protected]>
SigNoz · Apr 8, 2024 · 5953729 · 5953729
1 parent 2916d1c
commit 5953729
Showing 1 changed file with 7 additions and 5 deletions.
diff --git a/cmd/signozschemamigrator/migrate.go b/cmd/signozschemamigrator/migrate.go
@@ -107,12 +107,14 @@ func main() {
 
 	// set SIGNOZ_REPLICATED env var so that golang-migrate can use it
 	// the value of this env would replace all occurences of {{.SIGNOZ_REPLICATED}} in the migration files
+	signozReplicated := ""
+	logger.Info("Setting env var SIGNOZ_REPLICATED", zap.Bool("replication", replicationEnabled))
 	if replicationEnabled {
-		logger.Info("Setting env var SIGNOZ_REPLICATED", zap.Bool("replication", replicationEnabled))
-		err = os.Setenv("SIGNOZ_REPLICATED", "Replicated")
-		if err != nil {
-			logger.Fatal("Failed to set env var SIGNOZ_REPLICATED", zap.Error(err))
-		}
+		signozReplicated = "Replicated"
+	}
+	err = os.Setenv("SIGNOZ_REPLICATED", signozReplicated)
+	if err != nil {
+		logger.Fatal("Failed to set env var SIGNOZ_REPLICATED", zap.Error(err))
 	}
 
 	manager, err := migrationmanager.New(dsn, clusterName, disableDurationSortFeature, disableTimestampSortFeature, verboseLoggingEnabled, replicationEnabled)