Re: [Lsr] I-D Action: draft-ietf-lsr-ospfv3-extended-lsa-yang-20.txt

Em dom., 13 de ago. de 2023 às 18:16, Acee Lindem escreveu: > > Hi Renato, > > > > > On Aug 11, 2023, at 9:41 PM, Renato Westphal > > wrote: > > > > Em dom., 16 de jul. de 2023 às 22:37, Christian Hopps > > escreveu: > >>> On Jul 13, 2023, at 17:23, Acee Lindem wrote: > >>> > >>> Hi Renato, >

Re: [Lsr] I-D Action: draft-ietf-lsr-ospfv3-extended-lsa-yang-20.txt

Hi Renato, > On Aug 11, 2023, at 9:41 PM, Renato Westphal wrote: > > Em dom., 16 de jul. de 2023 às 22:37, Christian Hopps > escreveu: >>> On Jul 13, 2023, at 17:23, Acee Lindem wrote: >>> >>> Hi Renato, Lastly, this might just be a small nitpick of mine, but I don't think h

Re: [Lsr] I-D Action: draft-ietf-lsr-ospfv3-extended-lsa-yang-20.txt

Hi Renato, Chris, > On Aug 11, 2023, at 21:41, Renato Westphal wrote: > > Em dom., 16 de jul. de 2023 às 22:37, Christian Hopps > mailto:cho...@chopps.org>> escreveu: >>> On Jul 13, 2023, at 17:23, Acee Lindem wrote: >>> >>> Hi Renato, Lastly, this might just be a small nitpick of

Re: [Lsr] I-D Action: draft-ietf-lsr-ospfv3-extended-lsa-yang-20.txt

Em dom., 16 de jul. de 2023 às 22:37, Christian Hopps escreveu: > > On Jul 13, 2023, at 17:23, Acee Lindem wrote: > > > > Hi Renato, > >> > >> Lastly, this might just be a small nitpick of mine, but I don't think > >> having a "length" leaf for all TLVs and Sub-TLVs adds much value. In > >> my op

Re: [Lsr] I-D Action: draft-ietf-lsr-ospfv3-extended-lsa-yang-20.txt

> On Jul 13, 2023, at 17:23, Acee Lindem wrote: > > Hi Renato, > >> >> Lastly, this might just be a small nitpick of mine, but I don't think >> having a "length" leaf for all TLVs and Sub-TLVs adds much value. In >> my opinion, it's only relevant for unknown TLVs that couldn't be >> decoded;

Re: [Lsr] I-D Action: draft-ietf-lsr-ospfv3-extended-lsa-yang-20.txt

Hi Renato, Thank you for the review and comments. We really need this. About the "length", I agree with you and Acee that it's not necessary for some fixed length TLV. However, from coding and modeling perspective, I would think it's better to stay consistent. If we only need length for unknown-t

Re: [Lsr] I-D Action: draft-ietf-lsr-ospfv3-extended-lsa-yang-20.txt

Hi Renato, > On Jul 14, 2023, at 10:58, Renato Westphal wrote: > > Hi Acee, > > Thanks for accepting my suggestions. > > Em qui., 13 de jul. de 2023 às 18:23, Acee Lindem > escreveu: >>> Lastly, this might just be a small nitpick of mine, but I don't think >>> having a "length" leaf for all

Re: [Lsr] I-D Action: draft-ietf-lsr-ospfv3-extended-lsa-yang-20.txt

Hi Acee, Thanks for accepting my suggestions. Em qui., 13 de jul. de 2023 às 18:23, Acee Lindem escreveu: > > Lastly, this might just be a small nitpick of mine, but I don't think > > having a "length" leaf for all TLVs and Sub-TLVs adds much value. In > > my opinion, it's only relevant for unkn

Re: [Lsr] I-D Action: draft-ietf-lsr-ospfv3-extended-lsa-yang-20.txt

Hi Renato, > On Jul 13, 2023, at 10:32, Renato Westphal wrote: > > Hi all, > > I wanted to share some additional feedback now that I've implemented > the updated module. > > First, the "uses ospf:ospf-router-lsa-bits" statement should be > removed from the "inter-router-tlv" container since t

Re: [Lsr] I-D Action: draft-ietf-lsr-ospfv3-extended-lsa-yang-20.txt

Hi all, I wanted to share some additional feedback now that I've implemented the updated module. First, the "uses ospf:ospf-router-lsa-bits" statement should be removed from the "inter-router-tlv" container since the Inter-Area-Router TLV doesn't contain the "Router-LSA bits" field. Next, in the

[Lsr] I-D Action: draft-ietf-lsr-ospfv3-extended-lsa-yang-20.txt

A New Internet-Draft is available from the on-line Internet-Drafts directories. This Internet-Draft is a work item of the Link State Routing (LSR) WG of the IETF. Title : YANG Model for OSPFv3 Extended LSAs Authors : Acee Lindem Sharmila Palani