Hi Yingzhen, Thanks for your response. Please see my comments inline.
Em sáb., 4 de abr. de 2026 às 09:31, Yingzhen Qu <[email protected]> escreveu: > Hi Renato, > > Thanks for your review, I really appreciate the feedback. Please see my > answers below inline. > > Thanks, > Yingzhen > > On Sat, Apr 4, 2026 at 6:59 AM Acee Lindem <[email protected]> wrote: > >> Hey Renato, >> >> Thanks SO MUCH for your comments - much more useful than whining about >> LSR WG process from people who haven't contributed technically. >> >> > On Apr 1, 2026, at 5:53 PM, Renato Westphal <[email protected]> >> wrote: >> > >> > Hi all, >> > >> > I have a couple of questions regarding the ietf-isis-link-attr module: >> > >> > 1 - In the "asla-sub-tlv" list, is there a reason why nodes are not >> defined for all currently specified sub-sub-TLVs (such as TE metric, >> unidirectional delay, and others)? In the base module, both the >> "extended-is-neighbor" and "mt-is-neighbor" lists include nodes for these >> TE sub-TLVs. It's not clear why the ASLA Sub-TLV would be treated >> differently. >> >> I agree we should include every ASLA attribute that is in the base RFC >> 9130 model (and RFC 9129 for OSPF). > > > [Yingzhen]: I on purpose didn't list any specific sub-tlv here, but used > unknown-tlvs instead. My thought was that there would be new sub-tlvs > defined in the future, and for all these TLVs in the LSDB it's fine to > return raw data. This way we won't need new augmentations. A script or any > automation tool should easily parse these TLVs. > [Renato] Fair enough. I think, however, it would be more useful to operators if we modeled well-known ASLA attributes explicitly. When new attributes are defined in the future, both this module and the base IS-IS module can be augmented accordingly. > > >> > 2 - In the "interface-asla" list, could you clarify the meaning of the >> "metric-type" (uint8) and "metric" (uint32) configuration leafs? The >> current descriptions seem too vague. >> >> Yingzhen and I will discuss. This configuration should include ASLA >> attributes. > > > [Yingzhen]: The "metric-type" is the IANA IGP Metric-type: > https://www.iana.org/assignments/igp-parameters/igp-parameters.xhtml#igp-metric-type. > We can update the description. Is this what you meant? or you think we're > missing some configurations? > [Renato] I'm confused about the purpose of "metric-type" here. Isn't the metric type a concept specific to Flex-Algo definitions? I would expect this list to merely contain nodes to set or override link attributes (like TE metrics) per application. Best regards, Renato > >> >> Thanks, >> Acee >> >> >> >> > >> > Best regards, >> > Renato >> > >> > >> > Em ter., 31 de mar. de 2026 às 22:33, Acee Lindem < >> [email protected]> escreveu: >> > This version addresses Adrian Farrel's OPS Directorate comments. >> > >> > Thanks, >> > Acee >> > >> > > On Mar 31, 2026, at 1:56 PM, [email protected] wrote: >> > > >> > > Internet-Draft draft-ietf-lsr-isis-flex-algo-yang-06.txt is now >> available. It >> > > is a work item of the Link State Routing (LSR) WG of the IETF. >> > > >> > > Title: YANG Data Model for IS-IS Application-Specific Link >> Attributes and Flexible Algorithm >> > > Authors: Yingzhen Qu >> > > Acee Lindem >> > > Madhavi Joshi >> > > Name: draft-ietf-lsr-isis-flex-algo-yang-06.txt >> > > Pages: 31 >> > > Dates: 2026-03-31 >> > > >> > > Abstract: >> > > >> > > This document defines a YANG data model to support IS-IS >> Application- >> > > Specific Link Attributes and Flexible Algorithm. >> > > >> > > The IETF datatracker status page for this Internet-Draft is: >> > > https://datatracker.ietf.org/doc/draft-ietf-lsr-isis-flex-algo-yang/ >> > > >> > > There is also an HTML version available at: >> > > >> https://www.ietf.org/archive/id/draft-ietf-lsr-isis-flex-algo-yang-06.html >> > > >> > > A diff from the previous version is available at: >> > > >> https://author-tools.ietf.org/iddiff?url2=draft-ietf-lsr-isis-flex-algo-yang-06 >> > > >> > > Internet-Drafts are also available by rsync at: >> > > rsync.ietf.org::internet-drafts >> > > >> > > >> > > _______________________________________________ >> > > Lsr mailing list -- [email protected] >> > > To unsubscribe send an email to [email protected] >> > >> > _______________________________________________ >> > Lsr mailing list -- [email protected] >> > To unsubscribe send an email to [email protected] >> > >> > >> > -- >> > Renato Westphal >> > _______________________________________________ >> > Lsr mailing list -- [email protected] >> > To unsubscribe send an email to [email protected] >> >> _______________________________________________ >> Lsr mailing list -- [email protected] >> To unsubscribe send an email to [email protected] >> > -- Renato Westphal
_______________________________________________ Lsr mailing list -- [email protected] To unsubscribe send an email to [email protected]
