Skip to content

V2 Deploy Part 1#171

Merged
Joeysantoro merged 6 commits intofeat/v2from
feat/v2-Deploy
Sep 13, 2021
Merged

V2 Deploy Part 1#171
Joeysantoro merged 6 commits intofeat/v2from
feat/v2-Deploy

Conversation

@Joeysantoro
Copy link
Contributor

@Joeysantoro Joeysantoro commented Sep 13, 2021

This PR deploys everything in V2 except the FEI PCV wrappers which will come in a subsequent PR

Also fixes a bug in the UniswapPCVDeposit resistant balance calculation where the result is not scaled by the proportion held by the contract

  • seed TRIBE to LBP stabilizer
  • check that invertOracle direction is correct for TribeReserveStabilizer

"proxyAdminAddress" : { "artifact" : "ProxyAdmin", "address": "0xf8c2b645988b7658E7748BA637fE25bdD46A704A"},
"raiAddress": {"artifact" : "IERC20", "address" : "0x03ab458634910AaD20eF5f1C8ee96F1D6ac54919"},
"raiBondingCurveAddress": { "artifact": "BondingCurve", "address": "0x25d60212D47Dd8F6Ff0469367E4c6C98Cd3411A5"},
"rariPool19DpiAddress": { "artifact" : "CErc20Delegator", "address": "0xf06f65a6b7d2c401fcb8b3273d036d21fe2a5963"},
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added these copies of the indexCoop and poolParty PCV deposits which adhere to naming conventions

})
});

describe('Reweights', async () => {
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Deleted reweights from e2e which are not a part of v2

"voterAddress" : "0xB8f482539F2d3Ae2C9ea6076894df36D1f632775"
},
"upgrade" : {
"fip_10b": {
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fip 10b is the curve deposit which is currently separate from the main phase 1 deploy


logging ? console.log('Minting FEI to accounts[0]') : undefined;
await fei.mint(accounts[0], new BN('10000000000000000000000000000000000'));
await fei.mint(accounts[0], new BN('10000000000000000000000000'));
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this was messing up the CR oracle calculation so I lowered amount to 1M FEI

@xklob
Copy link
Contributor

xklob commented Sep 13, 2021

@Joeysantoro CI is showing two failed tests - did you have a look at them?

Copy link
Contributor

@xklob xklob left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

First pass appears fine, though see the couple of comments I made.
I'm going to do a second pass and poke at it with VsCode some.

@Joeysantoro
Copy link
Contributor Author

@Joeysantoro CI is showing two failed tests - did you have a look at them?

One was just an env failure, fixed the other

@Joeysantoro Joeysantoro merged commit a10b3f7 into feat/v2 Sep 13, 2021
@xklob xklob deleted the feat/v2-Deploy branch September 19, 2021 18:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments