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
Log segments that are being archived #7723
Conversation
Can you add a unit/integration test? Maybe to CliMultiTest. |
Also could add an assertion for the output in ArchiveCronTest, not sure how important that is. |
a subtle but big improvement to humans consuming the log! Here is suggestion to improve output when there are 200 segments:
to
|
+1 testing for the |
d5f3f5e
to
c7f5efa
Compare
I rebased on master and pushed tests for CliMulti + the cron archive output. FYI the cron archive output test takes ~ a minute (I created 1 website + 1 segment, I didn't add any visits). |
+1 - it's a real good feeling to see new tests for |
Looks good to me, I think the tests can be sped up later so if the build passes, I will merge. |
Btw the test could be made faster by appending to the command |
Log segments that are being archived
Fixes #7536
Example: