op-node: reset back to optimal receipt fetching methods#5231
op-node: reset back to optimal receipt fetching methods#5231mergify[bot] merged 2 commits intodevelopfrom
Conversation
…ack permanently to sub-optimal RPC
|
✅ Deploy Preview for opstack-docs canceled.
|
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## develop #5231 +/- ##
===========================================
- Coverage 40.07% 36.26% -3.81%
===========================================
Files 372 217 -155
Lines 23605 18975 -4630
Branches 832 0 -832
===========================================
- Hits 9459 6882 -2577
+ Misses 13438 11423 -2015
+ Partials 708 670 -38
Flags with carried forward coverage won't be shown. Click here to find out more.
|
trianglesphere
left a comment
There was a problem hiding this comment.
Fallback seems to work. Do we know if we've hit this before?
Also, on the testing side, how does the test verify that we undo the fallback?
|
This PR has been added to the merge queue, and will be merged soon. |
|
This PR is next in line to be merged, and will be merged as soon as checks pass. |
Description
Reset it back after a while to not fallback permanently to sub-optimal RPC methods for receipt fetching.
Tests
Updated test to see if method preference can be sticky and reset back if configured to do so.
TODOs
Fix CLI-3697