Skip to content

Conversation

@maximopalopoli
Copy link
Collaborator

@maximopalopoli maximopalopoli commented Dec 18, 2025

Description

This PR adds a retry logic with exponential backoff.

How to test

Normal workflow

  1. Start Ethereum Package:
make ethereum_package_start
  1. Start gateway and payments poller:
make agg_mode_gateway_start_local
make agg_mode_payments_poller_start_local
  1. Send funds and a sp1 proof:
make agg_mode_gateway_send_payment
make agg_mode_gateway_send_sp1_proof
  1. Aggregate the proof
make proof_aggregator_start_ethereum_package AGGREGATOR=sp1

Failure after retries

Do the same steps as before, but before step 4, go to the Docker container which maps the port 8545 (usually a el-X-geth-lighthouse) and shut it down.

Success after retries

The same as before, but when you see the first Started waiting until we can submit the aggregated proof. log, then start the container.

Type of change

Please delete options that are not relevant.

  • New feature
  • Bug fix
  • Optimization
  • Refactor

Checklist

  • “Hotfix” to testnet, everything else to staging
  • Linked to Github Issue
  • This change depends on code or research by an external entity
    • Acknowledgements were updated to give credit
  • Unit tests added
  • This change requires new documentation.
    • Documentation has been added/updated.
  • This change is an Optimization
    • Benchmarks added/run
  • Has a known issue
  • If your PR changes the Operator compatibility (Ex: Upgrade prover versions)
    • This PR adds compatibility for operator for both versions and do not change crates/docs/examples
    • This PR updates batcher and docs/examples to the newer version. This requires the operator are already updated to be compatible

@maximopalopoli maximopalopoli self-assigned this Dec 18, 2025
@maximopalopoli maximopalopoli marked this pull request as ready for review December 18, 2025 21:21
@JuArce
Copy link
Collaborator

JuArce commented Jan 5, 2026

I'd add a WARN log when there is an error. Ie "Error when ... Retrying in ..."

@MarcosNicolau MarcosNicolau force-pushed the feataggmode/retry-proof-bump-send-proof-verification-contract branch from 05f1613 to e828d99 Compare January 6, 2026 14:37
@JuArce JuArce added this pull request to the merge queue Jan 6, 2026
Merged via the queue into staging with commit dcfb204 Jan 6, 2026
3 checks passed
@JuArce JuArce deleted the feataggmode/retry-proof-bump-send-proof-verification-contract branch January 6, 2026 20:07
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.

4 participants