set uint later

Signed-off-by: Tim Vaillancourt <tim@timvaillancourt.com>

go/flags/endtoend/vtorc.txt

@@ -35,7 +35,7 @@ Flags:
       --config-persistence-min-interval duration                    minimum interval between persisting dynamic config changes back to disk (if no change has occurred, nothing is done). (default 1s)
       --config-type string                                          Config file type (omit to infer config type from file extension).
       --consul_auth_static_file string                              JSON File to read the topos/tokens from.
-      --discovery-max-concurrency uint                              Number of goroutines dedicated to doing hosts discovery (default 300)
+      --discovery-max-concurrency int                               Number of goroutines dedicated to doing hosts discovery (default 300)
       --emit_stats                                                  If set, emit stats to push-based monitoring and stats backends
       --grpc_auth_static_client_creds string                        When using grpc_static_auth in the server, this file provides the credentials to use to authenticate with server.
       --grpc_compression string                                     Which protocol to use for compressing gRPC. Default: nothing. Supported: snappy

go/vt/vtorc/config/config.go

@@ -47,24 +47,24 @@ const (
 )
 
 var (
-	discoveryMaxConcurrency        uint = 300
-	sqliteDataFile                      = "file::memory:?mode=memory&cache=shared"
-	instancePollTime                    = 5 * time.Second
-	snapshotTopologyInterval            = 0 * time.Hour
-	reasonableReplicationLag            = 10 * time.Second
-	auditFileLocation                   = ""
-	auditToBackend                      = false
-	auditToSyslog                       = false
-	auditPurgeDuration                  = 7 * 24 * time.Hour // Equivalent of 7 days
-	allowRecovery                       = true
-	recoveryPeriodBlockDuration         = 30 * time.Second
-	preventCrossCellFailover            = false
-	waitReplicasTimeout                 = 30 * time.Second
-	tolerableReplicationLag             = 0 * time.Second
-	topoInformationRefreshDuration      = 15 * time.Second
-	recoveryPollDuration                = 1 * time.Second
-	ersEnabled                          = true
-	convertTabletsWithErrantGTIDs       = false
+	discoveryMaxConcurrency        = 300
+	sqliteDataFile                 = "file::memory:?mode=memory&cache=shared"
+	instancePollTime               = 5 * time.Second
+	snapshotTopologyInterval       = 0 * time.Hour
+	reasonableReplicationLag       = 10 * time.Second
+	auditFileLocation              = ""
+	auditToBackend                 = false
+	auditToSyslog                  = false
+	auditPurgeDuration             = 7 * 24 * time.Hour // Equivalent of 7 days
+	allowRecovery                  = true
+	recoveryPeriodBlockDuration    = 30 * time.Second
+	preventCrossCellFailover       = false
+	waitReplicasTimeout            = 30 * time.Second
+	tolerableReplicationLag        = 0 * time.Second
+	topoInformationRefreshDuration = 15 * time.Second
+	recoveryPollDuration           = 1 * time.Second
+	ersEnabled                     = true
+	convertTabletsWithErrantGTIDs  = false
 )
 
 // RegisterFlags registers the flags required by VTOrc
@@ -86,7 +86,7 @@ func RegisterFlags(fs *pflag.FlagSet) {
 	fs.DurationVar(&recoveryPollDuration, "recovery-poll-duration", recoveryPollDuration, "Timer duration on which VTOrc polls its database to run a recovery")
 	fs.BoolVar(&ersEnabled, "allow-emergency-reparent", ersEnabled, "Whether VTOrc should be allowed to run emergency reparent operation when it detects a dead primary")
 	fs.BoolVar(&convertTabletsWithErrantGTIDs, "change-tablets-with-errant-gtid-to-drained", convertTabletsWithErrantGTIDs, "Whether VTOrc should be changing the type of tablets with errant GTIDs to DRAINED")
-	fs.UintVar(&discoveryMaxConcurrency, "discovery-max-concurrency", discoveryMaxConcurrency, "Number of goroutines dedicated to doing hosts discovery")
+	fs.IntVar(&discoveryMaxConcurrency, "discovery-max-concurrency", discoveryMaxConcurrency, "Number of goroutines dedicated to doing hosts discovery")
 }
 
 // Configuration makes for vtorc configuration input, which can be provided by user via JSON formatted file.
@@ -125,7 +125,7 @@ var readFileNames []string
 // UpdateConfigValuesFromFlags is used to update the config values from the flags defined.
 // This is done before we read any configuration files from the user. So the config files take precedence.
 func UpdateConfigValuesFromFlags() {
-	Config.DiscoveryMaxConcurrency = discoveryMaxConcurrency
+	Config.DiscoveryMaxConcurrency = uint(discoveryMaxConcurrency)
 	Config.SQLite3DataFile = sqliteDataFile
 	Config.InstancePollSeconds = uint(instancePollTime / time.Second)
 	Config.InstancePollSeconds = uint(instancePollTime / time.Second)