Pino Toscano
2017-Mar-10 14:09 UTC
[Libguestfs] [PATCH] appliance: run systemd-tmpfiles also for /var/run
Commit a6330e9d3af0f5286f1d53d909fd868387b67f69 enabled /run for systemd-tmpfiles: while this works fine in most of the cases, there are few tmpfiles configurations that still references /var/run instead of /run. As result, include also /var/run in the systemd-tmpfiles execution. --- appliance/init | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/appliance/init b/appliance/init index b951857..968429c 100755 --- a/appliance/init +++ b/appliance/init @@ -90,7 +90,7 @@ machine_id=$(dd if=/dev/urandom bs=16 count=1 status=none | od -x -A n) echo "${machine_id// /}" > /etc/machine-id # Set up tmpfiles (must run after kmod.conf is created above). -systemd-tmpfiles --prefix=/dev --prefix=/run --create --boot +systemd-tmpfiles --prefix=/dev --prefix=/run --prefix=/var/run --create --boot # Find udevd and run it directly. for f in /lib/systemd/systemd-udevd /usr/lib/systemd/systemd-udevd \ -- 2.9.3
Richard W.M. Jones
2017-Mar-10 21:44 UTC
Re: [Libguestfs] [PATCH] appliance: run systemd-tmpfiles also for /var/run
On Fri, Mar 10, 2017 at 03:09:46PM +0100, Pino Toscano wrote:> Commit a6330e9d3af0f5286f1d53d909fd868387b67f69 enabled /run for > systemd-tmpfiles: while this works fine in most of the cases, there are > few tmpfiles configurations that still references /var/run instead of > /run. As result, include also /var/run in the systemd-tmpfiles > execution. > --- > appliance/init | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/appliance/init b/appliance/init > index b951857..968429c 100755 > --- a/appliance/init > +++ b/appliance/init > @@ -90,7 +90,7 @@ machine_id=$(dd if=/dev/urandom bs=16 count=1 status=none | od -x -A n) > echo "${machine_id// /}" > /etc/machine-id > > # Set up tmpfiles (must run after kmod.conf is created above). > -systemd-tmpfiles --prefix=/dev --prefix=/run --create --boot > +systemd-tmpfiles --prefix=/dev --prefix=/run --prefix=/var/run --create --boot > > # Find udevd and run it directly. > for f in /lib/systemd/systemd-udevd /usr/lib/systemd/systemd-udevd \ > -- > 2.9.3Looks straightforward, ACK. Rich. -- Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones Read my programming and virtualization blog: http://rwmj.wordpress.com Fedora Windows cross-compiler. Compile Windows programs, test, and build Windows installers. Over 100 libraries supported. http://fedoraproject.org/wiki/MinGW
Possibly Parallel Threads
- [PATCH 5/5] appliance: run systemd-tmpfiles also for /run
- [PATCH 4/4] appliance: run systemd-tmpfiles also for /run
- [PATCH v2 0/4] Improve LVM handling in the appliance
- [PATCH 0/5] Improve LVM handling in the appliance
- [PATCH 0/2] appliance: When running systemd-tmpfiles, pass --boot