[Gen-art] Gen-ART Last Call review of draft-ietf-tsvwg-rfc4960-errata-06
Paul Kyzivat <pkyzivat@alum.mit.edu> Sun, 03 June 2018 19:02 UTC
Return-Path: <pkyzivat@alum.mit.edu>
X-Original-To: gen-art@ietfa.amsl.com
Delivered-To: gen-art@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id CFBE912762F for <gen-art@ietfa.amsl.com>; Sun, 3 Jun 2018 12:02:08 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.935
X-Spam-Level:
X-Spam-Status: No, score=-1.935 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RCVD_IN_DNSWL_LOW=-0.7, SPF_SOFTFAIL=0.665] autolearn=no 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 MNGap9SBf0bq for <gen-art@ietfa.amsl.com>; Sun, 3 Jun 2018 12:02:07 -0700 (PDT)
Received: from resqmta-ch2-07v.sys.comcast.net (resqmta-ch2-07v.sys.comcast.net [IPv6:2001:558:fe21:29:69:252:207:39]) (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 32C5D124D6C for <gen-art@ietf.org>; Sun, 3 Jun 2018 12:02:07 -0700 (PDT)
Received: from resomta-ch2-20v.sys.comcast.net ([69.252.207.116]) by resqmta-ch2-07v.sys.comcast.net with ESMTP id PXsWfMId0zvgzPYGUfPEsY; Sun, 03 Jun 2018 19:02:06 +0000
Received: from PaulKyzivatsMBP.localdomain ([24.62.227.142]) by resomta-ch2-20v.sys.comcast.net with ESMTPA id PYGTf7UdMTGiGPYGUfKM3s; Sun, 03 Jun 2018 19:02:06 +0000
From: Paul Kyzivat <pkyzivat@alum.mit.edu>
To: draft-ietf-tsvwg-rfc4960-errata.all@ietf.org
Cc: General Area Review Team <gen-art@ietf.org>
Message-ID: <b7644ac4-37f0-b536-6f35-c99dbdaddfd3@alum.mit.edu>
Date: Sun, 03 Jun 2018 15:02:05 -0400
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.10; rv:52.0) Gecko/20100101 Thunderbird/52.8.0
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"; format="flowed"
Content-Language: en-US
Content-Transfer-Encoding: 8bit
X-CMAE-Envelope: MS4wfBdaSts73N5uE5AnYHYlSUpGRaQ6MXTRRV6yUj4jDNd/ZhiBwSbPe+EzZ+crTP2NyGs3m4WtapxVEoXUC2ounwX5tU4q8HCUg+VvUwhCaPemKZHRzByE aDbPuJ2VOlfx9I3Ri4xyDufJhyG46wcROuS7YH1TO01vmIi0HdqzYunbXz+eqywQ4207vVq8JUBXzTdLh2JSCerrt0D6Hd5VuMA4QvmGqXNBzF5S2acpsgQ/ ryJi+B+dCKmLsB+DkjlH5Q==
Archived-At: <https://mailarchive.ietf.org/arch/msg/gen-art/KrvpQMvQD96ISoCcIfUz3V_vpVs>
Subject: [Gen-art] Gen-ART Last Call review of draft-ietf-tsvwg-rfc4960-errata-06
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.22
Precedence: list
List-Id: "GEN-ART: General Area Review Team" <gen-art.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/gen-art>, <mailto:gen-art-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/gen-art/>
List-Post: <mailto:gen-art@ietf.org>
List-Help: <mailto:gen-art-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/gen-art>, <mailto:gen-art-request@ietf.org?subject=subscribe>
X-List-Received-Date: Sun, 03 Jun 2018 19:02:09 -0000
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-rfc4960-errata-06 Reviewer: Paul Kyzivat Review Date: 2018-06-03 IETF LC End Date: 2018-06-04 IESG Telechat date: ? Summary: This draft is on the right track but has open issues, described in the review. Issues: Major: 1 Minor: 2 Nits: 1 1) MAJOR: The format of this document disturbs me. According to the abstract: ... This document provides deltas to RFC4960 and is organized in a time ordered way. The issues are listed in the order they were brought up. Because some text is changed several times the last delta in the text is the one which should be applied. This format makes the document hard to deal with. A developer who wants to implement sctp with some or all of the errata fixes will want to work from a variant of 4960 that incorporates all of those fixes - a bis. But it isn't clear how this document helps with that. I don't think you can start with 4960 and simply apply all the deltas sequentially, because overlapping changes won't work right. A developer won't be interested in the order in which errata were reported. An actual bis document would be more useful to a developer than this format. Is that not being done because doing so would be more difficult? Or because it isn't yet certain that these are the correct fixes? I think you should give some serious consideration of the most suitable form for this document, in the context of how it is intended to be used. 2) MINOR (maybe MAJOR): Discovering where one change is impacted by another change is hard. I dug into the details of the document to understand how many places there are actually overlaps between the changes in multiple sections. (It took a lot of work to do this.) I found five of these: - 3.1 / 3.23 - 3.3 / 3.43 - 3.5 / 3.10 - 3.6 / 3.23 - 3.24 / 3.32 (I don't guarantee that this list is exhaustive.) Of these, I think only one (3.1/3.23) explicitly indicates the conflict, and it only indicates it within 3.23. Most of the changes don't have any conflicts. And some of the conflicts could be removed by being more precise in indicating the change being made. In cases where this isn't possible, the presence of the conflict should be indicated in each section that has a conflict, with cross references. IOW, shift the burden of detecting conflicts from the reader to the document. 3) MINOR: Errata Tracking: Apparently each subsection of section 3 covers one erratum. But the errata numbers are not mentioned. Each section ought to reference the errata number it responds to. 4) NIT: In section 3.35 (DSCP Changes) the change to section 10.1 isn't properly indicated. It shows 'Old text' twice rather than 'Old text' and 'New text'.
- [Gen-art] Gen-ART Last Call review of draft-ietf-… Paul Kyzivat