Skip to content
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

Add CurveLpTokenUniv3Callee callee support #94

Closed
6 of 7 tasks
Tracked by #141 ...
valiafetisov opened this issue Feb 9, 2022 · 7 comments · Fixed by #156
Closed
6 of 7 tasks
Tracked by #141 ...

Add CurveLpTokenUniv3Callee callee support #94

valiafetisov opened this issue Feb 9, 2022 · 7 comments · Fixed by #156

Comments

@valiafetisov
Copy link
Contributor

valiafetisov commented Feb 9, 2022

Goal

Support new SteCRV collateral

Prerequisite

Context

New collateral types are coming and we need to add support for them. This time it can be a bit more difficult, since we don't have yet reference implementation of the auction-demo-keeper. There is a PR that implements auction-demo-keeper support for this callee: makerdao/auction-demo-keeper#60, main price fetching logic is within this block.

Tasks

  • Investigate callee implementation to document needed conversions
  • Implement callee functions, according to the Collateral onboarding guide
  • Test
@valiafetisov valiafetisov changed the title Add CurveLpTokenUniv3Callee callee support Add CurveLpTokenUniv3Callee callee support Feb 9, 2022
@LukSteib LukSteib mentioned this issue Mar 14, 2022
23 tasks
@LukSteib
Copy link
Contributor

CurveLpTokenUniv3Callee callee PR is merged

It is deployed: https://etherscan.io/address/0x71f2198849f3b1372ea90c079bd634928583f2d2

Related addresses (token address, clip address, etc) become part of the chainlog

They are now: Searching chainlog for CRVV1ETHSTETH

@valiafetisov
Copy link
Contributor Author

What about testing strategy for the callee? Would there be at least one auction on the mainnet?

@LukSteib
Copy link
Contributor

What about testing strategy for the callee? Would there be at least one auction on the mainnet?

Current status:

@LukSteib
Copy link
Contributor

Update:

  • it is now possible to create vaults with CRVV1ETHSTETH as collateral
  • 5 vaults exist already
    • I assume chaoslabs should be able to simulate a liquidation for this collateral type based on their existing approach

@valiafetisov
Copy link
Contributor Author

valiafetisov commented Mar 22, 2022

I assume chaoslabs should be able to simulate a liquidation for this collateral type based on their existing approach

Not really. Update from the chaoslabs:

We located 6 CRVV1ETHSTETH-A vaults but they all appear in inactive state (no debt) and hence are not elegible for liquidations (even when simulating price drops).
We will work on implementing the support for creating active vaults for a given collateral type but it is a larger engineering lift and will take more time on our side.
If you have active vault ids that you can share with us we will be able to create a simulation quickly (~couple of hours).

@LukSteib
Copy link
Contributor

Not good news.
I don't get the differentiation between active and inactive vaults tbh. The vault analytics (see below) show that the vaults associated to CRVV1ETHSTETH-A do have debt. However they seem to be different due to a) the vault ID doesn't follow the usual pattern b) there seems to be no Dai borrowed for these vaults c) no user wallet is associated with these vaults

https://maker.blockanalitica.com/vaults/CRVV1ETHSTETH-A/

image

@valiafetisov
Copy link
Contributor Author

valiafetisov commented Mar 22, 2022

no Dai borrowed for these vaults

Thanks for the link! Although I am not sure I understand how there can be a debt if no DAI was withdrawn? But yeah, I think this what they're referencing to: if no DAI was withdrawn, changing oracle price wouldn't make them "kickable"

@LukSteib LukSteib mentioned this issue Mar 29, 2022
16 tasks
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 a pull request may close this issue.

2 participants