Re: [nfsv4] feedback on draft-ietf-nfsv4-versioning-06

Mike Kupfer <mike.kupfer@oracle.com> Tue, 18 October 2016 18:01 UTC

Return-Path: <mike.kupfer@oracle.com>
X-Original-To: nfsv4@ietfa.amsl.com
Delivered-To: nfsv4@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 52D03129762 for <nfsv4@ietfa.amsl.com>; Tue, 18 Oct 2016 11:01:45 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -4.633
X-Spam-Level:
X-Spam-Status: No, score=-4.633 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RCVD_IN_DNSWL_MED=-2.3, RCVD_IN_MSPIKE_H2=-0.001, RP_MATCHES_RCVD=-0.431, SPF_PASS=-0.001] autolearn=ham autolearn_force=no
Received: from mail.ietf.org ([4.31.198.44]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id kZiMCJVDvX_v for <nfsv4@ietfa.amsl.com>; Tue, 18 Oct 2016 11:01:44 -0700 (PDT)
Received: from userp1040.oracle.com (userp1040.oracle.com [156.151.31.81]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 3AF78129731 for <nfsv4@ietf.org>; Tue, 18 Oct 2016 11:01:44 -0700 (PDT)
Received: from aserv0022.oracle.com (aserv0022.oracle.com [141.146.126.234]) by userp1040.oracle.com (Sentrion-MTA-4.3.2/Sentrion-MTA-4.3.2) with ESMTP id u9II1guv018429 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Tue, 18 Oct 2016 18:01:42 GMT
Received: from userv0122.oracle.com (userv0122.oracle.com [156.151.31.75]) by aserv0022.oracle.com (8.14.4/8.13.8) with ESMTP id u9II1fCV029745 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Tue, 18 Oct 2016 18:01:42 GMT
Received: from abhmp0016.oracle.com (abhmp0016.oracle.com [141.146.116.22]) by userv0122.oracle.com (8.14.4/8.14.4) with ESMTP id u9II1fGE002918; Tue, 18 Oct 2016 18:01:41 GMT
Received: from [10.132.144.95] (/10.132.144.95) by default (Oracle Beehive Gateway v4.0) with ESMTP ; Tue, 18 Oct 2016 11:01:41 -0700
To: David Noveck <davenoveck@gmail.com>
References: <ote0r37ek3eq.fsf@jurassic.us.oracle.com> <CADaq8jfvP3iGAk1GN-w3HBex7LEthoTm__0GD4XYMf-dEMvFSA@mail.gmail.com>
From: Mike Kupfer <mike.kupfer@oracle.com>
Organization: Oracle Corporation
Message-ID: <58066392.1010804@oracle.com>
Date: Tue, 18 Oct 2016 11:01:54 -0700
User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:38.0) Gecko/20100101 Thunderbird/38.7.1
MIME-Version: 1.0
In-Reply-To: <CADaq8jfvP3iGAk1GN-w3HBex7LEthoTm__0GD4XYMf-dEMvFSA@mail.gmail.com>
Content-Type: text/plain; charset="utf-8"; format="flowed"
Content-Transfer-Encoding: 7bit
X-Source-IP: aserv0022.oracle.com [141.146.126.234]
Archived-At: <https://mailarchive.ietf.org/arch/msg/nfsv4/F8iFQhmZBZkIPdgV7xicBz_nkCs>
Cc: "nfsv4@ietf.org" <nfsv4@ietf.org>
Subject: Re: [nfsv4] feedback on draft-ietf-nfsv4-versioning-06
X-BeenThere: nfsv4@ietf.org
X-Mailman-Version: 2.1.17
Precedence: list
List-Id: NFSv4 Working Group <nfsv4.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/nfsv4>, <mailto:nfsv4-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/nfsv4/>
List-Post: <mailto:nfsv4@ietf.org>
List-Help: <mailto:nfsv4-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/nfsv4>, <mailto:nfsv4-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 18 Oct 2016 18:01:45 -0000

On 18-Oct-16 3:54, David Noveck wrote:

>> Section 4.4.2
[...]
> Given the way this paragraph is written, the referent of "this" would be
> unclear.  I propose adding the following new final paragraph to section
> 4.4.2:
[...]

Looks good.

>> * Section 4.4.3
[...]
> I intend to add the following:
[...]
That's fine, too.

>> 1. end of page 5, top of page 6:
>
> I intend to replace the last sentence of the penultimate paragraph of
> the section by:
>
>     For example, one may refer to the set of REQUIRED features in a
>     given minor version that has not had a protocol correction since it
>     is the same for all variants within the minor version.  See section
>     9 for a discussion of correcting an existing minor version.

Editing nits:
- A comma before "since" might make it a little easier to read.
- "section 9" should be capitalized.

Otherwise looks good.

>    > 2. page 10:
>
> What I intend to do here is replace the lead-in paragraph for the
> bulleted list by the following:
>
>     Which of these are validly returned by the responder depends on the
>     status of the protocol element in the minor version specified in
>     the COMPOUND or CB_COMPOUND.  The possibilities which can
>     exist when dealing with  minor versions that have not been corrected
>     are listed below.  See section 9.1 for a discussion of the effects
>     of protocol correction.

Editing nit: "section 9.1" should be capitalized.

Otherwise looks good.

> The text in section 9.1 is TBD.

Okay.

thanks,
mike