Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Not all log files in --logdir are compressed #7353

Closed
ckamm opened this issue Jul 30, 2019 · 1 comment
Closed

Not all log files in --logdir are compressed #7353

ckamm opened this issue Jul 30, 2019 · 1 comment
Assignees
@ckamm
Labels
p4-low Low priority ReadyToTest QA, please validate the fix/enhancement type:bug
Milestone
2.6.0

Comments

@ckamm
Copy link
Contributor

ckamm commented Jul 30, 2019

In particular the log file that was active when the client was using when it was quit or crashed is never compressed.

  1. Start client with --logdir mypath
  2. Restart the client. Then quit it.
  3. There should be only one uncompressed logfile in mypath - the most recent one.
@ckamm ckamm added type:bug p4-low Low priority labels Jul 30, 2019
@ckamm ckamm added this to the 2.6.0 milestone Jul 30, 2019
@ckamm ckamm self-assigned this Jul 30, 2019
ckamm added a commit that referenced this issue Jul 30, 2019
@ckamm
LogDir: Compress last logfile on restart
dea09b5 
For #7353
@ckamm ckamm mentioned this issue Jul 30, 2019
Merged
ckamm added a commit that referenced this issue Aug 1, 2019
@ckamm
LogDir: Compress last logfile on restart
907945d 
For #7353
@ckamm ckamm added ReadyToTest QA, please validate the fix/enhancement and removed PR available labels Aug 1, 2019
@HanaGemela
Copy link
Contributor

Works as expected on macOS 10.14.6, client 2.6.0beta1 (build 12241)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ckamm ckamm

Labels
p4-low Low priority ReadyToTest QA, please validate the fix/enhancement type:bug
Projects
None yet
Milestone
2.6.0
Development

No branches or pull requests
2 participants
@ckamm @HanaGemela