fix: set retry gas limit if outbound is successful#3820
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. 🗂️ Base branches to auto review (1)
Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
Documentation and Community
|
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## release/v29 #3820 +/- ##
============================================
Coverage 64.40% 64.40%
============================================
Files 469 469
Lines 32915 32915
============================================
Hits 21200 21200
Misses 10745 10745
Partials 970 970
🚀 New features to boost your workflow:
|
* set retry gas limit if evm outbound is successful * add constant for retry gas limit
* fix: set retry gas limit if outbound is successful (#3820) * set retry gas limit if evm outbound is successful * add constant for retry gas limit * changelog * improve comment
* fix: set retry gas limit if outbound is successful (#3820) * set retry gas limit if evm outbound is successful * add constant for retry gas limit * parse solana inbounds from inner instructions * changelog * cleanup * update comment * use instruction index for event index * extract parsing logic to inbound parser * use simple counter for event index
Description
Currently if outbound is successful retry gas limit will be 0, meaning there won't be retry for voting if 500k is not enough.
This introduces retry gas limit 1M for all observers.
How Has This Been Tested?