Skip to content

Comments

Implement PDP schedule execution path and config wiring#621

Open
anjor wants to merge 4 commits intomainfrom
feat/pdp-scheduling-runner
Open

Implement PDP schedule execution path and config wiring#621
anjor wants to merge 4 commits intomainfrom
feat/pdp-scheduling-runner

Conversation

@anjor
Copy link
Collaborator

@anjor anjor commented Feb 20, 2026

Summary

  • implement an executable PDP scheduling path in deal-pusher (runPDPSchedule) with sequencing: proof-set ensure -> root queue -> tx confirmation -> deal persistence
  • add configurable PDP scheduling parameters to run deal-pusher flags: pdp-batch-size, pdp-gas-limit, pdp-confirmation-depth, pdp-poll-interval
  • add DealPusher option hooks for PDP dependencies/config and switch default schedule deal-type inference to delegated provider addresses
  • expand wiring tests to validate PDP scheduling behavior and default deal-type inference

Validation

  • go test ./service/dealpusher
  • go test ./cmd/run

@anjor anjor requested a review from parkan February 20, 2026 14:25
@parkan
Copy link
Collaborator

parkan commented Feb 20, 2026

based on test failure let's park this momentarily while I push up the work related to wallet extraction from db, early next week

@parkan
Copy link
Collaborator

parkan commented Feb 20, 2026

see #622 and data-preservation-programs/go-synapse#4

both need work but I need to sign off for the week

@anjor anjor force-pushed the feat/pdp-scheduling-runner branch 2 times, most recently from c93e4e2 to 1641a4e Compare February 24, 2026 14:18
@parkan
Copy link
Collaborator

parkan commented Feb 24, 2026

a few changes needed here to cleanly separate the Actor and Wallet concepts:

  • schedule.Preparation.Wallets → schedule.Preparation.Actors (or add a separate Wallet association on Preparation for PDP)
  • EnsureProofSet / QueueAddRoots now take signer.EVMSigner — caller resolves wallet→keystore→EVMSigner before calling
  • model.Wallet no longer has PrivateKey inline — use keystore.EVMSigner(ks, wallet) to get a signer
  • The WalletChooser picks actors (for market deals); PDP path needs a different resolution to get a wallet+signer

these are basically the changes from #621 (comment) that I mentioned as critical

@anjor anjor force-pushed the feat/pdp-scheduling-runner branch from 1641a4e to c03ce04 Compare February 24, 2026 16:35
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.

2 participants