Last Call Review of draft-ietf-mpls-mna-requirements-13
review-ietf-mpls-mna-requirements-13-genart-lc-hares-2024-04-30-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-mpls-mna-requirements-??
Reviewer: Susan Hares
Review Date: 2024-04-30
IETF LC End Date: 2024-05-06
IESG Telechat date: Not scheduled for a telechat

Summary: The content of the document provides a clear set of requirements.
Based on my understanding of the MPLS WG discussion on this topic,
these requirements are an accurate discussion of the requirements
desired by the Wg.

Major issues: None

Minor issues: None

Nits/editorial comments:
3 editorial comments based on English grammar and readability of text.

#1, Section 3.4, item 28
Problem: Spelling error
Old text:/
   28.  An MNA solution SHOULD support NAIs for both Pint-to-Point (P2P)
        and Point-to-Multipoint (P2MP) paths, but a specific NAI MAY be
        limited by the network action specification to only one or the
        other of these path types if there is a clear reason to do so./

New text:/
   28.  An MNA solution SHOULD support NAIs for both Point-to-Point (P2P)
        and Point-to-Multipoint (P2MP) paths, but a specific NAI MAY be
        limited by the network action specification to only one or the
        other of these path types if there is a clear reason to do so./

#2 Section 3.4, item 33
Problem: Two grammar issues.
1) e.g. - is normally followed by a comman ("e.g.,). The sentence structure
makes that difficult or impossible. 2. The sentences would be better split into
two pieces.

Current text: /
   33.  NAIs MUST only be inserted at LSRs that push a label onto the
        stack, e.g. head end LSRs and points of local repair (PLR), but
        can be processed by LSRs along the path of the LSP.
/
New test:/
Current text: /
   33.  NAIs MUST only be inserted at LSRs that push a label onto the
        stack but can be processed by LSRs along the path of the LSP.
        Two examples of LSRs that push a label onto the stack are
        the head end LSRs and points of local repair (PLR),
/

#3, section 3.4, item 39.
Problem: Grammar and Punctuation - The "i.e." in the sentence needs a "i.e.,".
Two options are given for resolution.

Current text: /
   39.  A network action solution specification MUST state where the
        NAIs are to be placed in the MPLS packet i.e. in-stack or post-
        stack./
Resolution suggestion 1: /
   39.  A network action solution specification MUST state where the
        NAIs are to be placed in the MPLS packet - i.e., in-stack or post-
        stack./
Resolution suggestion 2: /
   39.  A network action solution specification MUST state where the
        NAIs are to be placed in the MPLS packet, that is in-stack or post-
        stack./