Éric Vyncke's No Objection on draft-ietf-rtgwg-segment-routing-ti-lfa-13: (with COMMENT)

Éric Vyncke via Datatracker <noreply@ietf.org> Wed, 10 April 2024 08:39 UTC

Return-Path: <noreply@ietf.org>
X-Original-To: rtgwg@ietf.org
Delivered-To: rtgwg@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id A4317C14F5F3; Wed, 10 Apr 2024 01:39:31 -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-rtgwg-segment-routing-ti-lfa@ietf.org, rtgwg-chairs@ietf.org, rtgwg@ietf.org, stewart.bryant@gmail.com, stewart.bryant@gmail.com
Subject: Éric Vyncke's No Objection on draft-ietf-rtgwg-segment-routing-ti-lfa-13: (with COMMENT)
X-Test-IDTracker: no
X-IETF-IDTracker: 12.9.0
Auto-Submitted: auto-generated
Precedence: bulk
Reply-To: Éric Vyncke <evyncke@cisco.com>
Message-ID: <171273837165.36311.9375966485125934611@ietfa.amsl.com>
Date: Wed, 10 Apr 2024 01:39:31 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/rtgwg/q-wfPPDZ8pLDgwadJSk43ytl9AM>
X-BeenThere: rtgwg@ietf.org
X-Mailman-Version: 2.1.39
List-Id: Routing Area Working Group <rtgwg.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/rtgwg>, <mailto:rtgwg-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/rtgwg/>
List-Post: <mailto:rtgwg@ietf.org>
List-Help: <mailto:rtgwg-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/rtgwg>, <mailto:rtgwg-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 10 Apr 2024 08:39:31 -0000

Éric Vyncke has entered the following ballot position for
draft-ietf-rtgwg-segment-routing-ti-lfa-13: 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-rtgwg-segment-routing-ti-lfa/



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


# Éric Vyncke, INT AD, comments for draft-ietf-rtgwg-segment-routing-ti-lfa-13

Thank you for the work put into this document. The flow appears to be logical
and the text well explained, but to be honest it is too specific and too
acronyms-heavy for me, i.e., my review is rather superficial and I am trusting
the RTG ADs for their content review. Nevertheless, I like the clarity of
section 10.

Please find below some non-blocking COMMENT points (but replies would be
appreciated even if only for my own education), and some nits.

Special thanks to Stewart Bryant for the shepherd's detailed write-up including
the WG consensus *but it lack* the justification of the intended status. I like
the `This is a deployed protocol.` ;-) OTOH, the justification for *6* authors
is rather weak: `The document has taken seven year to get to this point and
seems to have settled at this number of authors.`

I hope that this review helps to improve the document,

Regards,

-éric

# COMMENTS (non-blocking)

## Abstract

Should "IP-FRR" be expanded ?

## Section 6

This section contains multiple "SHOULD" but does not explain when the "SHOULD"
can be bypassed.

## Section 8.2

I am afraid cannot parse `Then the packet is protected as if its were a transit
packet.`

## Section 12

This is value information of course even if the actual networks are not
referenced. Beside the depth of the SID list, I would have welcome the amount
of additional repair entries required in the node (is it simply destinations *
links ?) as it could have an impact of amount of states in the routers.

# NITS (non-blocking / cosmetic)

## Section 2

Usually acronyms are introduced *after* the expansion, e.g., not as in `TLDP
sessions (Targeted Label Distribution Protocol)`

## Section 2.1

This BCP14 template should probably better placed after the acronyms.