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

Implement basic validator data column custody #6767

Open
5 tasks
jimmygchen opened this issue Jan 8, 2025 · 0 comments
Open
5 tasks

Implement basic validator data column custody #6767

jimmygchen opened this issue Jan 8, 2025 · 0 comments
Assignees
Labels
das Data Availability Sampling fulu Required for the upcoming Fulu hard fork

Comments

@jimmygchen
Copy link
Member

jimmygchen commented Jan 8, 2025

Description

Implement spec: ethereum/consensus-specs#3871

This isn't mandatory for peerdas-devnet-4, but the change is compatible with the devnet - it means potentially storing more columns when running a node with validators. The spec hasn't been merged yet but client teams have started looking into this. We could either wait for the spec to be merged or implement this behind a CLI flag for testing purpose until the spec change is merged.

Tasks

Following the design in https://hackmd.io/@dapplion/validator_custody_no_backfill

  • make all codepaths read the CGC from a Mutex,
  • carefully persist new consistency data to disk
cgc_updates: Vec<(Slot, u64)>,
backfilled_cgc: (Slot, u64),
  • restart the backfill under certain conditions
  • update gossip subscriptions
  • cache the to-advertise CGC somewhere and update ENR + status at that moment
@jimmygchen jimmygchen added the das Data Availability Sampling label Jan 8, 2025
@jimmygchen jimmygchen self-assigned this Jan 15, 2025
@dapplion dapplion mentioned this issue Jan 25, 2025
52 tasks
@jimmygchen jimmygchen added the fulu Required for the upcoming Fulu hard fork label Jan 31, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
das Data Availability Sampling fulu Required for the upcoming Fulu hard fork
Projects
None yet
Development

No branches or pull requests

1 participant