Last Call Review of draft-ietf-idr-sr-policy-safi-09
review-ietf-idr-sr-policy-safi-09-genart-lc-romascanu-2024-11-06-00

Request Review of draft-ietf-idr-sr-policy-safi
Requested revision No specific revision (document currently at 10)
Type Last Call Review
Team General Area Review Team (Gen-ART) (genart)
Deadline 2024-11-11
Requested 2024-10-21
Authors Stefano Previdi , Clarence Filsfils , Ketan Talaulikar , Paul Mattes , Dhanendra Jain
I-D last updated 2024-11-06
Completed reviews Secdir Early review of -04 by Vincent Roca (diff)
Rtgdir Early review of -00 by Zhaohui (Jeffrey) Zhang (diff)
Tsvart Early review of -00 by David L. Black (diff)
Opsdir Early review of -01 by Nagendra Kumar Nainar (diff)
Genart Last Call review of -09 by Dan Romascanu (diff)
Opsdir Last Call review of -10 by Nagendra Kumar Nainar
Tsvart Last Call review of -10 by Magnus Westerlund
Assignment Reviewer Dan Romascanu
State Completed
Request Last Call review on draft-ietf-idr-sr-policy-safi by General Area Review Team (Gen-ART) Assigned
Posted at https://mailarchive.ietf.org/arch/msg/gen-art/djHrwpnggjTOX1BovN-6dgAE4zs
Reviewed revision 09 (document currently at 10)
Result Ready w/issues
Completed 2024-11-06
review-ietf-idr-sr-policy-safi-09-genart-lc-romascanu-2024-11-06-00
I am the assigned Gen-ART reviewer for this draft. The General Area
Review Team (Gen-ART) reviews all IETF documents being processed
by the IESG for the IETF Chair.  Please treat these comments just
like any other last call comments.

For more information, please see the FAQ at

<https://wiki.ietf.org/en/group/gen/GenArtFAQ>.

Document: draft-ietf-idr-sr-policy-safi-09

Reviewer: Dan Romascanu
Review Date: 2024-11-06
IETF LC End Date: 2024-11-11
IESG Telechat date: Not scheduled for a telechat

Summary:

This document specifies how BGP may be used to distribute SR Policy candidate
paths. It introduces a BGP SAFI to advertise a candidate path of a SR Policy
and defines sub-TLVs for the Tunnel Encapsulation Attribute for signaling
information about these candidate paths.It updates RFC9012.

This document is Ready with two Minor Issues.

Major issues:

Minor issues:

1. Section 6.10 includes:

>  Note to IANA (RFC editor to remove this before publication): The new
   registry creation request below is also present in the draft-ietf-
   pce-segment-routing-policy-cp.  IANA is requested to process the
   registry creation via the first of these two documents to reach
   publication stage and the authors of the other document would update
   the IANA considerations suitably.

As I understand, if this document reaches publication before
draft-ietf-pce-segment-routing-policy-cp, Section 6.6 of the latest will be
removed, and the opposite. However, the tables are not identical. In Table 10,
Section 6.10 in this document values 5-255 are Unassigned, while in Section 6.6
of draft-ietf-pce-segment-routing-policy-cp values 5-255 are Reserved.

2.The Manageability Considerations section refers to
[I-D.ietf-spring-sr-policy-yang] which is Expired.

Nits/editorial comments: