Skip to content

fix: wait for LEP-6 proof rows before report - #327

Merged
mateeullahmalik merged 1 commit into
masterfrom
fix/storage-challenge-report-sequencing-testnet
Sep 17, 2026
Merged

mateeullahmalik merged 1 commit into
masterfrom
fix/storage-challenge-report-sequencing-testnet

Conversation

@j-rafique

@j-rafique j-rafique commented Sep 16, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Adds a bounded SHADOW/SOFT wait before host_reporter submits an epoch report with assigned targets but zero LEP-6 proof rows.
  • Adds non-destructive CountResults(epochID) to the LEP-6 result buffer so host_reporter can check for pending rows without draining them.
  • Upgrades duplicate-submit-with-proof-row discard logging to WARN and includes proof result class counts.

Why

Testnet storage challenge reports were being submitted at epoch start before LEP-6 dispatch could finish compound proof work.

Late proof rows could then be discarded because the epoch report had already been accepted and the retry hit duplicate-report handling.

This keeps host-report liveness by using a bounded wait window, then still submitting empty proof rows after the window expires.

Validation

  • go test ./supernode/host_reporter ./supernode/storage_challenge ./pkg/metrics/lep6 ./supernode/status -count=1 — PASS
  • go test ./... -count=1 — PASS
  • git diff --check — PASS before commit

Rollout notes

  • Canary on the 2 testnet nodes already showing non-zero LEP-6 dispatch counters.
  • Observe 3–5 epochs and confirm storage_proof_results rows land on-chain when eligible tickets are hit.
  • After canary, enable storage_challenge.lep6.enabled=true across the fleet in batches.

Follow-up

  • Expose existing DispatchInternalFailuresTotal and DispatchSignFailuresTotal in status for better operator visibility.

@j-rafique j-rafique self-assigned this Sep 16, 2026
@j-rafique
j-rafique force-pushed the fix/storage-challenge-report-sequencing-testnet branch from 27015ff to 6dbd50e Compare September 16, 2026 14:09
@mateeullahmalik
mateeullahmalik merged commit 589a11c into master Sep 17, 2026
23 checks passed
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