Skip to content

[NET-673] [Alert Ok2jmG] ink-sepolia_No_Dumper_Data#485

Open
elina-chertova wants to merge 2 commits into
open-betafrom
alert-fix/ok2jmg-ink-sepolia-no-dumper-data-squid-sdk
Open

[NET-673] [Alert Ok2jmG] ink-sepolia_No_Dumper_Data#485
elina-chertova wants to merge 2 commits into
open-betafrom
alert-fix/ok2jmg-ink-sepolia-no-dumper-data-squid-sdk

Conversation

@elina-chertova

Copy link
Copy Markdown
Contributor

Automated fix proposal for alert Ok2jmG.

  • Alert: ink-sepolia_No_Dumper_Data
  • Base branch: open-beta
  • Investigation: /root/alert/incident-agent/agent-system/data/investigations/Ok2jmG
  • Report: /root/alert/incident-agent/agent-system/data/investigations/Ok2jmG/report.html

Reviewer quick view

  • Scope: 2 file(s) in evm

  • Root cause (agent): not explicitly captured

  • Summary: ink-sepolia_No_Dumper_Data (Ok2jmG) — Root Cause Confirmed

    Root cause: dump-ink-sepolia-0 is in CrashLoopBackOff (124 restarts, exit 1) because OP Stack
    deposit transactions (type 0x7e) legitimately omit nonce, but rpc-data.ts declares nonce:
    SMALL_QTY as mandatory — causing DataValidationError on every block containing a 0x7e tx.

    Fix — 2 files in evm/evm-rpc/src/ (squid-sdk, open-beta):

    1. rpc-data.ts (1 line): nonce: SMALL_QTY → nonce: option(SMALL_QTY)
    2. verification.ts (15 sites): BigInt(tx.nonce) → BigInt(assertNotNull(tx.nonce, 'tx.nonce is
      missing')) — prevents TypeScript compile failure now that nonce is optional

    Operator action required: After PR merges and new evm-dump image is built, update ink-sepolia.yaml
    image tag from 87fd334 to the new digest.

    Verdict: accept — implementer patch is correct and complete; root cause fully explained by direct
    log + code evidence.

Fix metadata

  • Fix class: rca_fix
  • Confidence: high
  • Evidence basis: logs, code
  • Falsification: If rpc-data.ts line 221 already reads nonce: option(SMALL_QTY) in
  • Follow-up: Merge PR → build new evm-dump image → update ink-sepolia.yaml image
    (Generated by the terminal-debate agent — values reflect the agent's self-assessment, not a verified verdict. Use them as a starting point for review.)

Summary

ink-sepolia_No_Dumper_Data (Ok2jmG) — Root Cause Confirmed

Root cause: dump-ink-sepolia-0 is in CrashLoopBackOff (124 restarts, exit 1) because OP Stack
deposit transactions (type 0x7e) legitimately omit nonce, but rpc-data.ts declares nonce:
SMALL_QTY as mandatory — causing DataValidationError on every block containing a 0x7e tx.

Fix — 2 files in evm/evm-rpc/src/ (squid-sdk, open-beta):

  1. rpc-data.ts (1 line): nonce: SMALL_QTY → nonce: option(SMALL_QTY)
  2. verification.ts (15 sites): BigInt(tx.nonce) → BigInt(assertNotNull(tx.nonce, 'tx.nonce is
    missing')) — prevents TypeScript compile failure now that nonce is optional

Operator action required: After PR merges and new evm-dump image is built, update ink-sepolia.yaml
image tag from 87fd334 to the new digest.

Verdict: accept — implementer patch is correct and complete; root cause fully explained by direct
log + code evidence.

Risk & rollout

  • Suggested rollout: canary / one-network-first, then broader rollout after signal is stable.
  • Rollback: revert this PR (or restore previous config values/files) if the incident signal worsens.

Reproduction status

Incident behavior was reproduced or corroborated strongly enough for a non-hypothesis fix proposal.

Validation checklist

  • Verify the original incident signal improves (logs/metrics/alerts) after deploy.
  • Verify no regression on sibling networks/providers/services touched by this change.
  • Confirm queue / delivery pipeline status returns to expected steady state.

Changed files

  • evm/evm-rpc/src/rpc-data.ts
  • evm/evm-rpc/src/verification.ts

Notify

cc @tmcgroul (automation opened this PR.)

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