linux-nfs.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Anna Schumaker <Anna.Schumaker@netapp.com>
To: "J. Bruce Fields" <bfields@fieldses.org>,
	Chuck Lever <chuck.lever@oracle.com>
Cc: Steve Dickson <SteveD@redhat.com>,
	"Adamson, Andy" <William.Adamson@netapp.com>,
	Trond Myklebust <trond.myklebust@primarydata.com>,
	Linux NFS Mailing List <linux-nfs@vger.kernel.org>
Subject: Re: [PATCH Version 3 0/2] Add multihostname support for NFSv4.1,2
Date: Fri, 29 Apr 2016 15:39:35 -0400	[thread overview]
Message-ID: <e0eaf201-e09b-04e9-ad33-aef12a5be4ba@Netapp.com> (raw)
In-Reply-To: <20160429180532.GB6035@fieldses.org>

On 04/29/2016 02:05 PM, J. Bruce Fields wrote:
> On Fri, Apr 29, 2016 at 11:24:30AM -0400, Chuck Lever wrote:
>>
>>> On Apr 29, 2016, at 10:46 AM, Steve Dickson <SteveD@redhat.com> wrote:
>>>
>>>
>>>
>>> On 04/29/2016 10:27 AM, Adamson, Andy wrote:
>>>> Hi Steve
>>>>
>>>> Yes, if we decide to keep the multiple hostname option, then a man page update is required. I don't think we have a consensus on using the multiple hostname mount option as a CLI to express session trunking addresses. Chuck Lever made some good points around not using multiple hostnames:
>>>>
>>>> ---- From Chuck: ----
>>>> - client admins can specify arbitrary hostnames on the command line; hostnames
>>>> for instance that correspond to some other server.
>>>>
>>>> - network conditions can change at anytime, making
>>>> the original set of trunks lop-sided, or some trunks
>>>> may become unreachable. What if the server reboots
>>>> with new i/f's or with one or more removed? The
>>>> client would likely have to remount in these cases
>>>> to adapt to network configuration changes.
>>>>
>>>> - multiple hostnames could be nailed into
>>>> /etc/fstab on potentially hundreds of clients. When
>>>> server or network configuration changes, there would
>>>> have to be a manual change on all these clients.
>>>> ----------
>>>>
>>>> What do you think? Should we keep the multiple hostname CLI as one method of expressing session trunking addresses?
>>> I would think so...
>>
>> I don't believe a mount CLI is an obvious good choice.
>>
>> The client and server should provide some indication
>> to each other that session trunking is supported. The
>> server should provide the proper configuration
>> parameters, which can change even while a client has
>> mounted the server.
>>
>> That's why I favor having the client perform a
>> GETATTR(fs_locations) on the server's pseudofs, via
>> which the server provides the correct addresses to
>> use. The client can poll for changes in the address
>> list on a regular basis.
>>
>> Please, let's automate this instead of having to
>> nail one more wonky feature into the mount CLI?
> 
> Yeah, I guess that makes sense.
> 
> My worries from the previous thread were that the fs_locations and
> fs_locations_info don't *really* give enough information to guarantee
> that trunking will be an improvement.
> 
> But I'd guess those cases aren't common (maybe fs_locations use isn't
> even that common).  Still, might want a way to opt out.
> 
> Maybe it would be worth documenting what the automatic probing does so
> that servers know how to influence it if desired.

Would it make sense to add configuration options to /etc/exports to set up how the server handles this?

Anna

> 
> --b.
> --
> 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
> 


  reply	other threads:[~2016-04-29 19:39 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-04-27 15:36 [PATCH Version 3 0/2] Add multihostname support for NFSv4.1,2 andros
2016-04-27 15:36 ` [PATCH Version 3 1/2] NFS parse NFSv4 multiple hostnames andros
2016-04-27 15:36 ` [PATCH Version 3 2/2] NFS add multiaddr mount option andros
2016-04-29 14:12 ` [PATCH Version 3 0/2] Add multihostname support for NFSv4.1,2 Steve Dickson
2016-04-29 14:27   ` Adamson, Andy
2016-04-29 14:46     ` Steve Dickson
2016-04-29 15:09       ` Adamson, Andy
2016-04-29 15:24       ` Chuck Lever
2016-04-29 15:50         ` Steve Dickson
2016-04-29 15:53           ` Anna Schumaker
2016-04-29 18:05         ` J. Bruce Fields
2016-04-29 19:39           ` Anna Schumaker [this message]
2016-04-29 20:03             ` J. Bruce Fields

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=e0eaf201-e09b-04e9-ad33-aef12a5be4ba@Netapp.com \
    --to=anna.schumaker@netapp.com \
    --cc=SteveD@redhat.com \
    --cc=William.Adamson@netapp.com \
    --cc=bfields@fieldses.org \
    --cc=chuck.lever@oracle.com \
    --cc=linux-nfs@vger.kernel.org \
    --cc=trond.myklebust@primarydata.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).