diff mbox

nfs-client.target: Removed a ordering cycle with nfs-server.service

Message ID 1412004817-14432-1-git-send-email-steved@redhat.com (mailing list archive)
State New, archived
Headers show

Commit Message

Steve Dickson Sept. 29, 2014, 3:33 p.m. UTC
On a VM that boots very fast and with out kerberos
enabled a systemd ordering cycle happens between
nfs-client and nfs-server units when both were
enabled.

Having nfs-client start the gssd daemons in the
same matter as the nfs-server does ('After=') seemed
to eliminated the ordering cycle.

Signed-off-by: Steve Dickson <steved@redhat.com>
---
 systemd/nfs-client.target | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

J. Bruce Fields Sept. 29, 2014, 6:32 p.m. UTC | #1
On Mon, Sep 29, 2014 at 11:33:37AM -0400, Steve Dickson wrote:
> On a VM that boots very fast and with out kerberos
> enabled a systemd ordering cycle happens between
> nfs-client and nfs-server units when both were
> enabled.
> 
> Having nfs-client start the gssd daemons in the
> same matter as the nfs-server does ('After=') seemed
> to eliminated the ordering cycle.

ACK.--b.

> 
> Signed-off-by: Steve Dickson <steved@redhat.com>
> ---
>  systemd/nfs-client.target | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/systemd/nfs-client.target b/systemd/nfs-client.target
> index 474f5e9..87a1ce8 100644
> --- a/systemd/nfs-client.target
> +++ b/systemd/nfs-client.target
> @@ -7,7 +7,7 @@ Wants=remote-fs-pre.target
>  # start that on demand if needed.
>  Wants=rpc-gssd.service rpc-svcgssd.service auth-rpcgss-module.service
>  Wants=nfs-blkmap.service rpc-statd-notify.service
> -Before=rpc-gssd.service rpc-svcgssd.service nfs-blkmap.service
> +After=rpc-gssd.service rpc-svcgssd.service nfs-blkmap.service
>  
>  [Install]
>  WantedBy=multi-user.target
> -- 
> 1.9.3
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Steve Dickson Sept. 30, 2014, 11:05 a.m. UTC | #2
On 09/29/2014 11:33 AM, Steve Dickson wrote:
> On a VM that boots very fast and with out kerberos
> enabled a systemd ordering cycle happens between
> nfs-client and nfs-server units when both were
> enabled.
> 
> Having nfs-client start the gssd daemons in the
> same matter as the nfs-server does ('After=') seemed
> to eliminated the ordering cycle.
> 
> Signed-off-by: Steve Dickson <steved@redhat.com>
Committed...

steved.

> ---
>  systemd/nfs-client.target | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/systemd/nfs-client.target b/systemd/nfs-client.target
> index 474f5e9..87a1ce8 100644
> --- a/systemd/nfs-client.target
> +++ b/systemd/nfs-client.target
> @@ -7,7 +7,7 @@ Wants=remote-fs-pre.target
>  # start that on demand if needed.
>  Wants=rpc-gssd.service rpc-svcgssd.service auth-rpcgss-module.service
>  Wants=nfs-blkmap.service rpc-statd-notify.service
> -Before=rpc-gssd.service rpc-svcgssd.service nfs-blkmap.service
> +After=rpc-gssd.service rpc-svcgssd.service nfs-blkmap.service
>  
>  [Install]
>  WantedBy=multi-user.target
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/systemd/nfs-client.target b/systemd/nfs-client.target
index 474f5e9..87a1ce8 100644
--- a/systemd/nfs-client.target
+++ b/systemd/nfs-client.target
@@ -7,7 +7,7 @@  Wants=remote-fs-pre.target
 # start that on demand if needed.
 Wants=rpc-gssd.service rpc-svcgssd.service auth-rpcgss-module.service
 Wants=nfs-blkmap.service rpc-statd-notify.service
-Before=rpc-gssd.service rpc-svcgssd.service nfs-blkmap.service
+After=rpc-gssd.service rpc-svcgssd.service nfs-blkmap.service
 
 [Install]
 WantedBy=multi-user.target