Fix consensus/execution split follower node genesis mismatch

Signed-off-by: Igor Braga <5835477+bragaigor@users.noreply.github.com>

Author: bragaigor

cmd/nitro/init/init.go

@@ -643,16 +643,10 @@ func OpenInitializeExecutionDB(ctx context.Context, consensusNodeEnabled bool, s
 			}
 		} else {
 			// TODO: What should parsedInitMessage be here? Or how should consensus provide such info to execution?
-			serializedChainConfig, err := json.Marshal(chainConfig)
+			parsedInitMessage, err = GetConsensusParsedInitMsg(ctx, true, chainId, l1Client, &rollupAddrs, chainConfig)
 			if err != nil {
 				return executionDB, nil, nil, err
 			}
-			parsedInitMessage = &arbostypes.ParsedInitMessage{
-				ChainId:               chainConfig.ChainID,
-				InitialL1BaseFee:      arbostypes.DefaultInitialL1BaseFee,
-				ChainConfig:           chainConfig,
-				SerializedChainConfig: serializedChainConfig,
-			}
 		}
 
 		l2BlockChain, err = getNewBlockchain(parsedInitMessage, config, initDataReader, chainConfig, genesisArbOSInit, executionDB, cacheConfig, tracer)

cmd/nitro/init/init_test.go

@@ -441,6 +441,7 @@ func TestOpenInitializeExecutionDBIncompatibleStateScheme(t *testing.T) {
 	// opening for the first time doesn't error
 	executionDB, _, blockchain, err := OpenInitializeExecutionDB(
 		ctx,
+		true,
 		stack,
 		&nodeConfig,
 		new(big.Int).SetUint64(nodeConfig.Chain.ID),
@@ -458,6 +459,7 @@ func TestOpenInitializeExecutionDBIncompatibleStateScheme(t *testing.T) {
 	// opening for the second time doesn't error
 	executionDB, _, blockchain, err = OpenInitializeExecutionDB(
 		ctx,
+		true,
 		stack,
 		&nodeConfig,
 		new(big.Int).SetUint64(nodeConfig.Chain.ID),
@@ -476,6 +478,7 @@ func TestOpenInitializeExecutionDBIncompatibleStateScheme(t *testing.T) {
 	nodeConfig.Execution.Caching.StateScheme = rawdb.HashScheme
 	_, _, _, err = OpenInitializeExecutionDB(
 		ctx,
+		true,
 		stack,
 		&nodeConfig,
 		new(big.Int).SetUint64(nodeConfig.Chain.ID),
@@ -704,6 +707,7 @@ func TestOpenInitializeExecutionDbEmptyInit(t *testing.T) {
 
 	executionDB, _, blockchain, err := OpenInitializeExecutionDB(
 		ctx,
+		true,
 		stack,
 		&nodeConfig,
 		new(big.Int).SetUint64(nodeConfig.Chain.ID),
@@ -1059,6 +1063,7 @@ func getInitHelper(t *testing.T, ownerAdress string, chainID uint64, emptyState
 
 	executionDB, _, _, err := OpenInitializeExecutionDB(
 		ctx,
+		true,
 		stack,
 		&nodeConfig,
 		new(big.Int).SetUint64(nodeConfig.Chain.ID),

cmd/nitro/nitro.go

@@ -314,12 +314,13 @@ func mainImpl() int {
 
 		log.Info("connected to l1 chain", "l1url", nodeConfig.ParentChain.Connection.URL, "l1chainid", nodeConfig.ParentChain.ID)
 
-		if consensusNodeEnabled {
-			rollupAddrs, err = chaininfo.GetRollupAddressesConfig(nodeConfig.Chain.ID, nodeConfig.Chain.Name, nodeConfig.Chain.InfoFiles, nodeConfig.Chain.InfoJson)
-			if err != nil {
-				log.Crit("error getting rollup addresses", "err", err)
-			}
+		// TODO: do we need rollupAddrs for execution node? We seem to need it for initialzing chainConfig
+		// if consensusNodeEnabled {
+		rollupAddrs, err = chaininfo.GetRollupAddressesConfig(nodeConfig.Chain.ID, nodeConfig.Chain.Name, nodeConfig.Chain.InfoFiles, nodeConfig.Chain.InfoJson)
+		if err != nil {
+			log.Crit("error getting rollup addresses", "err", err)
 		}
+		// }
 		arbSys, _ := precompilesgen.NewArbSys(types.ArbSysAddress, l1Client)
 		l1Reader, err = headerreader.New(ctx, l1Client, func() *headerreader.Config { return &liveNodeConfig.Get().Node.ParentChainReader }, arbSys)
 		if err != nil {