[PATCH][XBC] UBUNTU: [Debian] hyper-v -- Ensure that hv-kvp-daemon.service starts before walinuxagent.service

classic Classic list List threaded Threaded
5 messages Options
Reply | Threaded
Open this post in threaded view
|

[PATCH][XBC] UBUNTU: [Debian] hyper-v -- Ensure that hv-kvp-daemon.service starts before walinuxagent.service

Eric Desrochers
From: Paul Meyer <[hidden email]>

BugLink: https://bugs.launchpad.net/bugs/1739107

The hv-kvp-daemon service starts after walinuxagent. The default dependencies make hv-kvp-daemon
wait until the whole system is up before it can start, which is not necessary and blocks Azure
telemetry use case. This commit changes the hv-kvp-daemon.service unit file to start it up as
early as possible so that the channel for telemetry can be use.

Signed-off-by: Eric Desrochers <[hidden email]>
---
 debian/linux-cloud-tools-common.hv-kvp-daemon.service | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/debian/linux-cloud-tools-common.hv-kvp-daemon.service b/debian/linux-cloud-tools-common.hv-kvp-daemon.service
index c86a812..015c96f 100644
--- a/debian/linux-cloud-tools-common.hv-kvp-daemon.service
+++ b/debian/linux-cloud-tools-common.hv-kvp-daemon.service
@@ -4,6 +4,11 @@
 [Unit]
 Description=Hyper-V KVP Protocol Daemon
 ConditionVirtualization=microsoft
+DefaultDependencies=no
+After=systemd-remount-fs.service
+Before=shutdown.target cloud-init-local.service walinuxagent.service
+Conflicts=shutdown.target
+RequiresMountsFor=/var/lib/hyperv
 
 [Service]
 ExecStart=/usr/sbin/hv_kvp_daemon -n
--
2.7.4


--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team
Reply | Threaded
Open this post in threaded view
|

ACK: [PATCH][XBC] UBUNTU: [Debian] hyper-v -- Ensure that hv-kvp-daemon.service starts before walinuxagent.service

Marcelo Henrique Cerri
Acked-by: Marcelo Henrique Cerri <[hidden email]>

--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team

signature.asc (499 bytes) Download Attachment
Reply | Threaded
Open this post in threaded view
|

ACK: [PATCH][XBC] UBUNTU: [Debian] hyper-v -- Ensure that hv-kvp-daemon.service starts before walinuxagent.service

Kamal Mostafa-2
In reply to this post by Eric Desrochers

Acked-by: Kamal Mostafa <[hidden email]>

On Fri, Aug 03, 2018 at 10:49:07AM -0400, Eric Desrochers wrote:

> From: Paul Meyer <[hidden email]>
>
> BugLink: https://bugs.launchpad.net/bugs/1739107
>
> The hv-kvp-daemon service starts after walinuxagent. The default dependencies make hv-kvp-daemon
> wait until the whole system is up before it can start, which is not necessary and blocks Azure
> telemetry use case. This commit changes the hv-kvp-daemon.service unit file to start it up as
> early as possible so that the channel for telemetry can be use.
>
> Signed-off-by: Eric Desrochers <[hidden email]>
> ---
>  debian/linux-cloud-tools-common.hv-kvp-daemon.service | 5 +++++
>  1 file changed, 5 insertions(+)
>
> diff --git a/debian/linux-cloud-tools-common.hv-kvp-daemon.service b/debian/linux-cloud-tools-common.hv-kvp-daemon.service
> index c86a812..015c96f 100644
> --- a/debian/linux-cloud-tools-common.hv-kvp-daemon.service
> +++ b/debian/linux-cloud-tools-common.hv-kvp-daemon.service
> @@ -4,6 +4,11 @@
>  [Unit]
>  Description=Hyper-V KVP Protocol Daemon
>  ConditionVirtualization=microsoft
> +DefaultDependencies=no
> +After=systemd-remount-fs.service
> +Before=shutdown.target cloud-init-local.service walinuxagent.service
> +Conflicts=shutdown.target
> +RequiresMountsFor=/var/lib/hyperv
>  
>  [Service]
>  ExecStart=/usr/sbin/hv_kvp_daemon -n
> --
> 2.7.4
>
>
> --
> kernel-team mailing list
> [hidden email]
> https://lists.ubuntu.com/mailman/listinfo/kernel-team

--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team
Reply | Threaded
Open this post in threaded view
|

APPLIED[UC]: [PATCH][XBC] UBUNTU: [Debian] hyper-v -- Ensure that hv-kvp-daemon.service starts before walinuxagent.service

Thadeu Lima de Souza Cascardo-3
In reply to this post by Eric Desrochers
Applied to cosmic master-next branch and unstable master branch.

Thanks.
Cascardo.

Applied-to: cosmic/master-next
Applied-to: unstable/master

--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team
Reply | Threaded
Open this post in threaded view
|

APPLIED: [PATCH][XBC] UBUNTU: [Debian] hyper-v -- Ensure that hv-kvp-daemon.service starts before walinuxagent.service

Khalid Elmously
In reply to this post by Eric Desrochers
to xenial/master-next and bionic/master-next


On 2018-08-03 10:49:07 , Eric Desrochers wrote:

> From: Paul Meyer <[hidden email]>
>
> BugLink: https://bugs.launchpad.net/bugs/1739107
>
> The hv-kvp-daemon service starts after walinuxagent. The default dependencies make hv-kvp-daemon
> wait until the whole system is up before it can start, which is not necessary and blocks Azure
> telemetry use case. This commit changes the hv-kvp-daemon.service unit file to start it up as
> early as possible so that the channel for telemetry can be use.
>
> Signed-off-by: Eric Desrochers <[hidden email]>
> ---
>  debian/linux-cloud-tools-common.hv-kvp-daemon.service | 5 +++++
>  1 file changed, 5 insertions(+)
>
> diff --git a/debian/linux-cloud-tools-common.hv-kvp-daemon.service b/debian/linux-cloud-tools-common.hv-kvp-daemon.service
> index c86a812..015c96f 100644
> --- a/debian/linux-cloud-tools-common.hv-kvp-daemon.service
> +++ b/debian/linux-cloud-tools-common.hv-kvp-daemon.service
> @@ -4,6 +4,11 @@
>  [Unit]
>  Description=Hyper-V KVP Protocol Daemon
>  ConditionVirtualization=microsoft
> +DefaultDependencies=no
> +After=systemd-remount-fs.service
> +Before=shutdown.target cloud-init-local.service walinuxagent.service
> +Conflicts=shutdown.target
> +RequiresMountsFor=/var/lib/hyperv
>  
>  [Service]
>  ExecStart=/usr/sbin/hv_kvp_daemon -n
> --
> 2.7.4
>
>
> --
> kernel-team mailing list
> [hidden email]
> https://lists.ubuntu.com/mailman/listinfo/kernel-team

--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team