AW Green wrote:
Hi, David.
Sorry for responding offlist (I'm not a subscriber), but fwiw you don't
need to shutdown the i2pd daemon for log rotation. This is done instead
by sending the HUP signal to the i2pd process:
https://github.com/PurpleI2P/i2pd/issues/389
Conveniently, newsyslog(8) can send this signal for you when you provide
a pid_file argument in /etc/newsyslog.conf. For example, here is the
line I use in my install:
..
/var/log/i2pd/i2pd.log _i2pd:_i2pd 644 6 * $D13 Z /var/
lib/i2pd/i2pd.pid
..
Sending HUP to i2pd will also reload the tunnel config and change any
transient keys, so scheduled key rotation can be a useful side-effect of
this pid_file approach.
https://docs.i2pd.website/en/latest/user-guide/run/
Thank you for your work updating and improving this port!
Arthur
Hi Arthur,
Thank you for the detailed note, and for pointing this out.
You are right: using HUP through newsyslog(8)'s pid_file field is a
better approach than suggesting a graceful restart or shutdown for log
rotation. I had missed that i2pd handles log reopening this way, and the
additional tunnel config reload / transient key rotation side effect is
useful too.
The corrected README section would look like this:
```
Logging
=======
By default, the OpenBSD port of ${PKGSTEM} sends its log messages to
syslogd(8), which writes them to the /var/log/daemon file.
The default log level of ${PKGSTEM} ("warn") can be very verbose. You
may want to reduce this log verbosity by changing the "loglevel"
parameter in /etc/i2pd/i2pd.conf.
If you want log messages to be written to another file, e.g.
/var/log/i2pd/i2pd.log, you can change the "log" and "logfile"
parameters in /etc/i2pd/i2pd.conf. To have this log file rotated
automatically, add an entry to /etc/newsyslog.conf using the i2pd pid
file so newsyslog(8) can send SIGHUP to the daemon after rotation.
For example:
/var/log/i2pd/i2pd.log _i2pd:_i2pd 644 6 * $D13 Z
/var/i2pd/i2pd.pid
Sending SIGHUP is enough for log rotation, and also makes i2pd reload
its tunnel configuration and rotate transient keys.
```
Thanks again for the correction and for testing this in your install.
Best regards,
David.
