Re: [OSPF] Alvaro Retana's No Objection on draft-ietf-ospf-link-overload-13: (with COMMENT)

["Acee Lindem (acee)" <[e-mail]>]

P.S. BGP-LS Link Attributes should not be confused with BGP attributes…
Thanks
Acee s

From: Acee Lindem <[e-mail]>
Date: Thursday, February 1, 2018 at 5:42 PM
To: Alvaro Retana <[e-mail]>, Shraddha Hegde <[e-mail]>, The IESG <[e-mail]>
Cc: "[e-mail]" <[e-mail]>, "[e-mail]" <[e-mail]>, OSPF WG List <[e-mail]>
Subject: Re: Alvaro Retana's No Objection on draft-ietf-ospf-link-overload-13: (with COMMENT)

Hi Alvaro,

From: Alvaro Retana <[e-mail]>
Date: Thursday, February 1, 2018 at 5:17 PM
To: Shraddha Hegde <[e-mail]>, The IESG <[e-mail]>
Cc: Acee Lindem <[e-mail]>, "[e-mail]" <[e-mail]>, "[e-mail]" <[e-mail]>, OSPF WG List <[e-mail]>
Subject: RE: Alvaro Retana's No Objection on draft-ietf-ospf-link-overload-13: (with COMMENT)

On January 30, 2018 at 11:43:53 PM, Shraddha Hegde ([e-mail]<mailto:[e-mail]>) wrote:

...
(3) Section 4.5. mentions that a "new TLV called Graceful-Link-Shutdown is defined" for BGP-LS, but there are no details on the format, etc. The IANA Considerations section suggests a value, not for a TLV but for an NLRI Type!
<Shraddha> OK. Refered section 3.1 of RFC 7752 and described the contents of the TLV
IANA section seems ok to me. Could you be more specific what needs to change?


BGP-LS Link NLRI Registry [RFC7752] >>>>>>>Registry

i)Graceful-Link-Shutdown TLV - Suggested 1101 >>>>>>>TLV type

Maybe it’s just me and I just don’t understand…which is completely possible.  There are two points:

(1)

It looks like you’re defining a new Graceful-Link-Shutdown TLV for BGP-LS.  This TLV (based on the updated description) has no information in it.  How does the receiver know which link the sender is referring to?

It is a  BGP-LS link attribute so the link is identified in the link identifiers in the corresponding NLRI. This wasn’t apparent until the IANA description was fixed.

Thanks,

Acee






Note that for the OSPF graceful-link-shutdown sub-TLVs, you are indicating where to carry them so that there is an obvious indication of which link is being shutdown.  I would like to see explicitly specified how the receiver associates this TLV with the appropriate link.  Again, I may be missing the details.



(2)

The value for the TLV was reserved by IANA in the "BGP-LS NLRI-Types" registry, not in the "BGP-LS Node Descriptor, Link Descriptor, Prefix Descriptor, and Attribute TLVs” register, which is where I would have assumed a modifier to the link would reside.  IOW, according to the registry you are defining a new NLRI Type, not a new TLV — and, according to the updated description in the document there’s no information in this NLRI.

<Shraddha> The TLV code point registration should be in “BGP-LS Node Descriptor, Link Descriptor, Prefix Descriptor, and Attribute TLVs” I have corrected this in the document.

Will e-mail to IANA for correction as well.

Does that answer your concerns?

That addresses the concern #2 above.  I still don’t see anywhere how the receiver associates this (empty) TLV with the right link.

Alvaro.