Skip to main content

Early Review of draft-ietf-idr-bgpls-sr-vtn-mt-08
review-ietf-idr-bgpls-sr-vtn-mt-08-opsdir-early-nainar-2025-04-16-00

Request Review of draft-ietf-idr-bgpls-sr-vtn-mt
Requested revision No specific revision (document currently at 14)
Type Early Review
Team Ops Directorate (opsdir)
Deadline 2025-04-04
Requested 2025-03-15
Requested by Susan Hares
Authors Chongfeng Xie , Cong Li , Jie Dong , Zhenbin Li
I-D last updated 2025-10-21 (Latest revision 2025-10-16)
Completed reviews Secdir Early review of -07 by Corey Bonnell (diff)
Rtgdir Early review of -07 by Loa Andersson (diff)
Opsdir Early review of -08 by Nagendra Kumar Nainar (diff)
Genart IETF Last Call review of -12 by Christer Holmberg (diff)
Secdir IETF Last Call review of -12 by Corey Bonnell (diff)
Comments
This is an informational draft.
Assignment Reviewer Nagendra Kumar Nainar
State Completed
Request Early review on draft-ietf-idr-bgpls-sr-vtn-mt by Ops Directorate Assigned
Posted at https://mailarchive.ietf.org/arch/msg/ops-dir/O4aJ8VKPK3iCWGtagu_QeOViw7I
Reviewed revision 08 (document currently at 14)
Result Has nits
Completed 2025-04-16
review-ietf-idr-bgpls-sr-vtn-mt-08-opsdir-early-nainar-2025-04-16-00
Hi,

I have reviewed this document as part of the Operational directorate's ongoing
effort to review all IETF documents being processed by the IESG.  These
comments were written with the intent of improving the operational aspects of
the IETF drafts per guidelines in RFC5706.

Comments that are not addressed in last call may be included
in AD reviews during the IESG review.  Document editors and WG chairs should
treat these comments just like any other last call comments.

Overall Summary:

This draft is an information track document proposing the mechanism of
leveraging the BGL-LS MT extensions to carry the NRP attributes .Overall this
is a straight forward and a well written document that appears to addresses all
potential operational aspects. I am marking it as "have nits" only to address
the below that appears to be a typo error.

More details below:

"At the AS-level topology, different inter-domain NRPs may have
      different inter-AS connectivity.  In this case, different BGP Peer
      Set SIDs need to be allocated to represent the groups of BGP peers
      which can be used for load-balancing in each inter-domain NRP.
      The association between the BGP Peer Node SIDs and the MT-ID of
      the NRP needs to be advertised by the ASBR."

==> I think the above should be "Peer Set SIDs" instead of "Peer Node SIDs"

Thanks,
Nagendra