Message ID | 20200824165948.476427-3-marcelo.cerri@canonical.com |
---|---|
State | New |
Headers | show |
Series | LP:#1891224 - [Hyper-V] VSS and File Copy daemons intermittently fails to start | expand |
diff --git a/debian/linux-cloud-tools-common.hv-fcopy-daemon.service b/debian/linux-cloud-tools-common.hv-fcopy-daemon.service index 43a3ecf9ed6e..601376e3f67f 100644 --- a/debian/linux-cloud-tools-common.hv-fcopy-daemon.service +++ b/debian/linux-cloud-tools-common.hv-fcopy-daemon.service @@ -4,6 +4,8 @@ [Unit] Description=Hyper-V File Copy Protocol Daemon ConditionVirtualization=microsoft +ConditionPathExists=/dev/vmbus/hv_fcopy +BindsTo=sys-devices-virtual-misc-vmbus\x21hv_fcopy.device [Service] ExecStart=/usr/sbin/hv_fcopy_daemon -n diff --git a/debian/linux-cloud-tools-common.hv-fcopy-daemon.udev b/debian/linux-cloud-tools-common.hv-fcopy-daemon.udev new file mode 100644 index 000000000000..8f85edb87aeb --- /dev/null +++ b/debian/linux-cloud-tools-common.hv-fcopy-daemon.udev @@ -0,0 +1 @@ +SUBSYSTEM=="misc", KERNEL=="vmbus/hv_fcopy", TAG+="systemd", ENV{SYSTEMD_WANTS}+="hv-fcopy-daemon.service" diff --git a/debian/rules.d/3-binary-indep.mk b/debian/rules.d/3-binary-indep.mk index f4dcba21d620..511a5eb39211 100644 --- a/debian/rules.d/3-binary-indep.mk +++ b/debian/rules.d/3-binary-indep.mk @@ -202,6 +202,7 @@ ifeq ($(do_tools_hyperv),true) dh_installinit -p$(cloudpkg) -n --name hv-fcopy-daemon dh_installudev -p$(cloudpkg) -n --name hv-kvp-daemon dh_installudev -p$(cloudpkg) -n --name hv-vss-daemon + dh_installudev -p$(cloudpkg) -n --name hv-fcopy-daemon dh_systemd_enable -p$(cloudpkg) dh_installinit -p$(cloudpkg) -o --name hv-kvp-daemon dh_installinit -p$(cloudpkg) -o --name hv-vss-daemon
BugLink: https://bugs.launchpad.net/bugs/1891224 Update the systemd service for the fcopy daemon and add a new udev rule to ensure the service will not fail in case fcopy is not enabled for the guest. The udev rules is responsible for starting the service only when the fcopy device is available. In the unit file, the service is now bound to the device unit, so the service can be stopped if the device is removed. And the service is also conditioned to the existence of the device, this way we avoid starting and stopping the service during boot if the device is not available. Signed-off-by: Marcelo Henrique Cerri <marcelo.cerri@canonical.com> --- debian/linux-cloud-tools-common.hv-fcopy-daemon.service | 2 ++ debian/linux-cloud-tools-common.hv-fcopy-daemon.udev | 1 + debian/rules.d/3-binary-indep.mk | 1 + 3 files changed, 4 insertions(+) create mode 100644 debian/linux-cloud-tools-common.hv-fcopy-daemon.udev