[pve-devel] applied: [PATCH cluster] pve-cluster: dont pretend to be a time-sync provider

[Thomas Lamprecht]

On 1/15/19 8:46 AM, Fabian Grünbichler wrote:
> time-sync.target is a special passive unit, consumers (i.e., units that
> intend to say "I want to start after synchronized time has been
> established") should only order themselves after it. only providers
> (i.e., units that intend to say "I am responsible for synchronizing the
> clock") should pull it in via a dependency.
> 
> this and 2c7182b76f59b32efaba3135c095b9c722957af1 together also fix #998
> 

argh, I even had the systemd.special man page open and thought about this
when I looked at it last week.. -.-

applied, thanks.

> Signed-off-by: Fabian Grünbichler <f.gruenbichler at proxmox.com>
> ---
>  debian/pve-cluster.service | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/debian/pve-cluster.service b/debian/pve-cluster.service
> index 9cd1a0e..da163e8 100644
> --- a/debian/pve-cluster.service
> +++ b/debian/pve-cluster.service
> @@ -2,7 +2,6 @@
>  Description=The Proxmox VE cluster filesystem
>  ConditionFileIsExecutable=/usr/bin/pmxcfs
>  Wants=corosync.service
> -Wants=time-sync.target
>  Wants=rrdcached.service
>  Before=corosync.service
>  Before=ceph.service
>