Linux NFS development
 help / color / mirror / Atom feed
From: Chuck Lever <chuck.lever@oracle.com>
To: "J. Bruce Fields" <bfields@fieldses.org>
Cc: "Adamson, Andy" <William.Adamson@netapp.com>,
	Trond Myklebust <trond.myklebust@primarydata.com>,
	Martin Houry <martinhoury@gmail.com>,
	Linux NFS Mailing List <linux-nfs@vger.kernel.org>
Subject: Re: "Re: [PATCH RFC Version 1 0/6] Request for Comment: NFS4.1 Session Trunking"
Date: Fri, 19 Feb 2016 11:29:40 -0500	[thread overview]
Message-ID: <D09C120D-FA37-4F41-803F-42CF4CF939F2@oracle.com> (raw)
In-Reply-To: <20160219150135.GD17042@fieldses.org>


> On Feb 19, 2016, at 10:01 AM, J. Bruce Fields <bfields@fieldses.org> wrote:
> 
> On Thu, Feb 18, 2016 at 04:29:46PM -0500, Chuck Lever wrote:
>> True; there's a reason I never got to implementing
>> fs_locations_info on the Linux server for FedFS. There
>> are sticky problems around the mountd upcall that is
>> used to communicate this information to the kernel,
>> for example.
>> 
>> However, I don't agree that this is a good reason to
>> go with multiple hostnames on the mount command line.
>> I like Andy's plan to keep this CLI change out of the
>> long term upstream code, but continue to use it for
>> testing.
> 
> Even if we get an fs_locations_info-based solution to the point where
> it's the best default, don't you think we'll still want some kind of
> manual override on the client?

Perhaps! But that doesn't mean multiple hostnames in
particular is the right CLI.

Suppose a "notrunk" mount option was added. The client
would ignore the server's fs_locations_info and use
only the hostname provided on the command line.

Another idea is to provide a blacklist as a mount
option. "notrunk=IPv4,IPv4,[IPv6],[IPv6]"


> The decision here has to do with the path between client and server,
> and the server may not always be in the best position to make that
> decision.

The server is not making any decision about the path.
It simply provides a list of its i/f's.

The client may choose to use one or more of the IP
addresses on the server's list. The client is not
required to use all the server's IP addresses, and
it does have the ability to determine that some
addresses are not reachable (eg IPv6 addresses,
private networks, and so on), though that may take
a timeout or two.

But back to the client CLI:

With multiple hostnames, client admins can specify
arbitrary hostnames on the command line; hostnames
for instance that correspond to some other server.

And, 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.

Note that 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.


--
Chuck Lever





  reply	other threads:[~2016-02-19 16:29 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-02-17 10:31 "Re: [PATCH RFC Version 1 0/6] Request for Comment: NFS4.1 Session Trunking" Martin Houry
2016-02-17 14:50 ` Adamson, Andy
2016-02-17 16:34   ` J. Bruce Fields
2016-02-17 17:49     ` Adamson, Andy
2016-02-17 18:44       ` J. Bruce Fields
2016-02-17 19:57         ` Adamson, Andy
2016-02-17 19:06   ` Chuck Lever
2016-02-17 20:59     ` J. Bruce Fields
2016-02-17 22:35       ` Adamson, Andy
2016-02-17 22:52         ` Chuck Lever
2016-02-17 23:55           ` Trond Myklebust
2016-02-18 14:14             ` J. Bruce Fields
2016-02-18 14:38               ` Martin Houry
2016-02-18 18:32               ` Trond Myklebust
2016-02-18 19:41                 ` Adamson, Andy
2016-02-18 20:39                   ` J. Bruce Fields
2016-02-18 21:29                     ` Chuck Lever
2016-02-19 15:01                       ` J. Bruce Fields
2016-02-19 16:29                         ` Chuck Lever [this message]
2016-02-18 14:05           ` J. Bruce Fields
2016-02-18 11:28   ` Martin Houry

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=D09C120D-FA37-4F41-803F-42CF4CF939F2@oracle.com \
    --to=chuck.lever@oracle.com \
    --cc=William.Adamson@netapp.com \
    --cc=bfields@fieldses.org \
    --cc=linux-nfs@vger.kernel.org \
    --cc=martinhoury@gmail.com \
    --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