Skip to main content

Last Call Review of draft-ietf-mpls-spring-inter-domain-oam-13

Request Review of draft-ietf-mpls-spring-inter-domain-oam
Requested revision No specific revision (document currently at 14)
Type Last Call Review
Team Routing Area Directorate (rtgdir)
Deadline 2024-05-17
Requested 2024-05-03
Requested by Jim Guichard
Authors Shraddha Hegde , Kapil Arora , Mukul Srivastava , Samson Ninan , Nagendra Kumar Nainar
I-D last updated 2024-05-10
Completed reviews Secdir Last Call review of -14 by Chris M. Lonvick
Rtgdir Last Call review of -13 by Stig Venaas (diff)
Opsdir Last Call review of -14 by Qin Wu
Tsvart Last Call review of -14 by Michael Tüxen
Rtgdir Early review of -05 by Michael Richardson (diff)
Assignment Reviewer Stig Venaas
State Completed
Request Last Call review on draft-ietf-mpls-spring-inter-domain-oam by Routing Area Directorate Assigned
Posted at
Reviewed revision 13 (document currently at 14)
Result Has nits
Completed 2024-05-10
I have reviewed the document and in general it is quite well written and easy
to read. But I found some minor issues.

In 4.1 where it describes all the fields, it generally specifies how many
octets each field is, but for length it just says "Length is 8 octets". It
would be good to state that the length field is 2 octets, and also format it as
"Length: 2 octets ...." similar to how it is written for the other fields. Same
for 4.2 and 4.3.

In 4.1 it says "The S bit SHOULD be zero upon transmission". Why is it not a
MUST? When might one not set it to zero?

Apart from this I just found some minor editorial things. I'm listing them
below, although I guess the RFC Editor will catch these.

"A MPLS" should be "An MPLS"

Section 2:
"seggregated" should be "segregated"

Section 4:
"various segments types", should be "various segment types"

Section 4.2:

Section 4.3:

Section 6.3
"IPv4 /IPv6" and "[RFC7110].According"

Section 7:
"Identifiers(SIDs)" Should have space

Section 7.1:
"head end" should probably be "head-end"

Section 7.2.2:
. similarly

Section 8.1

Section 8.2
. echo