Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

No need to serialize initial_chain_id field when creating example genesis json? #1068

Closed
9 tasks
windycrypto opened this issue Jun 18, 2018 · 3 comments
Closed
9 tasks
Labels
1a Epic High level concept to be addressed. Description should contain a list referencing child User Stories 2a Discussion Needed Prompt for team to discuss at next stand up. 3c Enhancement Classification indicating a change to the functionality of the existing imlementation 4a Low Priority Priority indicating minimal impact to system/user -OR- an inexpensive workaround exists 6 Protocol Impact flag identifying the blockchain logic, consensus, validation, etc. documentation

Comments

@windycrypto
Copy link
Member

windycrypto commented Jun 18, 2018

I noticed that create_example_genesis() initialize the initial_chain_id to string "BOGUS"'s sha256 hash, while the generated genesis file will be entirely used by egenesis or by --genesis-json option to calculate the real chain id, the initial_chain_id field in the genesis file actually represents a wrong chain id.

It confused me a while, maybe not serialize the initial_chain_id field when creating example genesis? Since new devs&admins would mostly using example genesis as a base, this could confuse them too.

Task List

CORE TEAM TASK LIST

  • Evaluate / Prioritize Feature Request
  • Refine User Stories / Requirements
  • Define Test Cases
  • Design / Develop Solution
  • Perform QA/Testing
  • Update Documentation
@abitmore
Copy link
Member

It's a hard fork change since serialization change will lead to different chain id. IMHO does not worth the efforts. To reduce confusion, we need documentation.

@ryanRfox ryanRfox added 1a Epic High level concept to be addressed. Description should contain a list referencing child User Stories 2a Discussion Needed Prompt for team to discuss at next stand up. 3c Enhancement Classification indicating a change to the functionality of the existing imlementation 4a Low Priority Priority indicating minimal impact to system/user -OR- an inexpensive workaround exists 6 Protocol Impact flag identifying the blockchain logic, consensus, validation, etc. labels Jun 18, 2018
@ryanRfox
Copy link
Contributor

@cifer-lee I attempted to split this Issue into something we can do now #1069 (document the as-is) and later, fix it in the code (#1068).

@pmconrad
Copy link
Contributor

pmconrad commented Mar 1, 2019

This is obsolete since we removed the --create-genesis-json option in #1529 . Closing.

@pmconrad pmconrad closed this as completed Mar 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1a Epic High level concept to be addressed. Description should contain a list referencing child User Stories 2a Discussion Needed Prompt for team to discuss at next stand up. 3c Enhancement Classification indicating a change to the functionality of the existing imlementation 4a Low Priority Priority indicating minimal impact to system/user -OR- an inexpensive workaround exists 6 Protocol Impact flag identifying the blockchain logic, consensus, validation, etc. documentation
Projects
None yet
Development

No branches or pull requests

4 participants