[PATCH v5 1/3] libvirt.spec: relocate pre script of daemon-driver-qemu

Jim Fehlig jfehlig at suse.com
Thu Dec 16 23:07:09 UTC 2021


On 12/16/21 08:14, Olaf Hering wrote:
> Reduce the delta in an upcoming change.
> No change in behavior intended.
> 
> Signed-off-by: Olaf Hering <olaf at aepfle.de>
> ---
>   libvirt.spec.in | 32 +++++++++++++++-----------------
>   1 file changed, 15 insertions(+), 17 deletions(-)

IMO, putting the RPM scriptlet code within one conditional is a good change 
regardless of the outcome of this series.

> diff --git a/libvirt.spec.in b/libvirt.spec.in
> index f6c705564a..4675e67d15 100644
> --- a/libvirt.spec.in
> +++ b/libvirt.spec.in
> @@ -1466,6 +1466,21 @@ fi
>   
>   
>   %if %{with_qemu}
> +%pre daemon-driver-qemu
> +# We want soft static allocation of well-known ids, as disk images
> +# are commonly shared across NFS mounts by id rather than name; see
> +# https://fedoraproject.org/wiki/Packaging:UsersAndGroups
> +getent group kvm >/dev/null || groupadd -f -g 36 -r kvm
> +getent group qemu >/dev/null || groupadd -f -g 107 -r qemu
> +if ! getent passwd qemu >/dev/null; then
> +  if ! getent passwd 107 >/dev/null; then
> +    useradd -r -u 107 -g qemu -G kvm -d / -s /sbin/nologin -c "qemu user" qemu
> +  else
> +    useradd -r -g qemu -G kvm -d / -s /sbin/nologin -c "qemu user" qemu
> +  fi
> +fi
> +exit 0
> +
>   %post daemon-driver-qemu
>       %if %{with_modular_daemons}
>   %libvirt_daemon_systemd_post virtqemud
> @@ -1587,23 +1602,6 @@ done
>   %libvirt_daemon_perform_restart libvirtd
>   %libvirt_daemon_perform_restart virtnwfilterd
>   
> -%if %{with_qemu}
> -%pre daemon-driver-qemu
> -# We want soft static allocation of well-known ids, as disk images
> -# are commonly shared across NFS mounts by id rather than name; see
> -# https://fedoraproject.org/wiki/Packaging:UsersAndGroups
> -getent group kvm >/dev/null || groupadd -f -g 36 -r kvm
> -getent group qemu >/dev/null || groupadd -f -g 107 -r qemu
> -if ! getent passwd qemu >/dev/null; then
> -  if ! getent passwd 107 >/dev/null; then
> -    useradd -r -u 107 -g qemu -G kvm -d / -s /sbin/nologin -c "qemu user" qemu
> -  else
> -    useradd -r -g qemu -G kvm -d / -s /sbin/nologin -c "qemu user" qemu
> -  fi
> -fi
> -exit 0
> -%endif
> -

Same could be said for the below lxc %pre scriptlet

>   %if %{with_lxc}
>   %pre login-shell
>   getent group virtlogin >/dev/null || groupadd -r virtlogin

Regards,
Jim




More information about the libvir-list mailing list