[nfsv4] Alexey Melnikov's No Objection on draft-ietf-nfsv4-versioning-09: (with COMMENT)

Alexey Melnikov <aamelnikov@fastmail.fm> Tue, 23 May 2017 12:45 UTC

Return-Path: <aamelnikov@fastmail.fm>
X-Original-To: nfsv4@ietf.org
Delivered-To: nfsv4@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id C47B3129B30; Tue, 23 May 2017 05:45:12 -0700 (PDT)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Alexey Melnikov <aamelnikov@fastmail.fm>
To: The IESG <iesg@ietf.org>
Cc: draft-ietf-nfsv4-versioning@ietf.org, Spencer Shepler <spencer.shepler@gmail.com>, nfsv4-chairs@ietf.org, spencer.shepler@gmail.com, nfsv4@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 6.51.0
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <149554351279.16302.17399687976205233598.idtracker@ietfa.amsl.com>
Date: Tue, 23 May 2017 05:45:12 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/nfsv4/S3V0UH217-UtJApEoYxSonCbVrc>
Subject: [nfsv4] Alexey Melnikov's No Objection on draft-ietf-nfsv4-versioning-09: (with COMMENT)
X-BeenThere: nfsv4@ietf.org
X-Mailman-Version: 2.1.22
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, 23 May 2017 12:45:13 -0000

Alexey Melnikov has entered the following ballot position for
draft-ietf-nfsv4-versioning-09: No Objection

When responding, please keep the subject line intact and reply to all
email addresses included in the To and CC lines. (Feel free to cut this
introductory paragraph, however.)


Please refer to https://www.ietf.org/iesg/statement/discuss-criteria.html
for more information about IESG DISCUSS and COMMENT positions.


The document, along with other ballot positions, can be found here:
https://datatracker.ietf.org/doc/draft-ietf-nfsv4-versioning/



----------------------------------------------------------------------
COMMENT:
----------------------------------------------------------------------

This is one of the most comprehensive documents on
versionning/extensibility that I've seen. It is probably overkill for
many other protocols, but I am glad that you wrote it for NFSv4+.

I only have a few minor comments:

In Section 6:

   Extensions to the most recently published NFSv4 minor version may be
   made by publishing the extension as a Proposed Standard, unless the
   minor version in question has been defined as non-extensible.  A
   document need not update the document defining the minor version,

Do you mean "need not update" in the sense of not using "Updates" header
in the resulting RFC? If yes, I think you should make it clearer.

   which remains a valid description of the base variant of the minor
   version in question.

In Section 8:

   This section addresses issues related to rules #11 and #13 in the
   minor versioning rules in [RFC5661].

It would be good to add section number reference here (Section 2.7), to
save readers troubles trying to figure out what #11 and #13 mean.