Re: [nfsv4] Ben Campbell's No Objection on draft-ietf-nfsv4-mv0-trunking-update-03: (with COMMENT)

David Noveck <davenoveck@gmail.com> Wed, 16 January 2019 17:38 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 1965D124BE5; Wed, 16 Jan 2019 09:38:49 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.998
X-Spam-Level:
X-Spam-Status: No, score=-1.998 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_NONE=-0.0001, SPF_PASS=-0.001, URIBL_BLOCKED=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 XkU0txYPHAG4; Wed, 16 Jan 2019 09:38:47 -0800 (PST)
Received: from mail-oi1-x234.google.com (mail-oi1-x234.google.com [IPv6:2607:f8b0:4864:20::234]) (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 CDDE0124B0C; Wed, 16 Jan 2019 09:38:46 -0800 (PST)
Received: by mail-oi1-x234.google.com with SMTP id c206so3901698oib.0; Wed, 16 Jan 2019 09:38:46 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=956zQM9iJpNDkAfxQ1WwKBqihFRxmaJwzhONFnjmnZs=; b=TRd0f4lCLh5aZG9h58Fpqtp4605UHk0zVx/WAImZwUXmucmELkt6PGYvRpEmgORReH whRh43S7Jzki0YvnSUlIJYzv3oaG61v3CadJjXN2L93A2hFzfbaH6PsH5pdsRuS8jrsg Avh2tEtMuLYSnnrG5cgHBqxUQvcL4nNwB4ifsT2xpq6kiTUd8HtS9RjqF/W5xtTC9KBq kTQ1KnpzdTmAtrxYTIirG3YacFTkILjGRvNwZR0N6PeJ/AUp2eIT9nwQX3EKL1BSoxPf XIbTD2UNthsXJgt/cukpD0l9lGbBGdUdhIyND5p3IGgNKLxmSHK+DdzbjlbDxpK7Wzfd ApaA==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=956zQM9iJpNDkAfxQ1WwKBqihFRxmaJwzhONFnjmnZs=; b=NWmp2J4rOLkWJ7GwYwP5GcH0Fo+/c7NGvyzQygiIAj4q6x7baoacvmrWRvZrwqI/ml KwvfVHhyyfE8skfS6oZbbfThrQcwUPSPZLVPS8hi/7enx1A9nmGONLDFYmWdVARvagzq POZRISG1IWdwCTfOze2w1M69wktsqsu+eWrUhy8HGKOZ3+AZ8P+8z0A6ZHme6aYu/Doq ZAPAKONT+7B6w06wcIEkBAMNN2oyvirDtSVv12JaaNn5HjdTcF8u5cVclNY9QUwiSg9q H2elW1E473SYZjPzuRJEuvSC0268qzncng65VxfQu4vnf1GGzuAXseY3ur18DjDdbfh7 +Abg==
X-Gm-Message-State: AJcUukf/Fmgk0bKzriCYroORgKpXuZDojDkawbPFP6YbAvX2tTZcsN3s 713XSSgm+/z6/qAKSW2ZbKO/frKwnnGAsu9BFw8zN1OL
X-Google-Smtp-Source: ALg8bN4jwf7MlLv3vhRzVnJoBqd70TtxHCttunBwGLowHDohD+B0KfrdPgQda+fJxFkRPPGRp1Qgw0v/6Wvn5BycJsw=
X-Received: by 2002:aca:7297:: with SMTP id p145mr5191345oic.252.1547660325849; Wed, 16 Jan 2019 09:38:45 -0800 (PST)
MIME-Version: 1.0
References: <154707125334.5133.10041618200972102440.idtracker@ietfa.amsl.com>
In-Reply-To: <154707125334.5133.10041618200972102440.idtracker@ietfa.amsl.com>
From: David Noveck <davenoveck@gmail.com>
Date: Wed, 16 Jan 2019 12:38:34 -0500
Message-ID: <CADaq8jcfqYrGcdLiyav7zPhjkyVriRJa9jVbGcNSz=ONyduJWA@mail.gmail.com>
To: Ben Campbell <ben@nostrum.com>
Cc: The IESG <iesg@ietf.org>, draft-ietf-nfsv4-mv0-trunking-update@ietf.org, Spencer Shepler <spencer.shepler@gmail.com>, nfsv4-chairs@ietf.org, NFSv4 <nfsv4@ietf.org>
Content-Type: multipart/alternative; boundary="000000000000f48efe057f96bf33"
Archived-At: <https://mailarchive.ietf.org/arch/msg/nfsv4/qse8FsMvwjfhW2-3RPJdOiDqkCY>
Subject: Re: [nfsv4] Ben Campbell's No Objection on draft-ietf-nfsv4-mv0-trunking-update-03: (with COMMENT)
X-BeenThere: nfsv4@ietf.org
X-Mailman-Version: 2.1.29
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: Wed, 16 Jan 2019 17:38:49 -0000

> General: I don't expect a change in approach this late in the process,

Given that we have already gone through the working group process, making a
change at this point would occasion a significant delay.

> but I
> think the approach of this update (replacing, inserting, and updating
sections
> in the updated RFC) is pretty unfriendly to the readers.

On the othe hand, if we just produced an updated v4.0 section 8, the result
would be hard to review, in that you would force readers and reviewers to
diff old and new documents to see what has changed.

> This would make sense
> if we actually rendered patched versions of updated RFCs, but we do not.

We don't do so automatically or routinely but we do have the option to use
this document to produce an updated Section 8 for RFC7530.

> This
> leaves readers to flip back and forth to figure out the context of each
update.

I hope this is not too onerous

> §1: s/"which enables"/"that enables"

OK.  Will be in -04.

> §5.1 Does "updated introduction" to section 8.4 mean the same as "replaces
> section 8.4, but not it's subsections?"

Yes. I think we can make this clearer in -04 by updating the first
sub-bullet of the second bullet in Section 5.  It how reads:

Section 5.2.1 replaces the introductory material within Section 8.4 of
[RFC7530].

This could be replaced by:

Section 5.2.1 replaces the introductory material within Section 8.4 of
[RFC7530]*, *i.e., the material within that Section 8.4, exclusive of
subsections.


> §5.2.2 (and other sections that add a new subsection to 8.4). Are these
assumed
> to be inserted in a particular location under 8.4? That is, can you state
the
> new section numbers? Otherwise the reader is left to guess where these
would be
> inserted.

Will address in -04 by updating the material in the sub-bullets of the
second bullet within Section 5, to read as follows:


   - Section 5.2.2 is to be added as a new sub-section of Section 8.4
before the updated Section 8.4.1 of [RFC7530].  In a consolidated
document, it would appear as Section 8.4.1.


   - Section 5.2.3 is to be added as an additional new sub-section of
Section 8.4 before the updated Section 8.4.1 of [RFC7530]. In a
consolidated document, it would appear as Section 8.4.2.


   - Section 5.2.4 replaces Section 8.4.1 of [RFC7530], entitled "File
System Replication". In a consolidated document, it would appear as
Section 8.4.3.


   -  Section 5.2.5 replaces Section 8.4.2 of [RFC7530], entitled
"File System Migration".  In a consolidated document, it would appear
as Section 8.4.4.


   - Section 5.2.6 is to be added as a new sub-section of Section 8.4
before Section 8.4.3 of [RFC7530].   In a consolidated document, it
would appear as Section 8.4.4 while the existing Section 8.3 would
appear as Section 8.4.6.

> §6: What is meant by "outside section 8"?

I assume you are referring to the title of Section 6.  It is referring to
changes necessary in parts of [RFC7530] ouside of Section 8 of that
document.  Not sure how to make this clearer*.  *Would appreciate any
suggestions you might have.

On Wed, Jan 9, 2019 at 5:00 PM Ben Campbell <ben@nostrum.com> wrote:

> Ben Campbell has entered the following ballot position for
> draft-ietf-nfsv4-mv0-trunking-update-03: 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-mv0-trunking-update/
>
>
>
> ----------------------------------------------------------------------
> COMMENT:
> ----------------------------------------------------------------------
>
> General: I don't expect a change in approach this late in the process, but
> I
> think the approach of this update (replacing, inserting, and updating
> sections
> in the updated RFC) is pretty unfriendly to the readers. This would make
> sense
> if we actually rendered patched versions of updated RFCs, but we do not.
> This
> leaves readers to flip back and forth to figure out the context of each
> update.
>
> §1: s/"which enables"/"that enables"
>
> §5.1 Does "updated introduction" to section 8.4 mean the same as "replaces
> section 8.4, but not it's subsections?"
>
> §5.2.2 (and other sections that add a new subsection to 8.4). Are these
> assumed
> to be inserted in a particular location under 8.4? That is, can you state
> the
> new section numbers? Otherwise the reader is left to guess where these
> would be
> inserted.
>
> §6: What is meant by "outside section 8"?
>
>
>