Skip to content
New issue

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

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

Already on GitHub? # to your account

move genesis docs to genesis package and add note on initial staker handling #3574

Merged
merged 7 commits into from
Jan 6, 2025

Conversation

aaronbuchwald
Copy link
Collaborator

This PR moves the documentation for genesis JSON into the genesis/ package and adds an explanation of how funds for initialStakers are handled.

Specifically, if an address included in the genesis is marked as an initialStaker then the UTXOs described by its unlockSchedule are included in the the initial staker transaction instead of creating a UTXO in the genesis state of the P-Chain.

For reference, the flow is:

  1. Initial stakers are added to a set here
  2. When creating initial P-Chain UTXOs, we skip over any address included in initialStakers ref
  3. We create the initial validators with their locked stakes here

Co-authored-by: Stephen Buttolph <stephen@avalabs.org>
Signed-off-by: aaronbuchwald <aaron.buchwald56@gmail.com>
Copy link

github-actions bot commented Jan 5, 2025

This PR has become stale because it has been open for 30 days with no activity. Adding the lifecycle/frozen label will cause this PR to ignore lifecycle events.

Signed-off-by: Meaghan FitzGerald <meag.fitz@avalabs.org>
Signed-off-by: Meaghan FitzGerald <meag.fitz@avalabs.org>
@meaghanfitzgerald meaghanfitzgerald changed the base branch from master to api-docs-edits January 6, 2025 21:26
@meaghanfitzgerald
Copy link
Contributor

merging to combine with other edits to config docs

@meaghanfitzgerald meaghanfitzgerald merged commit 37d065d into api-docs-edits Jan 6, 2025
20 checks passed
@meaghanfitzgerald meaghanfitzgerald deleted the improve-genesis-docs branch January 6, 2025 21:29
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants