[trill] Alissa Cooper's Discuss on draft-ietf-trill-smart-endnodes-10: (with DISCUSS and COMMENT)

Alissa Cooper <alissa@cooperw.in> Tue, 06 March 2018 20:45 UTC

Return-Path: <alissa@cooperw.in>
X-Original-To: trill@ietf.org
Delivered-To: trill@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id A0FD91270B4; Tue, 6 Mar 2018 12:45:00 -0800 (PST)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Alissa Cooper <alissa@cooperw.in>
To: "The IESG" <iesg@ietf.org>
Cc: draft-ietf-trill-smart-endnodes@ietf.org, trill-chairs@ietf.org, shares@ndzh.com, trill@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 6.74.0
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <152036910061.28346.1223384759329179194.idtracker@ietfa.amsl.com>
Date: Tue, 06 Mar 2018 12:45:00 -0800
Archived-At: <https://mailarchive.ietf.org/arch/msg/trill/g40p63SLqq1YpRdLcEt-dy6ZtjM>
Subject: [trill] Alissa Cooper's Discuss on draft-ietf-trill-smart-endnodes-10: (with DISCUSS and COMMENT)
X-BeenThere: trill@ietf.org
X-Mailman-Version: 2.1.22
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: <https://mailarchive.ietf.org/arch/browse/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: Tue, 06 Mar 2018 20:45:01 -0000

Alissa Cooper has entered the following ballot position for
draft-ietf-trill-smart-endnodes-10: Discuss

When responding, please keep the subject line intact and reply to all
email addresses included in the To and CC lines. (Feel free to cut this
introductory paragraph, however.)


Please refer to https://www.ietf.org/iesg/statement/discuss-criteria.html
for more information about IESG DISCUSS and COMMENT positions.


The document, along with other ballot positions, can be found here:
https://datatracker.ietf.org/doc/draft-ietf-trill-smart-endnodes/



----------------------------------------------------------------------
DISCUSS:
----------------------------------------------------------------------

This should hopefully be easy to fix and was pointed out by the Gen-ART
reviewer:

All of section 4.3 is confusing as to what the length of the TLV really is.
Row 3 in the diagram says 2 bytes or 4 bytes, but the number of bits called out
in bullets 4 and 5 below it don't seem to add up to those things. Maybe it would
be better to draw a diagram with F=0 and a separate diagram with F=1.

Please make it clear both in the diagram and in the text what the expected
lengths of the fields are -- I find it particularly confusing that the number
of bits pictured doesn't align with the number of bits specified in the text
per field.


----------------------------------------------------------------------
COMMENT:
----------------------------------------------------------------------

Please also look at the Gen-ART reviewer's other comments.