Skip to content

Comments

fix(chainspec): regenerate dev.json with v2 validator config#2746

Open
jxom wants to merge 1 commit intomainfrom
fix/regenerate-dev-genesis
Open

fix(chainspec): regenerate dev.json with v2 validator config#2746
jxom wants to merge 1 commit intomainfrom
fix/regenerate-dev-genesis

Conversation

@jxom
Copy link
Member

@jxom jxom commented Feb 17, 2026

Summary

Fixes a regression introduced in cd46b33 where dev.json was not regenerated after the validator config v2 genesis changes. The embedded dev genesis had v1 validator state at 0xCCCC...0000, but with t2Time: 0 the node expects v2 validators at 0xCCCC...0001. This caused --chain dev nodes to fail to produce blocks.

Root Cause

cd46b33 changed genesis initialization to call initialize_validator_config_v2 when t2Time == 0, and updated test-genesis.json accordingly, but forgot to regenerate the embedded dev.json.

Fix

Regenerated dev.json with:

cargo x generate-genesis -o dev.json --accounts 10 --no-dkg-in-genesis

Testing

cargo test -p tempo-chainspec — 15/15 pass

@jxom jxom force-pushed the fix/regenerate-dev-genesis branch from 6f89bc0 to eae7fd2 Compare February 17, 2026 20:55
cd46b33 changed genesis initialization to use ValidatorConfigV2 when
t2Time == 0, but dev.json was not regenerated. The embedded dev genesis
still had v1 validator state at 0xCCCC...0000, causing --chain dev nodes
to fail to produce blocks.

Regenerated with:
  cargo x generate-genesis -o dev.json --accounts 10 --no-dkg-in-genesis

Also updated the doc comment in spec.rs to match the correct command.
@jxom jxom force-pushed the fix/regenerate-dev-genesis branch from eae7fd2 to 542980e Compare February 17, 2026 20:56
Copy link
Contributor

@SuperFluffy SuperFluffy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should probably take into account #2761

Unsure, but blocking just in case.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants