Re: [spring] WGLC for https://datatracker.ietf.org/doc/draft-ietf-spring-mpls-path-segment/

Hi,

I support the WGLC for draft-ietf-spring-mpls-path-segment. Please find my
“late” comments. Hope they are still useful.
<https://notes.ietf.org/#Minor>Minor

   - Section 2
      - The below text requires some more explanation

   The Path Segment may be used to identify an SR-MPLS Policy, its
   Candidate-Path (CP), or a SID List (SL)

   - The document says Path Segment “uniquely identify an SR path”, so how
      does that apply to SR policy/CP (which can have multiple paths)
needs to be
      explicit.
   - Section 2
      - This text

   Normally, the intermediate nodes will not see the Path Segment label
   and do not know how to process it. A Path Segment presenting to an
   intermediate node is an error condition.

   - Path segment is just another label with no special marking, and since
      it is from the egress label space it is possible that the same label is
      used by the transit node for some other purpose. Not sure about “do not
      know how to process it”, is there a better way to frame this?
   - Section 8
      - This needs to be expanded. The path segment does carry a label in
      the packet that can be used to identify the flows that are using a
      particular path. The path segment in the control plane could also be
      exploited for incorrect correlation between paths.

<https://notes.ietf.org/#Nits>Nits

   - Add references in the terminology section 1.2
   - Add references for SRLB/SRGB [RFC8402]
   - Section 2
      - s/SR-MPLS Policy/SR Policy/
   - Section 3
      - s/Path Computation Element Protocol (PCEP)/Path Computation Element
      Communication Protocol (PCEP)/
   - Section 5
      - s/[I-D.gandhi-mpls-ioam-sr]/[I-D.gandhi-mpls-ioam]
      - s/[I-D.gandhi-spring-stamp-srpm]/[I-D.ietf-spring-stamp-srpm]

Thanks!
Dhruv

On Wed, Jul 7, 2021 at 9:19 PM James Guichard <
james.n.guichard@futurewei.com> wrote:

> Dear WG:
>
>
>
> This email starts a 2 week Working Group Last Call for
> draft-ietf-spring-mpls-path-segment [1].
>
>
>
> Please read this document if you haven’t read the most recent version and
> send your comments to the SPRING WG list no later than July 21st 2021.
>
>
>
> If you are raising a point which you expect will be specifically debated
> on the mailing list, consider using a specific email/thread for this point.
>
>
>
> Lastly, if you are an author or contributor please response to indicate
> whether you know of any undisclosed IPR related to this document.
>
>
>
> Thanks!
>
>
>
> Jim, Joel & Bruno
>
>
>
> [1] https://datatracker.ietf.org/doc/draft-ietf-spring-mpls-path-segment/
>
>
>
>
>
>
>
>
> _______________________________________________
> spring mailing list
> spring@ietf.org
> https://www.ietf.org/mailman/listinfo/spring
>