[tsvwg] Genart telechat review of draft-ietf-tsvwg-ecn-experimentation-05

Brian Carpenter <brian.e.carpenter@gmail.com> Fri, 01 September 2017 02:01 UTC

Return-Path: <brian.e.carpenter@gmail.com>
X-Original-To: tsvwg@ietf.org
Delivered-To: tsvwg@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id 3423D1330F3; Thu, 31 Aug 2017 19:01:34 -0700 (PDT)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Brian Carpenter <brian.e.carpenter@gmail.com>
To: gen-art@ietf.org
Cc: draft-ietf-tsvwg-ecn-experimentation.all@ietf.org, tsvwg@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 6.59.0
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <150423129417.4604.2689843099266054478@ietfa.amsl.com>
Date: Thu, 31 Aug 2017 19:01:34 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/tsvwg/TYMecwCfWMz6qxEbPFRTPz6DoDY>
Subject: [tsvwg] Genart telechat review of draft-ietf-tsvwg-ecn-experimentation-05
X-BeenThere: tsvwg@ietf.org
X-Mailman-Version: 2.1.22
List-Id: Transport Area Working Group <tsvwg.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/tsvwg>, <mailto:tsvwg-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/tsvwg/>
List-Post: <mailto:tsvwg@ietf.org>
List-Help: <mailto:tsvwg-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/tsvwg>, <mailto:tsvwg-request@ietf.org?subject=subscribe>
X-List-Received-Date: Fri, 01 Sep 2017 02:01:34 -0000

Reviewer: Brian Carpenter
Review result: Ready with Issues

Gen-ART Last Call review of draft-ietf-tsvwg-ecn-experimentation-05

I am the assigned Gen-ART reviewer for this draft. The General Area
Review Team (Gen-ART) reviews all IETF documents being processed
by the IESG for the IETF Chair.  Please treat these comments just
like any other last call comments.

For more information, please see the FAQ at
<http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq>.

Document: draft-ietf-tsvwg-ecn-experimentation-05.txt
Reviewer: Brian Carpenter
Review Date: 2017-09-01
IETF LC End Date: 2017-09-14
IESG Telechat date: 2017-09-14

Summary: Ready with (minor) issues
--------

Comment: Very clear from the technical standpoint.
--------

Minor Issues:
-------------

> 3.  ECN Nonce and RFC 3540
...
> o  Updates RFC 3168 [RFC3168] to remove discussion of the ECN Nonce
>    and use of ECT(1) for that Nonce.  The specific text updates are
>    omitted for brevity.

I understand the desire for brevity, but this bothers me a bit. What is
the reader to make of RFC3168 Section 20.2, for example? My feeling is
that a short Appendix outlining the specific updates would be useful.
There's already too much spaghetti to untangle.

I see no reason why RFC3540 and RFC5622 need to be normative references
(and therefore downrefs). They aren't required reading in order to
understand this draft.

--