-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Added compress option to dnsmasq log files #4439
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 compress option to dnsmasq log files #4439
Conversation
|
@blueorangutan package |
|
@Spaceman1984 a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. |
|
Packaging result: ✔centos7 ✔centos8 ✔debian. JID-2325 |
|
@blueorangutan test |
|
@Spaceman1984 a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests |
|
Trillian test result (tid-3119)
|
|
Simple enough change, let's merge for master @DaanHoogland ? |
|
@PaulAngus merging this as it is small and not intrusive. cc @GabrielBrascher @weizhouapache. |
Description
This PR adds compression to dnsmasq log files
Fixes: #3756
Types of changes
Screenshots (if appropriate):
How Has This Been Tested?
Log into virtual router
Check /var/log/ directory, observe dnsmasq.log
Change system time 1 day ahead and reboot VR
Check /var/log/ directory, observe dnsmasq.log, dnsmasq.log1
Change system time 1 day ahead and reboot VR
There should be 3 log files now and 1 should be compressed
Check /var/log/ directory, observe dnsmasq.log, dnsmasq.log1, dnsmasq.log.2.gz