Skip to content

Commit 36e1a01

Browse files
OisinKynenflaig
authored andcommitted
fix: change hoodi's SECONDS_PER_ETH1_BLOCK config from 14s to 12s. (#7609)
**Motivation** Lodestar VC does not start with other BNs on Hoodi because of mismatching local and remote specs. ![lodestarestartingr](https://github.com/user-attachments/assets/61e40278-174c-404b-8d66-e5549b594778) **Description** The hoodi spec has changed the `SECONDS_PER_ETH1_BLOCKS:` value to 12 https://github.com/eth-clients/hoodi/blob/main/metadata/config.yaml#L53 while in lodestar, it [pulls the mainnet spec](https://github.com/ChainSafe/lodestar/blob/unstable/packages/config/src/chainConfig/networks/hoodi.ts#L9) which is still 14. **Steps to test or reproduce** ```sh # Curl a config endpoint on a hoodi BN and observe that the SECONDS_PER_ETH1_BLOCK == 12 not the expected 14. wget -qO- http://localhost:4000/eth/v1/config/spec ``` > ... "SECONDS_PER_ETH1_BLOCK":"12" ... It might be appropriate to update the mainnet or minimal spec at some point if the plan is to change it at Pectra. For now I put the change in hoodi.ts --------- Co-authored-by: Nico Flaig <[email protected]>
1 parent 40d04de commit 36e1a01

File tree

1 file changed

+5
-0
lines changed
  • packages/config/src/chainConfig/networks

1 file changed

+5
-0
lines changed

packages/config/src/chainConfig/networks/hoodi.ts

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -39,6 +39,11 @@ export const hoodiChainConfig: ChainConfig = {
3939
FULU_FORK_VERSION: b("0x70000910"),
4040
FULU_FORK_EPOCH: Infinity,
4141

42+
// Time parameters
43+
// ---------------------------------------------------------------
44+
// 12 (update from older mainnet default of 14)
45+
SECONDS_PER_ETH1_BLOCK: 12,
46+
4247
// Deposit contract
4348
// ---------------------------------------------------------------
4449
DEPOSIT_CHAIN_ID: 560048,

0 commit comments

Comments
 (0)