[Teas] Éric Vyncke's No Objection on draft-ietf-lsr-isis-rfc5316bis-04: (with COMMENT)
Éric Vyncke via Datatracker <noreply@ietf.org> Fri, 16 September 2022 08:27 UTC
Return-Path: <noreply@ietf.org>
X-Original-To: teas@ietf.org
Delivered-To: teas@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id AAED6C14F73F; Fri, 16 Sep 2022 01:27:43 -0700 (PDT)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit
From: Éric Vyncke via Datatracker <noreply@ietf.org>
To: The IESG <iesg@ietf.org>
Cc: draft-ietf-lsr-isis-rfc5316bis@ietf.org, lsr-chairs@ietf.org, lsr@ietf.org, chopps@chopps.org, teas@ietf.org, chopps@chopps.org
X-Test-IDTracker: no
X-IETF-IDTracker: 8.16.0
Auto-Submitted: auto-generated
Precedence: bulk
Reply-To: Éric Vyncke <evyncke@cisco.com>
Message-ID: <166331686368.59474.12583201274811500459@ietfa.amsl.com>
Date: Fri, 16 Sep 2022 01:27:43 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/teas/GmqzOP-8cUQuKYAJ4H1QnFe371E>
Subject: [Teas] Éric Vyncke's No Objection on draft-ietf-lsr-isis-rfc5316bis-04: (with COMMENT)
X-BeenThere: teas@ietf.org
X-Mailman-Version: 2.1.39
List-Id: Traffic Engineering Architecture and Signaling working group discussion list <teas.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/teas>, <mailto:teas-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/teas/>
List-Post: <mailto:teas@ietf.org>
List-Help: <mailto:teas-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/teas>, <mailto:teas-request@ietf.org?subject=subscribe>
X-List-Received-Date: Fri, 16 Sep 2022 08:27:43 -0000
Éric Vyncke has entered the following ballot position for draft-ietf-lsr-isis-rfc5316bis-04: 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/about/groups/iesg/statements/handling-ballot-positions/ for more information about how to handle DISCUSS and COMMENT positions. The document, along with other ballot positions, can be found here: https://datatracker.ietf.org/doc/draft-ietf-lsr-isis-rfc5316bis/ ---------------------------------------------------------------------- COMMENT: ---------------------------------------------------------------------- # Éric Vyncke, INT AD, draft-ietf-lsr-isis-rfc5316bis-04 CC @evyncke Thank you for the work put into this document especially about `This document builds on RFC 5316 by adding support for IPv6-only operation.` Please find below some non-blocking COMMENT points (but replies would be appreciated even if only for my own education). Special thanks to Christian Hopp for the shepherd's detailed write-up including the WG consensus *and* the justification of the intended status. I hope that this review helps to improve the document, Regards, -éric ## COMMENTS ### Section 3.1 ``` The Router ID field of the inter-AS reachability TLV is 4 octets in length, which contains the IPv4 Router ID of the router who generates the inter-AS reachability TLV. The Router ID SHOULD be identical to the value advertised in the Traffic Engineering Router ID TLV [RFC5305]. If no Traffic Engineering Router ID is assigned, the Router ID SHOULD be identical to an IP Interface Address [RFC1195] advertised by the originating IS. ``` AFAIK, the router ID is 'just' a 32-bit value that it is protocol version agnostic. So, s/IPv4 Router ID/Router ID/ ? Suggest: s/IP Interface Address [RFC1195]/IPv4 Interface Address [RFC1195]/ ? ### Section 6.1 & 6.2 `This document defines the following new IS-IS TLV type` but this type is already defined in RFC 5316, so does it still qualify as "new" ? Propose to rewrite the IANA section to simply request IANA to update the registries to point to this I-D rather than to RFC 5316. ### Section 7 While Les was not an author of RFC 5316, he is an author of this I-D, so no more need to acknowledge him ;-) ## Notes This review is in the ["IETF Comments" Markdown format][ICMF], You can use the [`ietf-comments` tool][ICT] to automatically convert this review into individual GitHub issues. [ICMF]: https://github.com/mnot/ietf-comments/blob/main/format.md [ICT]: https://github.com/mnot/ietf-comments
- [Teas] Éric Vyncke's No Objection on draft-ietf-l… Éric Vyncke via Datatracker
- Re: [Teas] Éric Vyncke's No Objection on draft-ie… John Scudder
- Re: [Teas] Éric Vyncke's No Objection on draft-ie… Eric Vyncke (evyncke)
- Re: [Teas] Éric Vyncke's No Objection on draft-ie… Acee Lindem (acee)
- Re: [Teas] Éric Vyncke's No Objection on draft-ie… t petch
- Re: [Teas] Éric Vyncke's No Objection on draft-ie… John Scudder
- Re: [Teas] Éric Vyncke's No Objection on draft-ie… t petch
- Re: [Teas] [Lsr] Éric Vyncke's No Objection on dr… Les Ginsberg (ginsberg)
- Re: [Teas] [Lsr] Éric Vyncke's No Objection on dr… t petch
- Re: [Teas] [Lsr] Éric Vyncke's No Objection on dr… Les Ginsberg (ginsberg)
- Re: [Teas] [Lsr] Éric Vyncke's No Objection on dr… John Scudder
- Re: [Teas] [Lsr] Éric Vyncke's No Objection on dr… Les Ginsberg (ginsberg)
- Re: [Teas] [Lsr] Éric Vyncke's No Objection on dr… t petch