FS#62310 - [chrony] Upstream systemd change to PIDFile location in service unit

Attached to Project: Community Packages
Opened by Troy Engel (TE) - Thursday, 11 April 2019, 13:48 GMT
Last edited by Levente Polyak (anthraxx) - Sunday, 30 August 2020, 11:36 GMT
Task Type Bug Report
Category Packages
Status Closed
Assigned To Levente Polyak (anthraxx)
Architecture All
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

I have filed a bug on the chrony-dev@ mailing list, this is a copy/paste of the bug report. The Arch packaging could sed the upstream unit file as one possible solution for now.

$ pacman -Q systemd chrony
systemd 241.93-1
chrony 3.4-1

=====

The latest version of system 241.93 has implemented a change to PID files being stored in subdirectories below /var/run - from my dmesg:

[173100.352314] systemd[1]: /usr/lib/systemd/system/chronyd.service:9: PIDFile= references path below legacy directory /var/run/, updating /var/run/chrony/chronyd.pid → /run/chrony/chronyd.pid; please update the unit file accordingly.

The upstream commit/merge: https://github.com/systemd/systemd/commit/983616735e0796756d82f337b6d57412ef9df04c#diff-cc81d855afbe30015a0e769bbf67567f

=====

It appears their mailing list archives update daily, I can provide a link to my thread once available. (https://chrony.tuxfamily.org/lists.html)
This task depends upon

Closed by  Levente Polyak (anthraxx)
Sunday, 30 August 2020, 11:36 GMT
Reason for closing:  Implemented
Additional comments about closing:  chrony-3.5.1-1
Comment by Troy Engel (TE) - Friday, 12 April 2019, 12:56 GMT
Mailing list archive: https://listengine.tuxfamily.org/chrony.tuxfamily.org/chrony-dev/2019/04/msg00011.html

The code has an internal coded path to the PID file, so just changing the unit is not enough. Upstream indicates they want this left as-is because it allows it to run as non-root (I think for non-systemd use?), so we have to add `--with-pidfile=/var/run/chronyd.pid` to the ./configure as well as updating the systemd unit file to match.

Based on the response, it might be something we have to do here downstream in packaging, as they intentionally want the default as what it is (/var/run/chrony/chronyd.pid) by design in ./configure defaults. $0.02

Loading...