Re: I-D Action: draft-ietf-bfd-yang-06.txt

2017-07-31 Thread Jeffrey Haas
On Fri, Jul 28, 2017 at 03:58:06PM -0700, Mahesh Jethanandani wrote: > The changes are done and pushed to GitHub. Use the grouping client-cfg-parms. Question: For implementations that use Echo mode, is that something the protocol client configuration impacts or is it chosen by the system automatic

Re: I-D Action: draft-ietf-bfd-yang-06.txt

2017-07-31 Thread Jeffrey Haas
On Sun, Jul 30, 2017 at 05:14:28PM +, Yingzhen Qu wrote: > Please see attached ospf bfd module. Base ospf module also needs to be > updated to remove the bfd enable leaf. ISIS model need to do the same change, > ietf-isis-bfd.yang will look the same as ietf-ospf-bfd.yang. > > Please let me k

Re: A question about RFC5884

2017-07-31 Thread Jeffrey Haas
On Tue, Jul 18, 2017 at 10:25:38AM +, Carlos Pignataro (cpignata) wrote: > I had not suggested it, but I think that idea has merit. If there are enough > updates needed to the spec based on additional running-code learning, or > ambiguities that are causing interoperable confusion, the net of

Re: WGLC for BFD Multipoint documents (ending July 14, 2017)

2017-07-31 Thread Jeffrey Haas
Working Group, We've received some good feedback for Santosh to include in the next revision of the documents and likely have enough discussion to move them forward in the publication process. At this point, we are waiting for targeted review from ALU/Nokia who actually has an implementation of t

RE: I-D Action: draft-ietf-bfd-yang-06.txt

2017-07-31 Thread Yingzhen Qu
Hi Mahesh, For the enable flag, I'd prefer to have it defined directly in ospf instead of using a grouping from BFD. I'm not sure how useful this grouping is, am I missing something? Thanks, Yingzhen -Original Message- From: Mahesh Jethanandani [mailto:mjethanand...@gmail.com] Sent: S

Re: Fwd: I-D Action: draft-chen-bfd-unsolicited-01.txt

2017-07-31 Thread Jeffrey Haas
Working Group, This draft was briefly mentioned as part of the IDR WG presentation on RS-BFD that I am a co-author on. That presentation is supposed to be linked here, but is not coming up. Hopefully it's a transient error: https://datatracker.ietf.org/meeting/99/materials/slides-99-idr-03-idr-

Re: I-D Action: draft-ietf-bfd-yang-06.txt

2017-07-31 Thread Acee Lindem (acee)
Hi Mahesh, On 7/31/17, 12:42 AM, "Mahesh Jethanandani" wrote: >Yingzhen, > >Overall the model looks good to me. > >I notice that you decided to (re)define the enable flag in the model. Is >that intentional? > >You are aware that there is another grouping called client-base-cfg-parms >that defin

Re: I-D Action: draft-ietf-bfd-yang-06.txt

2017-07-31 Thread Acee Lindem (acee)
Sigh, I mean “why don’t you add ‘enabled’…" On 7/31/17, 2:56 PM, "Acee Lindem (acee)" wrote: >Hi Mahesh, > >On 7/31/17, 12:42 AM, "Mahesh Jethanandani" >wrote: > >>Yingzhen, >> >>Overall the model looks good to me. >> >>I notice that you decided to (re)define the enable flag in the model. Is >

Re: I-D Action: draft-ietf-bfd-yang-06.txt

2017-07-31 Thread Mahesh Jethanandani
Yingzhen, How do you plan to indicate to the BFD engine that the session is enabled? > On Jul 31, 2017, at 10:26 AM, Yingzhen Qu wrote: > > Hi Mahesh, > > For the enable flag, I'd prefer to have it defined directly in ospf instead > of using a grouping from BFD. I'm not sure how useful this

Re: I-D Action: draft-ietf-bfd-yang-06.txt

2017-07-31 Thread Mahesh Jethanandani
Ok. Will do. > On Jul 31, 2017, at 12:05 PM, Acee Lindem (acee) wrote: > > Sigh, I mean “why don’t you add ‘enabled’…" > > On 7/31/17, 2:56 PM, "Acee Lindem (acee)" wrote: > >> Hi Mahesh, >> >> On 7/31/17, 12:42 AM, "Mahesh Jethanandani" >> wrote: >> >>> Yingzhen, >>> >>> Overall the mod

Re: I-D Action: draft-ietf-bfd-yang-06.txt

2017-07-31 Thread Mahesh Jethanandani
Jeff, > On Jul 31, 2017, at 10:05 AM, Jeffrey Haas wrote: > > On Fri, Jul 28, 2017 at 03:58:06PM -0700, Mahesh Jethanandani wrote: >> The changes are done and pushed to GitHub. Use the grouping client-cfg-parms. > > Question: For implementations that use Echo mode, is that something the > proto

Re: I-D Action: draft-ietf-bfd-yang-06.txt

2017-07-31 Thread Jeffrey Haas
Mahesh, > On Jul 31, 2017, at 4:26 PM, Mahesh Jethanandani > wrote: > > Jeff, > >> On Jul 31, 2017, at 10:05 AM, Jeffrey Haas wrote: >> >> On Fri, Jul 28, 2017 at 03:58:06PM -0700, Mahesh Jethanandani wrote: >>> The changes are done and pushed to GitHub. Use the grouping >>> client-cfg-parm

Re: I-D Action: draft-ietf-bfd-yang-06.txt

2017-07-31 Thread Ashesh Mishra
>From what I understand, the echo intervals have an inverse relationship with >their control interval counterparts. Faster echo allows for slower control >frame rate. So they are necessarily different values. That said, having never >implemented echo mode, I can't comment how they should get gro

Re: I-D Action: draft-ietf-bfd-yang-06.txt

2017-07-31 Thread Mahesh Jethanandani
> On Jul 31, 2017, at 1:55 PM, Jeffrey Haas wrote: > > Mahesh, > >> On Jul 31, 2017, at 4:26 PM, Mahesh Jethanandani >> wrote: >> >> Jeff, >> >>> On Jul 31, 2017, at 10:05 AM, Jeffrey Haas wrote: >>> >>> On Fri, Jul 28, 2017 at 03:58:06PM -0700, Mahesh Jethanandani wrote: The changes