config: drop obsolete SecondsPerBlock protocol configuration

Signed-off-by: Roman Khimov <roman@nspcc.ru>

ROADMAP.md

@@ -26,16 +26,6 @@ APIs/commands/configurations will be removed and here is a list of scheduled
 breaking changes. Consider changing your code/scripts/configurations if you're
 using anything mentioned here.
 
-## SecondsPerBlock protocol configuration
-
-With 0.100.0 version SecondsPerBlock protocol configuration setting was
-deprecated and replaced by a bit more generic and precise TimePerBlock
-(allowing for subsecond time). An informational message is printed on node
-startup to inform about this, it's very easy to deal with this configuration
-change, just replace one line.
-
-Removal of SecondsPerBlock is scheduled for May-June 2023 (~0.103.0 release).
-
 ## Services/node address and port configuration
 
 Version 0.100.0 of NeoGo introduces a multiple binding addresses capability to

docs/node-configuration.md

@@ -363,7 +363,6 @@ protocol-related settings described in the table below.
 | RemoveUntraceableBlocks | `bool`| `false` | Denotes whether old blocks should be removed from cache and database. If enabled, then only the last `MaxTraceableBlocks` are stored and accessible to smart contracts. Old MPT data is also deleted in accordance with `GarbageCollectionPeriod` setting. If enabled along with `P2PStateExchangeExtensions`, then old blocks and MPT states will be removed up to the second latest state synchronisation point (see `StateSyncInterval`). | This setting is deprecated in favor of the same setting in the ApplicationConfiguration and will be removed in future node versions. If both settings are used, setting any of them to true enables the function. |
 | ReservedAttributes | `bool` | `false` | Allows to have reserved attributes range for experimental or private purposes. |
 | SaveStorageBatch | `bool` | `false` | Enables storage batch saving before every persist. It is similar to StorageDump plugin for C# node. | This setting is deprecated in favor of the same setting in the ApplicationConfiguration and will be removed in future node versions. If both settings are used, setting any of them to true enables the function. |
-| SecondsPerBlock | `int` | `15` | Minimal time that should pass before next block is accepted. Deprecated: please use TimePerBlock setting (which overrides anything set here), SecondsPerBlock will be removed in future versions. |
 | SeedList | `[]string` | [] | List of initial nodes addresses used to establish connectivity. |
 | StandbyCommittee | `[]string` | [] | List of public keys of standby committee validators are chosen from. |
 | StateRootInHeader | `bool` | `false` | Enables storing state root in block header. | Experimental protocol extension! |

pkg/config/protocol_config.go

@@ -68,10 +68,6 @@ type (
 		//
 		// Deprecated: please use the same setting in the ApplicationConfiguration, this field will be removed in future versions.
 		SaveStorageBatch bool     `yaml:"SaveStorageBatch"`
-		// SecondsPerBlock is the time interval (in seconds) between blocks that consensus nodes work with.
-		//
-		// Deprecated: replaced by TimePerBlock, to be removed in future versions.
-		SecondsPerBlock  int      `yaml:"SecondsPerBlock"`
 		SeedList         []string `yaml:"SeedList"`
 		StandbyCommittee []string `yaml:"StandbyCommittee"`
 		// StateRooInHeader enables storing state root in block header.
@@ -260,7 +256,6 @@ func (p *ProtocolConfiguration) Equals(o *ProtocolConfiguration) bool {
 		p.RemoveUntraceableBlocks != o.RemoveUntraceableBlocks ||
 		p.ReservedAttributes != o.ReservedAttributes ||
 		p.SaveStorageBatch != o.SaveStorageBatch ||
-		p.SecondsPerBlock != o.SecondsPerBlock ||
 		p.StateRootInHeader != o.StateRootInHeader ||
 		p.StateSyncInterval != o.StateSyncInterval ||
 		p.TimePerBlock != o.TimePerBlock ||

pkg/core/blockchain.go

@@ -257,16 +257,10 @@ func NewBlockchain(s storage.Store, cfg config.Blockchain, log *zap.Logger) (*Bl
 			zap.Uint16("MaxTransactionsPerBlock", cfg.MaxTransactionsPerBlock))
 	}
 	if cfg.TimePerBlock <= 0 {
-		if cfg.SecondsPerBlock > 0 { //nolint:staticcheck // SA1019: cfg.SecondsPerBlock is deprecated
-			cfg.TimePerBlock = time.Duration(cfg.SecondsPerBlock) * time.Second //nolint:staticcheck // SA1019: cfg.SecondsPerBlock is deprecated
-			log.Info("TimePerBlock is not set, using deprecated SecondsPerBlock setting, consider updating your config",
-				zap.Duration("TimePerBlock", cfg.TimePerBlock))
-		} else {
 		cfg.TimePerBlock = defaultTimePerBlock
 		log.Info("TimePerBlock is not set or wrong, using default value",
 			zap.Duration("TimePerBlock", cfg.TimePerBlock))
 	}
-	}
 	if cfg.MaxValidUntilBlockIncrement == 0 {
 		const timePerDay = 24 * time.Hour
 

pkg/network/server_config.go

@@ -84,10 +84,6 @@ type (
 func NewServerConfig(cfg config.Config) (ServerConfig, error) {
 	appConfig := cfg.ApplicationConfiguration
 	protoConfig := cfg.ProtocolConfiguration
-	timePerBlock := protoConfig.TimePerBlock
-	if timePerBlock == 0 && protoConfig.SecondsPerBlock > 0 { //nolint:staticcheck // SA1019: protoConfig.SecondsPerBlock is deprecated
-		timePerBlock = time.Duration(protoConfig.SecondsPerBlock) * time.Second //nolint:staticcheck // SA1019: protoConfig.SecondsPerBlock is deprecated
-	}
 	dialTimeout := appConfig.P2P.DialTimeout
 	if dialTimeout == 0 && appConfig.DialTimeout > 0 { //nolint:staticcheck // SA1019: appConfig.DialTimeout is deprecated
 		dialTimeout = time.Duration(appConfig.DialTimeout) * time.Second //nolint:staticcheck // SA1019: appConfig.DialTimeout is deprecated
@@ -141,7 +137,7 @@ func NewServerConfig(cfg config.Config) (ServerConfig, error) {
 		MaxPeers:           maxPeers,
 		AttemptConnPeers:   attemptConnPeers,
 		MinPeers:           minPeers,
-		TimePerBlock:       timePerBlock,
+		TimePerBlock:       protoConfig.TimePerBlock,
 		OracleCfg:          appConfig.Oracle,
 		P2PNotaryCfg:       appConfig.P2PNotary,
 		StateRootCfg:       appConfig.StateRoot,