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

David Noveck <davenoveck@gmail.com> Tue, 23 May 2017 13:34 UTC

Return-Path: <davenoveck@gmail.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 0EF49129B49; Tue, 23 May 2017 06:34:39 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.699
X-Spam-Level:
X-Spam-Status: No, score=-2.699 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, FREEMAIL_FROM=0.001, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_LOW=-0.7, SPF_PASS=-0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (2048-bit key) header.d=gmail.com
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 GJe8i4ZkoLvl; Tue, 23 May 2017 06:34:36 -0700 (PDT)
Received: from mail-it0-x231.google.com (mail-it0-x231.google.com [IPv6:2607:f8b0:4001:c0b::231]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 8042D129B42; Tue, 23 May 2017 06:34:26 -0700 (PDT)
Received: by mail-it0-x231.google.com with SMTP id g126so19615563ith.0; Tue, 23 May 2017 06:34:26 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :cc; bh=l865R/Nmga7OJ8PRrAOVx/uYW2oZhNqjuJMxosMdIHA=; b=Z3lKhtR8oyKARUNz7oyZUsrxywwsQ9fcNfsQXSqJJCqFe5v+xd0iaqMFSguQJXAc5/ oG/PCwUJ5XOg3PKh4Bq5l8XJHOv6etuQjgnI4254u4GjfowQGGVyDbkVjG9EMEbpxCeZ Nx+jLv5EkUoupTyFJbFwLXE8UG9RHT9ytcuV6/foTA829yoy6N4+mA9kgPUm2U4zV6wF fE6mddM3NS3Z18SUyYWtjnhgpsz/cunNsOVocN2kKp9kcT4Pdh7N3u2NdRT73TkRzQ13 Ionv1EOY6L8+GvknwUa/GI0CQDjR3by/GRgoYgxbOIx4KnWqgQMhjjrnjcZeNLR4U8Hh iewQ==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to:cc; bh=l865R/Nmga7OJ8PRrAOVx/uYW2oZhNqjuJMxosMdIHA=; b=gt9PrVYM+8jyS+603R2jBkgcZnBp8YNN+kCXs9N642hc3OfIWLWoFqnJFiP6aNS7pd O2u4M01k5vWeolpRCNV5FubPDn5LyoQZog3CW3m4x+ixhJ7pYC8SO/dZz2LzqByrXSpv g2z/TR362NZFUVYu6qI6KcIdvpUWsc6HlIqaiizvljJHihNSkxDSL3viS914/vbwVzJd /2SgVFvhyHY2dblZr7mCNTOcMuoVCKEky4+HVBqnmgdFTrH5BCs41R2KKdZ3rtbJ8uXQ YQtAKegXTKxhqW8qc2rPN1kS4gCTG5TWoMK6B2WQGi4kLy6wW8LMKqXYSN8j1hLdLpdD KAKg==
X-Gm-Message-State: AODbwcD0poWa5pWbMoccDAsDLAEBOXjqOG8vpilTEniTspQA/G0uPhXJ rqiBwCDEslAR8x8vT1Rmebla39OFW7lf
X-Received: by 10.36.31.70 with SMTP id d67mr2948092itd.80.1495546465833; Tue, 23 May 2017 06:34:25 -0700 (PDT)
MIME-Version: 1.0
Received: by 10.107.4.148 with HTTP; Tue, 23 May 2017 06:34:25 -0700 (PDT)
In-Reply-To: <149554351279.16302.17399687976205233598.idtracker@ietfa.amsl.com>
References: <149554351279.16302.17399687976205233598.idtracker@ietfa.amsl.com>
From: David Noveck <davenoveck@gmail.com>
Date: Tue, 23 May 2017 09:34:25 -0400
Message-ID: <CADaq8jfqP0W-OM_HWMC_JAL7bR73+mNFyrnk8O1Rs6fEJB7AOg@mail.gmail.com>
To: Alexey Melnikov <aamelnikov@fastmail.fm>
Cc: The IESG <iesg@ietf.org>, draft-ietf-nfsv4-versioning@ietf.org, Spencer Shepler <spencer.shepler@gmail.com>, "nfsv4-chairs@ietf.org" <nfsv4-chairs@ietf.org>, "nfsv4@ietf.org" <nfsv4@ietf.org>
Content-Type: multipart/alternative; boundary="001a1144a060d75d100550310ba2"
Archived-At: <https://mailarchive.ietf.org/arch/msg/nfsv4/xnNTsgCq-8oUnThHXwJ_xpm5_jQ>
Subject: Re: [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
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, 23 May 2017 13:34:39 -0000

>  It is probably overkill for
> many other protocols, but I am glad that you wrote it for NFSv4+.

I'm glad that it has been written :-)

There were some people in the working group who initially thought it was
overkill for NFSv4 as well.  The reason it had to be written is that we
have a set of standards-track documents prohibiting these sorts of
extensions.

> 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

Yes.  Will clarify.

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

Will add the reference.

On Tue, May 23, 2017 at 8:45 AM, Alexey Melnikov <aamelnikov@fastmail.fm>
wrote:

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