Hi,
I'm afraid the draft has some serious issues that would need to be
addressed if it is to become a WG document.
Below comments use ISIS as an example, but most of it applies to OSPF as
well.
1. The draft says:
"ISIS[RFC5316] defines the Inter-AS Reachability TLV to carry the TE
information about inter-AS links. This TLV can be used to transfer
the information about the stub link which is located at the boundary
of one AS. This document defines the Stub-Link sub-TLV within this
TLV to identify the stub link and transfer the associated attributes."
- there is an existing mechanisms to advertise inter-AS link. There is
existing mechanisms to advertise external prefix. What exactly is the
reason to define a new TLV?
It's still not clear to me what exactly is the use case for the new TLVs
defined in the document.
2. Looking at the proposed ISIS Stub-link Sub-TLV, which is a sub-TLV of
the existing Inter-AS Reachability TLV:
- it advertises prefix. The advertisement of the prefix and link
information is strictly decoupled in ISIS. Here the proposal is to
advertise the prefix inside the Inter-AS Reachability TLV, which is
advertising a link. Why do we need the prefix of the inter-AS link to be
advertised inside the link advertisement?
2. The new ISIS Stub-link Sub-TLV includes sub-TLVs, the text says:
"Sub-TLVs: Existing sub-TLVs that defined within "Sub-TLVs for TLVs
22, 23, 25, 141, 222, and 223" can be included if necessary."
The parent Inter-AS Reachability TLV already has Sub-TLVs from the exact
same space. Not to mention that the new sub-TLV itself comes from the
same space. Looks broken.
3. Link Type in ISIS Stub-link Sub-TLV - what is it used for and why do
we need it?
thanks,
Peter
On 04/01/2022 07:58, Christian Hopps wrote:
Hi Folks,
This begins a 2 week WG Adoption Call for the following draft:
https://datatracker.ietf.org/doc/draft-wang-lsr-stub-link-attributes/
Please indicate your support or objections by January 18th, 2022.
Authors, please respond to the list indicating whether you are aware of any IPR
that applies to these drafts.
Thanks,
Chris.
_______________________________________________
Lsr mailing list
Lsr@ietf.org
https://www.ietf.org/mailman/listinfo/lsr
_______________________________________________
Lsr mailing list
Lsr@ietf.org
https://www.ietf.org/mailman/listinfo/lsr
