Systemd unit file reads settings from wrong path
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
pacemaker (Ubuntu) |
Fix Released
|
Undecided
|
Unassigned | ||
Xenial |
Fix Released
|
Medium
|
Heitor Alves de Siqueira |
Bug Description
[Impact]
Systemd Unit file doesn't read any settings by default
[Description]
The unit file shipped with the Xenial pacemaker package tries to read environment settings from /etc/sysconfig/ instead of /etc/default/. The result is that settings defined in /etc/default/
Since the /etc/default/
[Test Case]
1) Deploy a Xenial container:
$ lxc launch ubuntu:xenial pacemaker
2) Update container and install pacemaker:
root@
3) Change default pacemaker log location:
root@
4) Restart pacemaker service and verify that log file exists:
root@
root@
ls: cannot access '/tmp/pacemaker
After fixing the systemd unit, changes to /etc/default/
root@
-rw-rw---- 1 hacluster haclient 27335 Mar 7 20:46 /tmp/pacemaker.log
[Regression Potential]
The regression potential for this should be very low, since the configuration file is already being created by default and other systemd unit files are using the /etc/default config. In case the file doesn't exist or the user removed it, the "-" prefix will gracefully ignore the missing file according to the systemd.exec manual [0].
Nonetheless, the new package will be tested with autopkgtests and the fix will be validated in a reproduction environment.
[0] https:/
Changed in pacemaker (Ubuntu Xenial): | |
status: | New → Confirmed |
Changed in pacemaker (Ubuntu): | |
status: | Confirmed → Fix Released |
Changed in pacemaker (Ubuntu Xenial): | |
assignee: | nobody → Heitor R. Alves de Siqueira (halves) |
Changed in pacemaker (Ubuntu): | |
assignee: | Heitor R. Alves de Siqueira (halves) → nobody |
tags: | added: sts-sponsor-ddstreet |
description: | updated |
Changed in pacemaker (Ubuntu Xenial): | |
status: | Confirmed → In Progress |
importance: | Undecided → Medium |
tags: | removed: sts-sponsor sts-sponsor-ddstreet |
Patch v2:
Instead of adding the /etc/default location, change all /etc/sysconfig files to their /etc/default counterpart since the sysconfig folder does not exist in Debian based distros.