-
Notifications
You must be signed in to change notification settings - Fork 94
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
memory leak causes data corruption that check:construction not working properly #170
Comments
Thanks for posting this, @Yrp ! Would you mind including: |
All the logs are here: |
Would you mind uploading your configuration file too? Interesting memory blow up here:
|
It would also help if you could run |
the configuration file is here Have re-run rosetta-cli check:construction, tomorrow I'll see if the old data still exists! |
On an unrelated note, you may want to use the newly created Construction DSL instead of the json-based workflow specification for construction tests (much easier to read and maintain): https://github.com/coinbase/rosetta-cli/blob/master/examples/configuration/ethereum.ros This is how you link it to your config file: https://github.com/coinbase/rosetta-cli/blob/72051b6a51d638403835956090b2ec0268e2e129/examples/configuration/ethereum.json#L25 |
It's amazing. I will replace it with the new DSL sometime. |
Were you able to collect any additional data here @Yrp? we haven't been able to reproduce this locally 😢 |
Describe the bug
oom log
check:construction error log
The text was updated successfully, but these errors were encountered: