-
Notifications
You must be signed in to change notification settings - Fork 579
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
[dev.icinga.com #9249] logrotate fails since the "su" directive was removed #2998
Comments
Updated by mwaldmueller on 2015-05-12 06:47:19 00:00 It works if you add a "su" line before the "create" line in /etc/logrotate.d/icinga2:
|
Updated by purplecarrot on 2015-05-12 07:27:05 00:00
Yeah, adding back these two lines work. Here's a patch. |
Updated by mfriedrich on 2015-05-12 07:57:41 00:00
That won't work on Ubuntu 12.04 LTS - see #8190. The clean fix is to determine the ubuntu version in cmake and install a separate logrotate file omitting the "su" lines. Can you do that instead please? |
Updated by mfriedrich on 2015-05-12 07:57:55 00:00
|
Updated by gbeutner on 2016-03-29 12:59:59 00:00
|
Updated by purplecarrot on 2016-03-29 13:00:06 00:00
Applied in changeset 4970c45. |
Updated by gbeutner on 2016-04-20 08:15:49 00:00
|
Updated by mfriedrich on 2016-06-06 12:46:58 00:00
|
This issue has been migrated from Redmine: https://dev.icinga.com/issues/9249
Created by purplecarrot on 2015-05-11 14:50:34 00:00
Assignee: purplecarrot
Status: Resolved (closed on 2016-03-29 13:00:06 00:00)
Target Version: 2.4.5
Last Update: 2016-04-20 08:15:49 00:00 (in Redmine)
The following commit messes with logrotate because of the su removal: https://dev.icinga.org/projects/i2/repository/revisions/a28369593a1bc74ae20db55a643d2bac4cd8a0f2
Since then, logrotate throws errors
Attachments
Changesets
2016-03-29 12:59:06 00:00 by purplecarrot 4970c45
2016-04-20 08:07:24 00:00 by purplecarrot 45786e2
Relations:
The text was updated successfully, but these errors were encountered: