-
Notifications
You must be signed in to change notification settings - Fork 1
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
MU07 - Chainlink Relayers #206
Conversation
Co-authored-by: chapati <philip.paetz@me.com>
* feat: mento gov utils scripts * fix: interfaces uses correct pragma * feat: pass and queue proposals * feat: remove --no-cache from script calls * fix: typo on a revert message * feat: improve logging * fix: remove tree that is pushed * fix: remove merge artifacts * feat: update readme
* chore: reproduce deployment broadcast files from on-chain data * Update bin/reproduce-mainnet-gov.ts Co-authored-by: Ryan Noble <ryanjnoble@gmail.com> * Apply suggestions from code review Co-authored-by: baroooo <baranseltekin@gmail.com> * feat: regenerate json files after Baran's suggestions --------- Co-authored-by: Ryan Noble <ryanjnoble@gmail.com> Co-authored-by: baroooo <baranseltekin@gmail.com>
I tried running the proposal on Baklava but it seams that the cLabs contracts team have also abandoned Baklava and SortedOracles is not up to date with the equivalent token implementation :( |
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.
one last typo, then good to go from my side
|
||
function getProxyAdminOwner() internal view returns (address) { | ||
if (ChainLib.isCelo()) { | ||
return 0x655133d8E90F8190ed5c1F0f3710F602800C0150; // Mento Labs multisig |
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.
address looks correct, but maybe get it from contracts.dependency("MentoLabsMultisig")
?
Merging currently blocked until this is merged celo-org/governance#472 |
MU07 Overview
relayed:CELOPHP
andrelayed:PHPUSD
.relayed:CELOPHP
as the "equivalent token" for the PSO token address. Meaning that when paying with gas, the blockchain node will use therelayed:CELOPHP
rate feed when gas is payed in the PSO token.Testing / development helpers
In order to have as close a structure as possible on testnets, this PR also deploys a few
MockChainlinkAggregator
contracts:PHPUSD
to AlfjoresPHPUSD
to BaklavaCELOUSD
to BaklavaUsing these on Alfajores we have:
relayed:CELOPHP
built from chainlink's own testnet aggregator ofCELOUSD
and our mock aggregator ofPHPUSD
relayed:PHPUSD
built from our mock aggregator ofPHPUSD
And on Baklava we have everything based only on our mock aggregators.
Scripts to interact with relayers:
RelayerStatus
Get a status output for all defined relayers:
RelayerReport
Trigger report on all relayers.
UpdateMockChainlinkAggregators
Update the mock aggregators on testnets with values from the mainnet aggregators that they mirror.
Other Changes
forge_script
helper functions because it got out of hand and it was only used in a couple of placesverify:celoscan
command to check ABIs on celoscan first, and skip verification.Related Issues