[Gen-art] Gen-ART LC review of draft-ietf-trill-channel-tunnel-09
"Peter Yee" <peter@akayla.com> Sat, 02 July 2016 06:33 UTC
Return-Path: <peter@akayla.com>
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 535CC12B059; Fri, 1 Jul 2016 23:33:53 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.92
X-Spam-Level:
X-Spam-Status: No, score=-1.92 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RCVD_IN_DNSWL_NONE=-0.0001, RCVD_IN_MSPIKE_H3=-0.01, RCVD_IN_MSPIKE_WL=-0.01] autolearn=ham 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 V6S-XDHLunC0; Fri, 1 Jul 2016 23:33:51 -0700 (PDT)
Received: from p3plsmtpa09-09.prod.phx3.secureserver.net (p3plsmtpa09-09.prod.phx3.secureserver.net [173.201.193.238]) (using TLSv1.2 with cipher DHE-RSA-AES128-SHA (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 748D912D0D1; Fri, 1 Jul 2016 23:33:51 -0700 (PDT)
Received: from spectre ([173.8.184.78]) by p3plsmtpa09-09.prod.phx3.secureserver.net with id DiZq1t0061huGat01iZquH; Fri, 01 Jul 2016 23:33:51 -0700
From: Peter Yee <peter@akayla.com>
To: draft-ietf-trill-channel-tunnel.all@ietf.org
Date: Fri, 01 Jul 2016 23:33:52 -0700
Message-ID: <01df01d1d42b$b3602580$1a207080$@akayla.com>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
X-Mailer: Microsoft Outlook 14.0
Thread-Index: AdHUKLFveMyBoLVvSHiWLX9iuVl1iw==
Content-Language: en-us
Archived-At: <https://mailarchive.ietf.org/arch/msg/gen-art/4eTb8hg8Gc2Q1McI0bz8bEHyFCg>
Cc: gen-art@ietf.org, ietf@ietf.org
Subject: [Gen-art] Gen-ART LC review of draft-ietf-trill-channel-tunnel-09
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.17
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: Sat, 02 Jul 2016 06:33:53 -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 comment. For background on Gen-ART, please see the FAQ at <http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq> Document: draft-ietf-trill-channel-tunnel-09 Reviewer: Peter Yee Review Date: July 1, 2016 IETF LC End Date: July 1, 2016 IESG Telechat date: July 7, 2016 Summary: This draft is basically ready for publication as a Proposed Standard, but has some nits that should be fixed before publication. [Ready with nits] This draft extends TRILL RBridge Channels so that they can transmit additional, tunneled message types. Security services for RBridge Channel messages can be provisioned via RFC 5310 authentication and/or DTLS. The draft is well-written and easy to understand in the larger TRILL context. Major issues: None Minor issues: None Nits: General: For cases of "[RFC5310] Based authentication" to "[RFC5310]-based authentication". Watch for one instance of "RFC 5310 Based" as well. Specific: Page 3, Section 1, 1st paragraph, last sentence: delete the comma following "link". Page 4, "HKDF" definition: Change "Hash based" to "HMAC-based". Page 4, "MTU" definition: add a period at the end of the definition for consistency. Page 4, "Sz" definition: change "Campus wide" to "Campus-wide". Page 6, 1st full paragraph, 1st sentence: suggest changing "RBridge Channel Extension Protocol" to "Extended RBridge Channel Protocol" as this is the usage throughout the rest of the document. Page 8, Section 3.1, 3rd sentence: insert "tunneled" before "data". I hope this will help clarity when referring back to Figure 2.4 which includes "Tunneled Data". Page 8, Section 3.2, 1st sentence: append "(tunneled data)" after "payload". This is done for the same reason, although I'm not recommending doing this for all further occurrences of "payload" in other sections as I hope the connection is made by that point. Page 12, 1st paragraph, 1st sentence: change "link local" to "link-local". Page 12, 1st paragraph, 2nd sentence: change "These constructed addresses" to "A constructed address". Page 14, Section 4, 2nd paragraph, 1st sentence: change "use" to "used". Page 14, Section 4, 3rd paragraph, 1st sentence: change "DTLS based" to "DTLS-based". Page 14, Section 4, 4th paragraph, 2nd sentence: change "data accessible" to "data-accessible". Page 15, 1st partial paragraph, last sentence: insert "the" before "output-derived". Page 16, 1st bullet item: change "or" to "on". Page 17, 1st paragraph: delete the comma after "keying". Page 18, 2nd full paragraph, last sentence: change "secuirty" to "security". Page 20, Section 6.2, 1st paragraph: change "a" to "an". Page 21, Section 7, 3rd paragraph, 2nd sentence: delete "processing of". Or change "processing" to "process".
- Re: [Gen-art] Gen-ART LC review of draft-ietf-tri… Peter Yee
- Re: [Gen-art] Gen-ART LC review of draft-ietf-tri… Donald Eastlake
- [Gen-art] Gen-ART LC review of draft-ietf-trill-c… Peter Yee