Skip to content

Conversation

vedhavyas
Copy link
Contributor

@vedhavyas vedhavyas commented Aug 12, 2025

Adds chronos testnet chainspec.

@jfrank-summit @nazar-pc I have left some TODOs in there would appreciate some info on those.

@nazar-pc as for the pot_external_entropy, can I pick a random 16byte entropy or are there any specific steps you would like to do to derive one

Once all TODOs are finalized, will generate a raw chainspec and remove any remnants of taurus

  • Creates Chronos tesnet raw chain spec with bootstrap nodes included
  • Removed Taurus and replaced with Chronos
  • This commit removes compiled chain spec for chronos post raw spec generation

Code contributor checklist:

@nazar-pc
Copy link
Member

@nazar-pc as for the pot_external_entropy, can I pick a random 16byte entropy or are there any specific steps you would like to do to derive one

Could technically be anything, IIRC it is not limited to 16 bytes either. Could pick the most recent Bitcoin block hash like mainnet did. Also entropy is technically optional.

@vedhavyas
Copy link
Contributor Author

Alright, @jfrank-summit can you do a final pass before I generate a raw spec please?

I have given Council members some balance since Sudo is derived from their Multisig and Sudo 1000 Ai3.

jfrank-summit
jfrank-summit previously approved these changes Aug 13, 2025
Copy link
Member

@jfrank-summit jfrank-summit left a comment

Choose a reason for hiding this comment

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

LGTM

@vedhavyas vedhavyas marked this pull request as ready for review August 14, 2025 07:53
@vedhavyas vedhavyas requested a review from teor2345 as a code owner August 14, 2025 07:53
Copy link

🛡️ Immunefi PR Reviews

We noticed that your project isn't set up for automatic code reviews. If you'd like this PR reviewed by the Immunefi team, you can request it manually using the link below:

🔗 Send this PR in for review

Once submitted, we'll take care of assigning a reviewer and follow up here.

@vedhavyas
Copy link
Contributor Author

This is good for review

Copy link
Member

@jfrank-summit jfrank-summit left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@teor2345 teor2345 left a comment

Choose a reason for hiding this comment

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

Looks good to me!

There are some confusing names in the code that are used for testnet and mainnet, like:

  • CI: Generate testnet chain specifications / domain genesis storages
  • Code: mainnet_config(get_testnet_genesis_by_spec_id(SpecId::Mainnet))

But they aren't a blocker for this PR, I'll fix them separately.

@vedhavyas vedhavyas added this pull request to the merge queue Aug 15, 2025
Merged via the queue into main with commit 4f9c7ed Aug 15, 2025
13 checks passed
@vedhavyas vedhavyas deleted the chronos_testnet branch August 15, 2025 09:04
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.

5 participants