Renamed obsolete PLGM_REPLICA_SET env var PLGM_REPLICASET_NAME

Author: daniel-percona

README.md

@@ -94,7 +94,7 @@ Environment Variables (Overrides):
   PLGM_USERNAME                Database User
   PLGM_PASSWORD                Database Password (Recommended: Use Prompt)
   PLGM_DIRECT_CONNECTION       Force direct connection (true/false)
-  PLGM_REPLICA_SET             Replica Set name
+  PLGM_REPLICASET_NAME             Replica Set name
   PLGM_READ_PREFERENCE         nearest
 
  [Workload Core]
@@ -206,7 +206,7 @@ You can override any setting in `config.yaml` using environment variables. This
 | **Connection** | | | |
 | `uri` | `PLGM_URI` | Target MongoDB connection URI | `mongodb://user:pass@host:27017` |
 | `direct_connection` | `PLGM_DIRECT_CONNECTION` | Force direct connection (bypass topology discovery) | `true` |
-| `replicaset_name` | `PLGM_REPLICA_SET` | Replica Set name (required for sharded clusters/RS) | `rs0` |
+| `replicaset_name` | `PLGM_REPLICASET_NAME` | Replica Set name (required for sharded clusters/RS) | `rs0` |
 | `read_preference` | `PLGM_READ_PREFERENCE` | By default, an application directs its read operations to the primary member in a replica set. You can specify a read preference to send read operations to secondaries. | `nearest` |
 | `username` | `PLGM_USERNAME` |	Database User | `admin` |
 | ***can not be set via config*** | `PLGM_PASSWORD` |	Database Password (if not set, plgm will prompt) | `password123` |

cmd/plgm/main.go

@@ -45,7 +45,7 @@ func main() {
 		fmt.Fprintf(os.Stderr, "  %-35s %s\n", "PLGM_USERNAME", "Database User")
 		fmt.Fprintf(os.Stderr, "  %-35s %s\n", "PLGM_PASSWORD", "Database Password (Recommended: Use Prompt)")
 		fmt.Fprintf(os.Stderr, "  %-35s %s\n", "PLGM_DIRECT_CONNECTION", "Force direct connection (true/false)")
-		fmt.Fprintf(os.Stderr, "  %-35s %s\n", "PLGM_REPLICA_SET", "Replica Set name")
+		fmt.Fprintf(os.Stderr, "  %-35s %s\n", "PLGM_REPLICASET_NAME", "Replica Set name")
 		fmt.Fprintf(os.Stderr, "  %-35s %s\n", "PLGM_READ_PREFERENCE", "nearest")
 
 		fmt.Fprintf(os.Stderr, "\n [Workload Core]\n")

config.yaml

@@ -15,7 +15,7 @@ connection_params:
   username: "root"
   auth_source: admin             # Database to authenticate against
   direct_connection: true        # Connect directly to the host (bypass topology discovery if single node)
-  replicaset_name: rs            # Required if connecting to a Replica Set
+  replicaset_name: rs            # Required if connecting to a Replica Set, set to "" if connecting to a sharded cluster
   read_preference: primary
   connection_timeout: 30         # Time (seconds) to wait for an initial connection
   server_selection_timeout: 15   # Time (seconds) to wait to find a suitable server for an op

internal/config/config.go

@@ -120,7 +120,6 @@ func applyEnvOverrides(cfg *AppConfig) {
 	}
 
 	// 2. Default Workload (Explicit Override)
-	// If the user sets this Env Var, it takes precedence over everything.
 	if v := os.Getenv("PLGM_DEFAULT_WORKLOAD"); v != "" {
 		if b, err := strconv.ParseBool(v); err == nil {
 			cfg.DefaultWorkload = b
@@ -141,9 +140,11 @@ func applyEnvOverrides(cfg *AppConfig) {
 			cfg.ConnectionParams.DirectConnection = b
 		}
 	}
-	if v, exists := os.LookupEnv("PLGM_REPLICA_SET"); exists {
+
+	if v, exists := os.LookupEnv("PLGM_REPLICASET_NAME"); exists {
 		cfg.ConnectionParams.ReplicaSetName = v
 	}
+
 	if v, exists := os.LookupEnv("PLGM_READ_PREFERENCE"); exists {
 		cfg.ConnectionParams.ReadPreference = v
 	}
@@ -268,7 +269,6 @@ func applyEnvOverrides(cfg *AppConfig) {
 }
 
 func normalizePercentages(cfg *AppConfig) {
-	// If transactions are disabled, ignore their percentage entirely
 	if !cfg.UseTransactions {
 		cfg.TransactionPercent = 0
 	}