-
Notifications
You must be signed in to change notification settings - Fork 432
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update IGP oracles #3244
Update IGP oracles #3244
Conversation
|
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #3244 +/- ##
=======================================
Coverage 67.65% 67.65%
=======================================
Files 99 99
Lines 1014 1014
Branches 106 106
=======================================
Hits 686 686
Misses 284 284
Partials 44 44
|
typescript/infra/scripts/gas-oracle/compare-token-exchange-rates.ts
Outdated
Show resolved
Hide resolved
typescript/infra/scripts/gas-oracle/compare-token-exchange-rates.ts
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
### Description Simplify IGP oracle config and deployment by removing multiple storage gas oracles. Update infra to use new IGP configs. Improve IGP deployer logs. Use updated exchange rates. Update 20% margin to apply only when rates are unequal ### Drive-by changes - Rework ownable logic slightly - Add mantapacific to safe addresses ### Backward compatibility - Oracle comparison is now done in igp/core fork tests ### Testing - CI fork tests - manual mainnet deploy
Description
Simplify IGP oracle config and deployment by removing multiple storage gas oracles. Update infra to use new IGP configs. Improve IGP deployer logs. Use updated exchange rates. Update 20% margin to apply only when rates are unequal
Drive-by changes
Related issues
Backward compatibility
Testing