-
Notifications
You must be signed in to change notification settings - Fork 22
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
fix rejectPercent maybe other stuff? #1192
Conversation
Test Results247 tests 245 ✅ 1m 33s ⏱️ For more details on these failures, see this check. Results for commit 6f53028. ♻️ This comment has been updated with latest results. |
second patch changes "lag from 595483.20s to 6d21h" ... e.g:
becomes:
so it does not upset the columns that way either. |
I just noticed my fancy formatting has a problem... I made a conversion routine to make like 3h4m (3 hours and 4 minutes) but I didn't do the reverse conversion, which is needed. another commit coming. |
fixed now. |
@reidsunderland noticed a metric calculation that is off.
Now that people can see the metrics on real deployments, problems should be a lot more obvious. it might be worthwhile rooting around to see if other problems are detected.
We can maybe catch a couple with this PR.