[trill] Shepherd review of draft-ietf-trill-cmt-04
Donald Eastlake <d3e3e3@gmail.com> Wed, 21 January 2015 22:36 UTC
Return-Path: <d3e3e3@gmail.com>
X-Original-To: trill@ietfa.amsl.com
Delivered-To: trill@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 9F9BC1A1A79 for <trill@ietfa.amsl.com>; Wed, 21 Jan 2015 14:36:12 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.75
X-Spam-Level:
X-Spam-Status: No, score=-1.75 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, FREEMAIL_ENVFROM_END_DIGIT=0.25, FREEMAIL_FROM=0.001, SPF_PASS=-0.001] autolearn=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 t9_Bq0QgU5Ky for <trill@ietfa.amsl.com>; Wed, 21 Jan 2015 14:36:11 -0800 (PST)
Received: from mail-oi0-x230.google.com (mail-oi0-x230.google.com [IPv6:2607:f8b0:4003:c06::230]) (using TLSv1 with cipher ECDHE-RSA-RC4-SHA (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 5C45A1A8982 for <trill@ietf.org>; Wed, 21 Jan 2015 14:36:11 -0800 (PST)
Received: by mail-oi0-f48.google.com with SMTP id v63so3825621oia.7 for <trill@ietf.org>; Wed, 21 Jan 2015 14:36:10 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:from:date:message-id:subject:to:content-type; bh=TOweoeLMj5bKU+44FPVsDAzgMKo83HtT56mSRPKpN1s=; b=xyD6yNv/h9FUg8cMoI7IAd9ijYN7DoJ8HN5ulwi4ZYU7VYi/KeqzjFyQvzm4SlFxLW mFxQPCcFauJPW2wBXpOV4yq2lLJ5KPUgg9cUz5nggonGiGkdVp0CCNNP8qXIAjR+2BvX ZoaDJWOSBAZMUIKlsUJtAC/AGlblmepZjAyVYX5NjLTirZwfnHZNnEa30Aijg3r8AP4U XK8JYOelY0IN6GjPB2o3tKNcu/TmM3dSAw8DxChw1ZJAi0z1kYdfPYTnClkdXLHwGk8U YUojbFJ5Wr8kzmJmx6g7WWqikCXrq3Qi0xMH6PhMDWbTt2P7IHFqY92xsfdmkKbIPNwE 0oTQ==
X-Received: by 10.182.153.133 with SMTP id vg5mr26633398obb.20.1421879770607; Wed, 21 Jan 2015 14:36:10 -0800 (PST)
MIME-Version: 1.0
Received: by 10.76.104.104 with HTTP; Wed, 21 Jan 2015 14:35:50 -0800 (PST)
From: Donald Eastlake <d3e3e3@gmail.com>
Date: Wed, 21 Jan 2015 17:35:50 -0500
Message-ID: <CAF4+nEHb1J96htYrTtYtaodgY89vF=i8CFG3yVC2zdB8jsuEsA@mail.gmail.com>
To: "trill@ietf.org" <trill@ietf.org>
Content-Type: text/plain; charset="UTF-8"
Archived-At: <http://mailarchive.ietf.org/arch/msg/trill/2suK7BOtm24yDrnL93VhxmMBA5g>
Subject: [trill] Shepherd review of draft-ietf-trill-cmt-04
X-BeenThere: trill@ietf.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: "Developing a hybrid router/bridge." <trill.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/trill>, <mailto:trill-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/trill/>
List-Post: <mailto:trill@ietf.org>
List-Help: <mailto:trill-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/trill>, <mailto:trill-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 21 Jan 2015 22:36:12 -0000
Hi, Here is the result of my review of draft-ietf-trill-cmt-04. Consider all these point to just be my opinion; I'm happy to hear counter-arguments. - There are apparently two lines that are too long. One is in Section 5.4.1 where there is an "-OR-" which looks OK but is actually at the far end of a line so long that it wraps around. I think the other one was in the IANA Considerations, which was 2 characters over the limit for a draft, but I suggest a replacement IANA Considerations Section below in any case. - The Abstract should note that the document updates RFC 6325. - There are a few places where Fine Grained Labels should be mentioned as well as VLANs and cases where "Data Label" or the like should be used to include both. (I'll send details on this and other suggestions directly to the document authors.) - The Introduction should clearly state how RFC 6325 is updated by adding something like: "This document updates [RFC6325] by changing how distribution trees are constructed when tree affinity advertisements are present." - The Acronym section should include AF, FGL, and a couple other entries. When RFC 7379 went through the IESG, we ended up changing "Classic Ethernet" to "Customer Ethernet" so we might as well do that now. - Affinity sub-TLV can appear in MT Capability TLVs as well as Router Capability TLVs. - Since the link aggregation standard has been revised to include Distributed Resilient Network Interconnect, mention that in addition to MC-LAG and, for parallelism with RFC 7379, use the phrase Local Active-Active Link Protocol. - Make the IANA Considerations section a bit more precise by replacing it with the following: "This document requires no IANA actions because the "Affinity Supported" capability bit and Affinity sub-TLV have been specified and allocated in [RFC7176]." References: - References should be updated for drafts that have been published as RFCs and for the recently published update to 802.1AX. - There is one reference to [RFC5410] that should be [RFC5310] I think. - The references mentioned in the text and those in the References sections should be brought into alignment. Minor suggestion: - On title page, make intended status "Proposed Standard". The PROTO template implies that is more desirable than the less specific "Standards Track". Thanks, Donald (Document Shepherd) ============================= Donald E. Eastlake 3rd +1-508-333-2270 (cell) 155 Beaver Street, Milford, MA 01757 USA d3e3e3@gmail.com
- [trill] Shepherd review of draft-ietf-trill-cmt-04 Donald Eastlake