Skip to content
This repository has been archived by the owner on May 11, 2024. It is now read-only.

fix(prover): fix maxRetry configuration when submitting proofs #364

Merged
merged 1 commit into from
Aug 16, 2023

Conversation

davidtaikocha
Copy link
Member

@davidtaikocha davidtaikocha commented Aug 15, 2023

We should not return an error when hitting the maxRetry--proofSubmissionMaxRetry) config, otherwise, it will still be retried by the main proof submission loop, and --proofSubmissionMaxRetry flag wont work:
image

@codecov
Copy link

codecov bot commented Aug 15, 2023

Codecov Report

Merging #364 (fae3089) into main (24b9309) will increase coverage by 0.40%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #364      +/-   ##
==========================================
+ Coverage   52.07%   52.48%   +0.40%     
==========================================
  Files          35       35              
  Lines        4499     4502       +3     
==========================================
+ Hits         2343     2363      +20     
+ Misses       1893     1876      -17     
  Partials      263      263              
Files Changed Coverage Δ
prover/proof_submitter/util.go 48.27% <0.00%> (-1.73%) ⬇️

... and 2 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@davidtaikocha davidtaikocha merged commit b6cd4db into main Aug 16, 2023
5 checks passed
@davidtaikocha davidtaikocha deleted the fix-max-retry branch August 16, 2023 02:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants