Skip to main content

Last Call Review of draft-ietf-spring-nsh-sr-11
review-ietf-spring-nsh-sr-11-rtgdir-lc-bryant-2022-05-28-00

Request Review of draft-ietf-spring-nsh-sr-11
Requested revision 11 (document currently at 15)
Type Last Call Review
Team Routing Area Directorate (rtgdir)
Deadline 2022-05-05
Requested 2022-04-21
Requested by Andrew Alston
Authors Jim Guichard , Jeff Tantsura
I-D last updated 2022-05-28
Completed reviews Intdir Early review of -08 by Dave Thaler (diff)
Rtgdir Early review of -07 by Mike McBride (diff)
Rtgdir Last Call review of -11 by Stewart Bryant (diff)
Secdir Last Call review of -11 by Derrell Piper (diff)
Genart Last Call review of -11 by Linda Dunbar (diff)
Comments
Did my own review of the doc and had one or two things sorted from version 10, but could use a wider review.
Assignment Reviewer Stewart Bryant
State Completed
Request Last Call review on draft-ietf-spring-nsh-sr by Routing Area Directorate Assigned
Posted at https://mailarchive.ietf.org/arch/msg/rtg-dir/crA4bep14FUfUloQdDwiheyySUo
Reviewed revision 11 (document currently at 15)
Result Ready
Completed 2022-05-28
review-ietf-spring-nsh-sr-11-rtgdir-lc-bryant-2022-05-28-00
This is a well written document that is ready for publication.

There is one point that the IESG should ponder. The authors have asked for a IP
type assignment. This is a limited registry that needs to last the lifetime of
the IP protocol suite. NSH started its life 9 years ago and has been a standard
for 4 years and in all this time has not needed such as allocation. Neither
SRv6 nor NSH are petite or lightweight protocols. So I wonder if the
identification of NSH should happen at the IP layer as proposed, or whether an
intermediate multiplexing layer such as UDP should be used? The extra
processing for UDP is one test and the extra MTU is 8 octets. The decision for
the IESG is whether in their view the extent of deployment and the gain in
performance is such that they should authorise the allocation of the IP type.

One small point, I assume that the IANA allocation in Section 11.2 "SRv6
Endpoint Behaviors" should come from the FCFS series of code points.