config: consistent default MinerTargetBlockTimeSeconds

vocdoni · May 8, 2024 · 8f6c02b · 8f6c02b
1 parent a29dcbc
commit 8f6c02b
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 2 deletions.
diff --git a/cmd/node/main.go b/cmd/node/main.go
@@ -201,7 +201,7 @@ func loadConfig() *config.Config {
 	flag.Bool("vochainStateSyncFetchParamsFromRPC", true,
 		"allow statesync to fetch TrustHash and TrustHeight from the first RPCServer")
 
-	flag.Int("vochainMinerTargetBlockTimeSeconds", 10,
+	flag.Int("vochainMinerTargetBlockTimeSeconds", config.DefaultMinerTargetBlockTimeSeconds,
 		"vochain consensus block time target (in seconds)")
 	flag.Bool("vochainSkipPreviousOffchainData", false,
 		"if enabled the census downloader will import all existing census")

diff --git a/config/defaults.go b/config/defaults.go
@@ -0,0 +1,6 @@
+package config
+
+// These consts are defaults used in VochainCfg
+const (
+	DefaultMinerTargetBlockTimeSeconds = 10
+)
diff --git a/vochain/start.go b/vochain/start.go
@@ -85,7 +85,7 @@ func newTendermint(app *BaseApplication,
 	}
 
 	// consensus config
-	blockTime := 8
+	blockTime := config.DefaultMinerTargetBlockTimeSeconds
 	if localConfig.MinerTargetBlockTimeSeconds > 0 {
 		blockTime = localConfig.MinerTargetBlockTimeSeconds
 	}