Re: [mpls] Fw: Re: Requested review of draft-xp-mpls-spring-lsp-ping-path-sid

Many thanks, Xiao and Loa! [fixed the subject line with the full I-D
filename]

For the only pending comment, on the range of sub-TLVs and respective IANA
action, I would suggest:

*OLD:*

6.
<https://datatracker.ietf.org/doc/html/draft-xp-mpls-spring-lsp-ping-path-sid-09#section-6>IANA
Considerations

IANA is requested to assign three new sub-TLVs from the "sub-TLVs for TLV
Types 1, 16, and 21" subregistry of the "Multi-Protocol Label Switching
(MPLS) Label Switched Paths (LSPs) Ping Parameters" registry.

*NEW:*

6.
<https://datatracker.ietf.org/doc/html/draft-xp-mpls-spring-lsp-ping-path-sid-09#section-6>IANA
Considerations

IANA is requested to assign three new sub-TLVs from the *Standards Action
range of the* "sub-TLVs for TLV Types 1, 16, and 21" subregistry of the
"Multi-Protocol Label Switching (MPLS) Label Switched Paths (LSPs) Ping
Parameters" registry. *[RFC 9041]*
Add Normative Reference to *[RFC 9041]*.

Thanks!

Carlos.

On Fri, Jan 26, 2024 at 3:37 AM <xiao.min2@zte.com.cn> wrote:

> Hi Loa,
>
>
> Thank you for the prompt reply.
>
> A new -09 revision has been posted, which I believe addresses all your
> comments within the groups 1&3 . Link as below.
>
>
> https://datatracker.ietf.org/doc/html/draft-xp-mpls-spring-lsp-ping-path-sid-09
>
> Please see inline.
> Original
> *From: *loa@pi.nu <loa@pi.nu>
> *To: *肖敏10093570;
> *Cc: *loa.pi.nu@gmail.com <loa.pi.nu@gmail.com>;mpls@ietf.org <
> mpls@ietf.org>;
> *Date: *2024年01月26日 14:55
> *Subject: **Re: [mpls] Fw: Re: Requested review of draft-xp-mpls*
> Xiao,
>
> Thanks for quick responses. I'm comfortable with most of it - everything
> that that I think is a done deal. Just a few comments.
>
>
>
> > 1.1 Abstract
> > ------------
>
>
> > 1.4 Section 4.
> > --------------
> >
> > In the first paragraph you say:
> >
> > "The MPLS LSP Ping procedures MAY be initiated by the headend..."
> >
> > I don't doubt that it can be started from the headend, but is the
> > requirement language really needed in this draft??
> >  [XM]>>> How about removing the first sentence? Or if you have any
> > suggested text change, I'm glad to apply.
>
> I looked at removing the first sentence, it works, but cut out a useful
> reference. I would, at least for now, use a de-capitalized may.
>
> "The MPLS LSP Ping procedures may be initiated by the headend..."
>
> Note: As per an earlier mail I have updated my comments, so it makes sense.
>
> [XM-2]>>> The text change as you suggested has been incorporated in -09
> revision.
>
>
> > 2. To be fixed before WGLC
> >
>
> > 2.2 Introduction
> > ------------
> > Do we have a good reference to Target Forwarding Equivalence Class?
> >  [XM]>>> Is https://datatracker.ietf.org/doc/html/rfc8029#section-3.2
>  that
> > you're looking for?
>
> Yes - the reference is fine. Re-checking I see what partly confused me
>
> [XM-2]>>> OK.
> >
> >
> > 2.x IANA Considerations
> >
> > The registry that you are requesting  your three new Sub-TLVs should be
> > assigned from has 8 different ranges, you need to pick one per sub-TLV
> > and indicate that as the range you want it to be assigned from.
> >  [XM]>>> OK. Will check the details later to make the registry accurate..
>
> The key is that you need to make two decisions
>
> - the Registration Procedure, I tend to think "Standards Action"
> - and what a node that does not recognize the sub-TLV will, I then
>   to think "send error message"
>
> [XM-2]>>> Thanks for the detailed suggestions. The two decisions would be
> made after this draft is adopted.
>
>
> And I would strongly suggest that you treat a 3 new sub-TLVs equally.
>
> [XM-2]>>> I fully agree.
>
>
> Cheers,
>
> Xiao Min
>
>
> /Loa
>
> >
> >
>
> >
> >
>
> >
> >
> > /Loa
> >  Thanks again for your time.
> >
> > Best Regards,
> > Xiao Min_______________________________________________
> > mpls mailing list
> > mpls@ietf.org
> > https://www.ietf.org/mailman/listinfo/mpls
> >
>
>
> _______________________________________________
> mpls mailing list
> mpls@ietf.org
> https://www.ietf.org/mailman/listinfo/mpls
>