[mpls] Roman Danyliw's No Objection on draft-ietf-mpls-rfc8287-len-clarification-02: (with COMMENT)

Roman Danyliw via Datatracker <noreply@ietf.org> Mon, 05 August 2019 12:25 UTC

Return-Path: <noreply@ietf.org>
X-Original-To: mpls@ietf.org
Delivered-To: mpls@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id 8762312015B; Mon, 5 Aug 2019 05:25:54 -0700 (PDT)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit
From: Roman Danyliw via Datatracker <noreply@ietf.org>
To: "The IESG" <iesg@ietf.org>
Cc: draft-ietf-mpls-rfc8287-len-clarification@ietf.org, Loa Andersson <loa@pi.nu>, mpls-chairs@ietf.org, loa@pi.nu, mpls@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 6.99.1
Auto-Submitted: auto-generated
Precedence: bulk
Reply-To: Roman Danyliw <rdd@cert.org>
Message-ID: <156500795454.24520.16742493473141475413.idtracker@ietfa.amsl.com>
Date: Mon, 05 Aug 2019 05:25:54 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/mpls/L_D50pfFRc0Z1g7Hu3GjY4bYkYg>
Subject: [mpls] Roman Danyliw's No Objection on draft-ietf-mpls-rfc8287-len-clarification-02: (with COMMENT)
X-BeenThere: mpls@ietf.org
X-Mailman-Version: 2.1.29
List-Id: Multi-Protocol Label Switching WG <mpls.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/mpls>, <mailto:mpls-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/mpls/>
List-Post: <mailto:mpls@ietf.org>
List-Help: <mailto:mpls-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/mpls>, <mailto:mpls-request@ietf.org?subject=subscribe>
X-List-Received-Date: Mon, 05 Aug 2019 12:25:55 -0000

Roman Danyliw has entered the following ballot position for
draft-ietf-mpls-rfc8287-len-clarification-02: No Objection

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-mpls-rfc8287-len-clarification/



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

** Section 4, per “The figures in section 5.1, 5.2 and 5.3 of [RFC8287] are
replaced by the below figures in section 4.1, 4.2 and 4.3 respectively”:

--  none of the diagram in Section 4.1, 4.2 or 4.3 are explicitly labeled as
figures

-- section 4.3 contains two “figures” (one is called a table in the text and
the other has no designation).  Which one of these is supposed to be a
replacement for RFC8287 Section 5.3?

** Section 4.3.  Assuming that the second figure is the replacement for
RFC8287’s Section 5.3 figure, the length is still confusing to me.  The figure
in this draft appears to be a specific instance of the populated Sub-TLV.  The
existing figure in RFC8287 appears to be a generic depiction.  The new figure
doesn’t appear to be relevant (or presented incorrect information) if Adj Type
= 1 and Protocol = 1 (for example).

** Section 4.3.  Typo.  s/Protocol =0/Protocol = 0/

** Section 6. Recommend clarifying that there are no additional security
considerations (not that there aren’t any).  s/This document updates [RFC8287]
and does not introduce any security considerations/This document updates
[RFC8287] and does not introduce any additional security considerations/