Opened 11 years ago

Closed 6 years ago

#40184 closed enhancement (wontfix)

logrotate @3.8.6_2 enhance startup item for hourly directive

Reported by: nonstop.server@… Owned by:
Priority: Normal Milestone:
Component: ports Version:
Keywords: haspatch Cc: mkae (Marko Käning)
Port: logrotate

Description

Enhance the example launchd plist to grant users the option to use the hourly directive.

hourly Log files are rotated every hour. Note that usually logrotate is
       configured to be run by cron daily. You have to change this con-
       figuration  and run logrotate hourly to be able to really rotate
       logs hourly.

The proposed patch has also two other changes, namely two added keys, LowPriorityIO and Nice which are used in the plist for newsyslogd.

LowPriorityIO <boolean>
     This optional key specifies whether the kernel should consider this daemon to be low
     priority when doing file system I/O.
Nice <integer>
     This optional key specifies what nice(3) value should be applied to the daemon.

Attachments (1)

org.macports.logrotate.plist.example.diff (745 bytes) - added by nonstop.server@… 11 years ago.
proposed patch

Download all attachments as: .zip

Change History (6)

Changed 11 years ago by nonstop.server@…

proposed patch

comment:1 Changed 11 years ago by pixilla (Bradley Giesbrecht)

Cc: pixilla@… removed
Owner: changed from macports-tickets@… to pixilla@…

This being an example file I am unsure how removing the StartCalendarInterval Hour directive is a better example. Do your StartCalendarInterval changes represent a more common use case?

The LowPriorityIO and Nice directives could be helpful to the user.

comment:2 in reply to:  1 Changed 11 years ago by nonstop.server@…

Replying to pixilla@…:

Your right it is not a better example by removing the Hour directive, but maybe it would be helpful to add a comment about the hourly option.
Something like:

"You have to remove the Hour directive to run logrotate hourly and be able to really rotate logs hourly."

comment:3 Changed 8 years ago by mkae (Marko Käning)

Cc: mkae added

comment:4 Changed 6 years ago by mf2k (Frank Schima)

Owner: pixilla deleted
Status: newassigned
Version: 2.2.0

See #57827

comment:5 Changed 6 years ago by ctreleaven (Craig Treleaven)

Resolution: wontfix
Status: assignedclosed

Users are free to edit the example launchd plist to any setting they want.

Note: See TracTickets for help on using tickets.