Skip to content
This repository has been archived by the owner on Apr 18, 2023. It is now read-only.

Issues with mcd-cdp ls with a large number of urns #9

Open
EdNoepel opened this issue Aug 23, 2019 · 3 comments
Open

Issues with mcd-cdp ls with a large number of urns #9

EdNoepel opened this issue Aug 23, 2019 · 3 comments

Comments

@EdNoepel
Copy link
Contributor

When ~1000 urns are open, mcd -C testnet cdp ls fails with an Argument list too long error. It's also worth mentioning it required 1000000 gas just to get that error. I'd hope that the required amount of gas wouldn't keep increasing as more CDPs are opened.

@gbalabasquer
Copy link
Contributor

I understand the Argument list too long issue but what does the cdp listing has to do with the gas amount? There is not even a transaction there.

@EdNoepel
Copy link
Contributor Author

Are you sure about that?

(_virtualenv) ed@duckmaker:~/Projects/auction-keeper/tests/manual$ mcd -C testnet cdp ls --gas=1
seth-rpc: {"id":1,"jsonrpc":"2.0","method":"eth_call","params":[{"data":"0xc455279100000000000000000000000000a329c0648769a73afac7f9381e0
8fb43dbea72","from":"0x00a329c0648769A73afAc7F9381E08FB43dBEA72","gas":"0x1","to":"0xcd9a70C13C88863ECE51B302a77D2EB98fBBBD65"},"latest"
]}
seth-rpc: error:   code       -32015
seth-rpc: error:   data       NotEnoughBaseGas { required: 22616, got: 1 }
seth-rpc: error:   message    Transaction execution error.

@gbalabasquer
Copy link
Contributor

It is using the support contract to get the CDPs list. I guess that last error has to do with you forcing 1 gas.
You could try manually with this:
seth call "$GET_CDPS" "getCdpsDesc(address,address)" "$CDP_MANAGER" "$YOUR_PROXY_ADDR" without specifying any amount of gas, and previously having executed unset ETH_GAS).

Anyway I don't think it will be normal 1 address has so many CDPs.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants