]> xenbits.xensource.com Git - libvirt.git/commit
libvirt-guests: Fix dependency ordering in service file
authorMartin Kletzander <mkletzan@redhat.com>
Tue, 30 Aug 2022 06:29:53 +0000 (08:29 +0200)
committerMartin Kletzander <mkletzan@redhat.com>
Mon, 26 Sep 2022 11:04:48 +0000 (13:04 +0200)
commit59d30adacd1d5a4de8c25200ebfc666bd180fe1e
treedea48935315a9f340c8967be79fab533b9681c43
parente9fd6de8fc4e7c8f616bb6e11051c4929075dcbf
libvirt-guests: Fix dependency ordering in service file

After some debugging and discussion with systemd team it turns out we
are misusing the ordering in libvirt-guests.service.  That happened
because we want to support both monolithic and modular daemon setups and
on top of that we also want to support socket activation and services
without socket activation.  Unfortunately this is impossible to express
in the unit file because of how transactions are handled in systemd when
dependencies are resolved and multiple actions (jobs) are queued.  For
explanation from Michal Sekletar see comment #7 in the BZ this patch is
fixing:

https://bugzilla.redhat.com/show_bug.cgi?id=1964855#c7

In order to support all the scenarios this patch also amends the
manpages so that users that are changing the default can also read how
to correct the dependency ordering in libvirt-guests unit file.

Ideally we would also keep the existing configuration during upgrade,
but due to our huge support matrix this seems hardly feasible as it
could introduce even more problems.

Signed-off-by: Martin Kletzander <mkletzan@redhat.com>
Reviewed-by: Michal Privoznik <mprivozn@redhat.com>
docs/manpages/libvirtd.rst
docs/manpages/virtlxcd.rst
docs/manpages/virtqemud.rst
docs/manpages/virtvboxd.rst
docs/manpages/virtvzd.rst
docs/manpages/virtxend.rst
tools/libvirt-guests.service.in