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
Added balance reporter #1154
Added balance reporter #1154
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1154 +/- ##
==========================================
- Coverage 56.16% 55.96% -0.20%
==========================================
Files 484 485 +1
Lines 29820 29929 +109
==========================================
+ Hits 16747 16751 +4
- Misses 10787 10893 +106
+ Partials 2286 2285 -1
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
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.
Any chance we can also output how long these reporters take? Then we can decide if we want them always on
Done. I edited the description to reflect this. |
bors merge |
Added a reporter, that outputs all the
FlowVault
s ad their locations in a big json.I use gojq to get data from the json.
examples:
cat balance_report_1628871716.json | gojq '[.Data.[].Balance] | add'
cat balance_report_1628871716.json | gojq '.Data | group_by(.LastComposite) | map({LastComposite: .[0].LastComposite, Balance: map(.Balance) | add }) | sort_by(-.Balance) '
This PR also adds elapsed times of migrations and reporters to the logs. The output will look like this: