Skip to content

Cleanup stake_ci uses, factor out multi-epoch leader #5211

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

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

akhinvasara-jumptrading
Copy link
Contributor

  • Clean up several 'uses' of stake_ci where it's not actually used.
  • Implement fd_multi_epoch_leaders_t to handle stake updates and store/query leader schedules for multiple (currently assumed 2) epochs.
  • Replace stake_ci with the new multi_epoch_leaders structure in the handful of cases we just care about leader schedule info, and not shred destinations
  • minor refactors to use the new fd_stake_weight_msg_t struct to encode/decode stake update headers

To minimize blast radius, shred's fd_stake_ci is left unchanged at the cost of some code duplication, which ideally would be avoided by having stake_ci use the new multi_epoch_leader

Each consumer is modified in a separate commit, for easy reverts if needed. Review best done commit by commit, focusing on 60a75b9.

@akhinvasara-jumptrading akhinvasara-jumptrading marked this pull request as ready for review May 30, 2025 16:21
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.

1 participant