[mpls] Adam Roach's Discuss on draft-ietf-mpls-spring-lsp-ping-11: (with DISCUSS and COMMENT)
Adam Roach <adam@nostrum.com> Wed, 11 October 2017 22:44 UTC
Return-Path: <adam@nostrum.com>
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 B8E211326DF; Wed, 11 Oct 2017 15:44:55 -0700 (PDT)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Adam Roach <adam@nostrum.com>
To: The IESG <iesg@ietf.org>
Cc: draft-ietf-mpls-spring-lsp-ping@ietf.org, draft-ietf-mpls-spring-lsp-ping@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.63.1
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <150776189575.16711.15905921797919281380.idtracker@ietfa.amsl.com>
Date: Wed, 11 Oct 2017 15:44:55 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/mpls/TXGS_xW1tirsRQgHfI0OEWXAR9w>
Subject: [mpls] Adam Roach's Discuss on draft-ietf-mpls-spring-lsp-ping-11: (with DISCUSS and COMMENT)
X-BeenThere: mpls@ietf.org
X-Mailman-Version: 2.1.22
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: Wed, 11 Oct 2017 22:44:56 -0000
Adam Roach has entered the following ballot position for draft-ietf-mpls-spring-lsp-ping-11: 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-mpls-spring-lsp-ping/ ---------------------------------------------------------------------- DISCUSS: ---------------------------------------------------------------------- Section 5.3 indicates that "Advertising Node Identifier" and "Receiving Node Identifier" are "4 or 6 octets." There are two issues that arise with the way this is currently specified, both of which can lead to a lack of interoperability: 1. While implementors might infer that "Protocol=1" results in a 4-byte value, and that "Protocol=2" results in a 6-byte value, it's a bit unclear what length is to be used here for "Protocol=0." 2. The descriptions for both of these fields include: "When Protocol is set to 1, then the 32 rightmost bits represent OSPF Router ID." This implies that the field is *wider* than 32 bits when Protocol=1, which leaves deep ambiguity about the circumstances under which the field is allowed to be 4 octets. I would strongly recommend that this section add clear language that unambiguously spells out how implementations are expected to select the field width for the four variable-width fields in this Sub-TLV (the two I cite above as well as the interface ID fields). ---------------------------------------------------------------------- COMMENT: ---------------------------------------------------------------------- Sections 5.1, 5.2, and 5.3 define "Reserved" fields without indication of how these fields should be treated. Recommend each of these be defined to "MUST be set to 0 on send, MUST be ignored on receipt" -- this is the scheme that maximizes the ability to use them in the future. Section 5.3 sefines three values for "Adj Type": 0, 4, and 6. Please either state that all other values are and will always be an error, or create an IANA registry for this field. Section 5.3 sefines three values for "Protocol": 0, 1, and 2. Please either state that all other values are and will always be an error, or create an IANA registry for this field.
- [mpls] Adam Roach's Discuss on draft-ietf-mpls-sp… Adam Roach
- Re: [mpls] Adam Roach's Discuss on draft-ietf-mpl… Carlos Pignataro (cpignata)
- Re: [mpls] Adam Roach's Discuss on draft-ietf-mpl… Adam Roach
- Re: [mpls] Adam Roach's Discuss on draft-ietf-mpl… Carlos Pignataro (cpignata)
- Re: [mpls] Adam Roach's Discuss on draft-ietf-mpl… Carlos Pignataro (cpignata)