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

[Loa Andersson <[e-mail]>]

George,

I don't think that this really addresses Alvaro's point, waiting for
Alvaro to acknowledge my point, after that I was going to suggest.

NEW:

   The Type of the Relay Node Address Stack TLV will be allocated by
   IANA from the range Range defined in [RFC4379] as "optional TLVs
   that can be silently dropped if not recognized².  An LSR that does
   not recognize the TLV SHOULD ignore it.

I think that does addresses the concern that one can use any number
in the range. We also need the range pointed out explicitly in the
IANA section.

/Loa

On 2015-09-30 21:41, George Swallow (swallow) wrote:
> Loa, Lizhong -
>
> I suggest:
>
> 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 chosen from the range
>     Range defined in [RFC4379] as "optional TLVs that can be silently
>     dropped if not recognized².  An LSR that does
>     not recognize the TLV SHOULD ignore it.
>
>
> George
>
>
>
> On 9/30/15, 1:30 AM, "Loa Andersson" <[e-mail]> wrote:
>
>> Alvaro,
>>
>> Pleaese see your first comment.
>>
>> On 2015-09-30 11:11, Alvaro Retana wrote:
>>> Alvaro Retana has entered the following ballot position for
>>> draft-ietf-mpls-lsp-ping-relay-reply-10: No Objection
>>>
>>> When responding, please keep the subject line intact and reply to all
>>> email addresses included in the To and CC lines. (Feel free to cut this
>>> introductory paragraph, however.)
>>>
>>>
>>> Please refer to
>>> https://www.ietf.org/iesg/statement/discuss-criteria.html
>>> for more information about IESG DISCUSS and COMMENT positions.
>>>
>>>
>>> The document, along with other ballot positions, can be found here:
>>> https://datatracker.ietf.org/doc/draft-ietf-mpls-lsp-ping-relay-reply/
>>>
>>>
>>>
>>> ----------------------------------------------------------------------
>>> COMMENT:
>>> ----------------------------------------------------------------------
>>>
>>> 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.
>>
>> 0-16383         Standards Action	
>>                  This range is for mandatory TLVs or for optional TLVs
>>                  that require an error message if not recognized.
>>
>> 16384-31743	Specification Required	
>>                  Experimental RFC needed
>>
>> 32768-49161	Standards Action	
>>                  This range is for optional TLVs that can be silently
>>                  dropped if not recognized.
>>
>> 49162-64511	Specification Required	Experimental RFC needed
>>
>> So asking for a TLV from the 32768-49161 range will give you an optional
>> TLV that can be silently dropped if not recognized.
>>
>> /Loa
>>
>>
>>>
>>> 2. Section 4.1. (Sending an Echo Request) says that the "Relay Node
>>> Address Stack TLV MUST be carried in the Echo Request message if the
>>> relay functionality is required².  How does the initiator know that it
>>> needs the functionality?
>>>
>>> 3. Section 4.2. (Receiving an Echo Request) "A second or more address
>>> entries MAY also be added if necessary, depending on implementation.²
>>> Isn¹t this document defining how the implementation should work?  What
>>> are the cases where these additional entries may be added?
>>>
>>>
>