Re: [mpls] Alvaro Retana's No Objection on draft-ietf-mpls-lsp-ping-relay-reply-10: (with COMMENT)

["Alvaro Retana (aretana)" <[e-mail]>]

On 9/30/15, 12:30 AM, "Loa Andersson" <[e-mail]> wrote:

Hi!

Sorry that I was cryptic..

>>
>>I have some comments/questions:
>>
>>1. TBD2 is the Relay Node Address Stack TLV Type.  There seems to be some
>>confusion in the text: Section 4.2. (Receiving an Echo Request)  says
>>that the "Type of the Relay Node Address Stack TLV is chosen from the
>>range 32768 to 49161Š² giving the impression that any value can be used,
>>while 8.2. (New TLV) in the IANA Considerations says that a "suggested
>>value should be assigned² giving me the impression that the assignment is
>>just a suggestion (and somehow reinforcing the text in 4.2), but the
>>original definition in 3.2. (Relay Node Address Stack) simply says that
>>the "value should be assigned by IANA².  Assuming that you simply want an
>>assignment and that it would be what is used, please clean the text up; I
>>suggest just referring to the value as TBD2 (in 4.2 and 8.2) and
>>explicitly including the text about the assignment and the range (from
>>3.2) in 8.2.
>
>I'm not sure how to read this - it looks like you are removing the range
>information. However for LSP Ping the ranges are important. The IANA
>section should state which range the code point should come from. Here
>are the allocation policies and how TLVs from the different ranges are
>reted.

In short, the range should only matter in the IANA Considerations so that
the right assignment is made.  In the rest of the text referencing TBD2
should be enough.

Explicitly, this is what I¹m proposing:

Section 3.2:

OLD
      Type: value is TBD2.  The value should be assigned by IANA from
      32768-49161 as suggested by [RFC4379] Section 3.

NEW
      Type: value is TBD2.


Section 4.2:

OLD
   The Type of the Relay Node Address Stack TLV is chosen from the range
   32768 to 49161 so that (per section 3 of [RFC4379]) an LSR that does
   not recognize the TLV knows that the TLV is optional and can safely
   ignore it.

NEW
   The Type of the Relay Node Address Stack TLV is TBD2 so that
   (per section 3 of [RFC4379]) an LSR that does
   not recognize the TLV knows that the TLV is optional and can safely
   ignore it.


Section 8.2:

OLD
   A suggested value should be assigned from "Standards Action" range
   (32768-49161) as suggested by [RFC4379] Section 3, using the first
   free value within this range.

NEW
   A value should be assigned from "Standards Action" range
   (32768-49161) as suggested by [RFC4379] Section 3, using the first
   free value within this range.





BTW, I found a new nit:  In Section 3.3. (MTU Exceeded Return Code)
s/TBD2/TBD3

Thanks!

Alvaro.