[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