aligned_layer icon indicating copy to clipboard operation
aligned_layer copied to clipboard

feat(operator): `SendSignedTaskResponse` Retries

Open PatStiles opened this issue 1 year ago • 0 comments

Add Retry Logic to SendSignedTaskResponse

Description

#closes 1415

Note: I tested this running a local devnet and had no issues. However, I encountered issues stopping the aggregator while writing a unit test and found the rpc client function panic'd when the connection was cut. I discussed this with @Oppen but will take time to investigate it further.

How to test

In devnet

  1. make anvil_start_with_block_time
  2. make aggregator_start
  3. make operator_full_registration CONFIG_FILE=config-files/config-operator-1.yaml
  4. make operator_start CONFIG_FILE=config-files/config-operator-1.yaml
  5. make batcher_start_local
  6. make telemetry_full_start

Once you have the system running, send proofs

  1. make batcher_send_burst_groth16
  2. Go to jaeger http://localhost:16686/search
  3. You should see the traces correctly created and task verified image

Type of change

  • [x] New feature

Checklist

  • [https://github.com/yetanotherco/aligned_layer/issues/1415] Linked to Github Issue
  • [ ] This change depends on code or research by an external entity
    • [ ] Acknowledgements were updated to give credit
  • [x] 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
    • Link to the open issue addressing it
  • [ ] If your PR changes the Operator compatibility (Ex: Upgrade prover versions)
    • [ ] This PR adds compatibility for operator for both versions and do not change batcher/docs/examples
    • [ ] This PR updates batcher and docs/examples to the newer version. This requires the operator are already updated to be compatible

PatStiles avatar Nov 16 '24 05:11 PatStiles