Skip to main content

Last Call Review of draft-ietf-mpls-1stnibble-10
review-ietf-mpls-1stnibble-10-opsdir-lc-wu-2024-10-31-01

Request Review of draft-ietf-mpls-1stnibble
Requested revision No specific revision (document currently at 12)
Type Last Call Review
Team Ops Directorate (opsdir)
Deadline 2024-11-01
Requested 2024-10-18
Authors Kireeti Kompella , Stewart Bryant , Matthew Bocci , Greg Mirsky , Loa Andersson , Jie Dong
I-D last updated 2024-10-31
Completed reviews Rtgdir Early review of -02 by Joel M. Halpern (diff)
Secdir Last Call review of -11 by Daniel Migault (diff)
Opsdir Last Call review of -10 by Qin Wu (diff)
Genart Last Call review of -10 by Joel M. Halpern (diff)
Assignment Reviewer Qin Wu
State Completed
Request Last Call review on draft-ietf-mpls-1stnibble by Ops Directorate Assigned
Posted at https://mailarchive.ietf.org/arch/msg/ops-dir/lVEPsB3AnuaBP1ntT3KpB76L_Cg
Reviewed revision 10 (document currently at 12)
Result Has nits
Completed 2024-10-31
review-ietf-mpls-1stnibble-10-opsdir-lc-wu-2024-10-31-01
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. 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.

This document specifies the new IANA registry for the first Nibble Following a
Label Stack. In addition, this document provide requirements for registering
new values and recommendation for MPLS packet processing. This document is well
written and is on the right track, however, I do have a few comments for
questions and clarifications:

1. Abstract
Abstract said:
“this memo sets out some documentation requirements for registering new values.
Finally, it provides some recommendations that make processing MPLS packets
easier and more robust.”

Abstract also said:
“This document updates RFC 4928 by deprecating the heuristic method for
identifying the type of packet encapsulated in MPLS.”

I am wondering whether deprecating the heuristic method for identifying the
type of packet encapsulated in MPLS is seen as recommendation or requirements,
if yes, is such duplicated?

2. Abstract
Some place in the abstract uses the term "This memo", some place in the place
uses the term "this document", the same comment is applied to the introduction
section, suggest to make these terms consistent? Suggest to use the term "This
document".

3. Section 1
Introduction said:
“
This memo introduces a requirement and a recommendation. The first builds on
Section 2.1.1, and the second deprecates the use of the heuristic in Section
2.1.1.1 and recommends using a dedicated label value for load balancing. ”

Abstract also said:
“
this memo sets out some documentation requirements for registering new values.
Finally, it provides some recommendations that make processing MPLS packets
easier and more robust. ”

How many requirements and recommendations are specified by this document, one
or many, it looks the abstract is not consist with Introduction for this.
Secondly, where these requirements and recommendations are documented? Only
section 2.1.1 and section 2.1.1.1, or we have some other sections? My
suggestion is to have two clear sections to document requirements and
recommendations separately.

4. Section 3.1
Section 3.1 said:
“
The assignment policy for the registry is Standards Action.
“
Can we add reference for standard action, I think it should be RFC8126.