-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
Issue #4446: Fixing logrotate config file. #4447
Conversation
This issue switches from "cp" to "install" for the logrotate file, with the option to specify the mode. Also uses the name "influxdb" instead of "influxd".
If we are going to do this, we need to tweak the post-install script to remove any existing file named Running |
Sample session on Ubuntu 14.04 system:
|
I think this is good to merge, but I need to add more changes to this to remove any pre-existing rotate file. |
@linsomniac -- have you signed the CLA? |
Shouldn't be a problem, I need to have someone review it first. Probably can sign it tomorrow. |
Signed both by me and the company CTO. |
Thanks @linsomniac |
Issue #4446: Fixing logrotate config file.
the flags have to match the flags with the exception of beign lower case and all `_` be changed to `-` in the flag. This is a result of using the cobra flag to env var mapping with the `-` replaced to `_`. issue: #4447
This issue switches from "cp" to "install" for the logrotate file,
with the option to specify the mode. Also uses the name "influxdb"
instead of "influxd".