Commit report for v3.1.x branch - logrotate

Matthew Newton mcn4 at
Fri Sep 18 00:29:34 CEST 2015

On Fri, Sep 18, 2015 at 12:00:02AM +0200, The git bot wrote:
> ======
> Revert "logrotate: send a HUP after rotation"
> This is wrong, copyrotate is the correct command to use

That method can lose log data, and really a hack for programs that
can't close and reopen their own log files.

The right way is to move the logfile, then signal the daemon to
reopen its logfile, which is what the original version did.

But the question really is how to signal. "killall -HUP radiusd"
is verging on wrong (a variant of kill -HUP `cat $PIDFILE` would
be more correct). I suspect /etc/init.d/freeradius reload is
trying to do the latter, but is a) deprecated (it should be
something like service freeradius reload) and b) possibly won't
play well with systemd, however much any of us may hate it.

The example in logrotate.conf(5) is

    kill -HUP `cat /var/run/`

which would be best IMO - it doesn't play with init scripts or
touch anything systemd related, is not as drastic as killall, and
has no race.



Matthew Newton, Ph.D. <mcn4 at>

Systems Specialist, Infrastructure Services,
I.T. Services, University of Leicester, Leicester LE1 7RH, United Kingdom

For IT help contact helpdesk extn. 2253, <ithelp at>

More information about the Freeradius-Devel mailing list