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

2017-08-14 Thread Reshad Rahman (rrahman)
Jeff, I agree. Regards, Reshad. On 2017-08-14, 1:10 PM, "Jeffrey Haas" wrote: > >> On Aug 14, 2017, at 11:24 AM, Reshad Rahman (rrahman) >> wrote: >> >> I am gradually catching up to emails so I may not have absorbed all the >> emails I have gone through yetŠ. >> >> Regarding echo config,

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

2017-08-14 Thread Jeffrey Haas
> On Aug 14, 2017, at 11:24 AM, Reshad Rahman (rrahman) > wrote: > > I am gradually catching up to emails so I may not have absorbed all the > emails I have gone through yetŠ. > > Regarding echo config, we agreed in Chicago to remove the echo config > based on the fact that implementations of

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

2017-08-14 Thread Reshad Rahman (rrahman)
I am gradually catching up to emails so I may not have absorbed all the emails I have gone through yetŠ. Regarding echo config, we agreed in Chicago to remove the echo config based on the fact that implementations of echo are vendor specific. e.g. An implementation which has echo as continuous wou

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

2017-08-07 Thread Mahesh Jethanandani
> On Aug 7, 2017, at 9:47 AM, Jeffrey Haas wrote: > > On Tue, Aug 01, 2017 at 05:52:01PM -0700, Mahesh Jethanandani wrote: >> I will be uploading the changes shortly. > > Are we at the point where we should look at the model contents again? The BFD model is uploaded and should be evaluated to

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

2017-08-07 Thread Jeffrey Haas
On Tue, Aug 01, 2017 at 05:52:01PM -0700, Mahesh Jethanandani wrote: > I will be uploading the changes shortly. Are we at the point where we should look at the model contents again? -- Jeff

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

2017-08-01 Thread Mahesh Jethanandani
> On Aug 1, 2017, at 9:33 AM, Jeffrey Haas wrote: > > On Tue, Aug 01, 2017 at 08:33:38AM -0700, Mahesh Jethanandani wrote: >>> I'm ambivalent. This depends really on real world behavior. >>> >>> As we saw from some brief googling yesterday on Cisco IOS/IOS-XR docs, that >>> implementation does

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

2017-08-01 Thread Jeffrey Haas
On Tue, Aug 01, 2017 at 08:33:38AM -0700, Mahesh Jethanandani wrote: > > I'm ambivalent. This depends really on real world behavior. > > > > As we saw from some brief googling yesterday on Cisco IOS/IOS-XR docs, that > > implementation doesn't appear to expose echo intervals as a separably > > co

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

2017-08-01 Thread Mahesh Jethanandani
> On Aug 1, 2017, at 7:41 AM, Jeffrey Haas wrote: > > Mahesh, > > On Mon, Jul 31, 2017 at 05:17:34PM -0700, Mahesh Jethanandani wrote: >>> My point, unless my very quick glance at the module mislead me, is that you >>> can't configure to use echo mode - if it's supported - in the grouping >>>

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

2017-08-01 Thread Jeffrey Haas
Mahesh, On Mon, Jul 31, 2017 at 05:17:34PM -0700, Mahesh Jethanandani wrote: > > My point, unless my very quick glance at the module mislead me, is that you > > can't configure to use echo mode - if it's supported - in the grouping > > imported by the IGPs. > > In the current model, it is model

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

2017-08-01 Thread Jeffrey Haas
Ashesh, On Mon, Jul 31, 2017 at 09:39:57PM +, Ashesh Mishra wrote: > 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, ha

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

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 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 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 Mahesh Jethanandani
d 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 know your commetns. >>>> >>>> Thanks, >>>> Yingzhen >>>> >>>>

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

2017-07-31 Thread Mahesh Jethanandani
acee) ; Reshad Rahman ; > Jeffrey Haas ; rtg-bfd@ietf.org; > draft-ietf-bfd-y...@ietf.org; draft-ietf-ospf-y...@ietf.org > Subject: Re: I-D Action: draft-ietf-bfd-yang-06.txt > > Yingzhen, > > Overall the model looks good to me. > > I notice that you decided to (re)define t

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

2017-07-31 Thread Acee Lindem (acee)
-Original Message- >>> From: Mahesh Jethanandani [mailto:mjethanand...@gmail.com] >>> Sent: Friday, July 28, 2017 2:25 PM >>> To: Acee Lindem (acee) >>> Cc: Reshad Rahman ; Yingzhen Qu >>>; Jeffrey Haas ; >>>rtg-bfd@ietf.org; draft-ietf-bfd-y...@

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

2017-07-31 Thread Acee Lindem (acee)
; >> -Original Message- >> From: Mahesh Jethanandani [mailto:mjethanand...@gmail.com] >> Sent: Friday, July 28, 2017 2:25 PM >> To: Acee Lindem (acee) >> Cc: Reshad Rahman ; Yingzhen Qu >>; Jeffrey Haas ; >>rtg-bfd@ietf.org; draft-ietf-bfd-y...@ie

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

2017-07-31 Thread Yingzhen Qu
com] Sent: Sunday, July 30, 2017 9:43 PM To: Yingzhen Qu Cc: Acee Lindem (acee) ; Reshad Rahman ; Jeffrey Haas ; rtg-bfd@ietf.org; draft-ietf-bfd-y...@ietf.org; draft-ietf-ospf-y...@ietf.org Subject: Re: I-D Action: draft-ietf-bfd-yang-06.txt Yingzhen, Overall the model looks good to me. I n

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: 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-30 Thread Mahesh Jethanandani
rds, >>>>>>> Reshad. >>>>>>> >>>>>>> >>>>>>> >>>>>>> On 2017-07-27, 3:07 PM, "Acee Lindem (acee)" wrote: >>>>>>> >>>>>>>> Hi Reshad,

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

2017-07-30 Thread Yingzhen Qu
to me since the >>>>>>> grouping bfd-client-ext-cfg-parms uses the grouping >>>>>>> bfd-grouping-base-cfg-parms which only contains the enabled >>>>>>> leaf. I believe you meant to use bfd-grouping-common-cfg-parms >>>>

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

2017-07-28 Thread Mahesh Jethanandani
gt;>> >>>>>>>> The bfd-client-ext-cfg-parms grouping should use >>>>>>>> bfd-types:bfd-grouping-common-cfg-parms rather than >>>>>>>> bfd-types:bfd-client-base-cfg-parms - no? This would be more obvious >>>>>&

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

2017-07-28 Thread Mahesh Jethanandani
e >>>>>>> bfd-types:bfd-grouping-common-cfg-parms rather than >>>>>>> bfd-types:bfd-client-base-cfg-parms - no? This would be more obvious >>>>>>> w/o >>>>>>> the client module. >>>>>>> >>

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

2017-07-28 Thread Reshad Rahman (rrahman)
t;>> >>>>>>> Regards, >>>>>>> Reshad. >>>>>>> >>>>>>> >>>>>>> >>>>>>> On 2017-07-27, 3:07 PM, "Acee Lindem (acee)" >>>>>>>

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

2017-07-28 Thread Mahesh Jethanandani
t;> indirection. In fact, it looks wrong to me since the grouping >>>>>>> bfd-client-ext-cfg-parms uses the grouping >>>>>>> bfd-grouping-base-cfg-parms >>>>>>> which only contains the enabled leaf. I believe you meant to use >>

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

2017-07-28 Thread Yingzhen Qu
draft-ietf-bfd-y...@ietf.org; draft-ietf-ospf-y...@ietf.org Subject: Re: I-D Action: draft-ietf-bfd-yang-06.txt Thanks Yingzhen. FYI I will be on holiday for the next 2 weeks. Acee, can you please get in touch with ISIS YANG authors? Regards, Reshad. On 2017-07-28, 12:25 PM, "Yingzhen Qu&quo

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

2017-07-28 Thread Reshad Rahman (rrahman)
t;bfd-types:bfd-grouping-common-cfg-parms rather than >>>>>>>bfd-types:bfd-client-base-cfg-parms - no? This would be more obvious >>>>>>>w/o >>>>>>>the client module. >>>>>>> >>>>>&

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

2017-07-28 Thread Yingzhen Qu
rg Cc: draft-ietf-bfd-y...@ietf.org; draft-ietf-ospf-y...@ietf.org Subject: Re: I-D Action: draft-ietf-bfd-yang-06.txt Thanks much Reshad - Yingzhen will be adding but the ietf-ospf-bfd module to the OSPF model and draft. Acee On 7/28/17, 11:08 AM, "Reshad Rahman (rrahman)" wrote: &

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

2017-07-28 Thread Acee Lindem (acee)
;>>Thanks, >>>>>>Acee >>>>>> >>>>>> >>>>>>> >>>>>>>Regards, >>>>>>>Reshad. >>>>>>> >>>>>>> >>>>>>> >>>>&

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

2017-07-28 Thread Reshad Rahman (rrahman)
odel for clients? Why can’t they just use >>>>>>>ietf-bfd-types.yang? I’d like to avoid the unnecessary levels of >>>>>>>indirection. In fact, it looks wrong to me since the grouping >>>>>>>bfd-client-ext-cfg-parms uses the grouping >>&g

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

2017-07-27 Thread Acee Lindem (acee)
e >>>>>>bfd-grouping-common-cfg-parms in the other new model. However, I >>>>>>don’t >>>>>>see >>>>>>any reason why client shouldn’t use this directly. >>>>>>Thanks, >>>>>>Acee >>>>

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

2017-07-27 Thread Reshad Rahman (rrahman)
>see >>>>>any reason why client shouldn’t use this directly. >>>>>Thanks, >>>>>Acee >>>>> >>>>>On 7/25/17, 2:33 PM, "Reshad Rahman (rrahman)" >>>>>wrote: >>>>> >>>>

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

2017-07-27 Thread Acee Lindem (acee)
grouping is available @ >>>>>https://github.com/jhaas-pfrc/ietf-bfd-yang/blob/master/src/yang/ietf- >>>>>b >>>>>f >>>>>d >>>>>- >>>>>c >>>>>lients.yang >>>>> >>

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

2017-07-27 Thread Reshad Rahman (rrahman)
egards, >>>>Reshad. >>>> >>>>On 2017-07-21, 12:22 PM, "Yingzhen Qu" wrote: >>>> >>>>>Hi Reshad, >>>>> >>>>>Thanks for the summary. >>>>> >>>>>Both ospf and isis model

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

2017-07-27 Thread Acee Lindem (acee)
; >>>>Thanks for the summary. >>>> >>>>Both ospf and isis models will make corresponding changes when the new >>>>BFD grouping is available. >>>> >>>>Thanks, >>>>Yingzhen >>>> >>>>-Origi

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

2017-07-27 Thread Reshad Rahman (rrahman)
t;Thanks, >>>Yingzhen >>> >>>-----Original Message----- >>>From: Reshad Rahman (rrahman) [mailto:rrah...@cisco.com] >>>Sent: Thursday, July 20, 2017 7:19 AM >>>To: Jeffrey Haas ; rtg-bfd@ietf.org >>>Cc: draft-ietf-bfd-y...@ietf.org; draft

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

2017-07-27 Thread Acee Lindem (acee)
isco.com] >>Sent: Thursday, July 20, 2017 7:19 AM >>To: Jeffrey Haas ; rtg-bfd@ietf.org >>Cc: draft-ietf-bfd-y...@ietf.org; draft-ietf-ospf-y...@ietf.org >>Subject: Re: I-D Action: draft-ietf-bfd-yang-06.txt >> >>We (BFD and OSPF YANG authors) had a discussion y

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

2017-07-25 Thread Yingzhen Qu
-ietf-bfd-y...@ietf.org; draft-ietf-ospf-y...@ietf.org Subject: Re: I-D Action: draft-ietf-bfd-yang-06.txt Hi Yingzhen, The grouping is available @ https://github.com/jhaas-pfrc/ietf-bfd-yang/blob/master/src/yang/ietf-bfd-c lients.yang If you¹d like changes to the grouping, send me an email. Re

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

2017-07-25 Thread Reshad Rahman (rrahman)
;To: Jeffrey Haas ; rtg-bfd@ietf.org >Cc: draft-ietf-bfd-y...@ietf.org; draft-ietf-ospf-y...@ietf.org >Subject: Re: I-D Action: draft-ietf-bfd-yang-06.txt > >We (BFD and OSPF YANG authors) had a discussion yesterday. > >The agreement is that since IGP peers are auto-discovered, we

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

2017-07-21 Thread Yingzhen Qu
; rtg-bfd@ietf.org Cc: draft-ietf-bfd-y...@ietf.org; draft-ietf-ospf-y...@ietf.org Subject: Re: I-D Action: draft-ietf-bfd-yang-06.txt We (BFD and OSPF YANG authors) had a discussion yesterday. The agreement is that since IGP peers are auto-discovered, we want to add back the basic BFD config

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

2017-07-20 Thread Reshad Rahman (rrahman)
We (BFD and OSPF YANG authors) had a discussion yesterday. The agreement is that since IGP peers are auto-discovered, we want to add back the basic BFD config (multiplier + intervals) in IGP via a grouping. BFD will provide that grouping in a specific YANG module. IGP BFD YANG will be in a separat

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

2017-07-05 Thread Jeffrey Haas
Thanks authors for the edits on the BFD yang module. This gets us a significant step closer to alignment with the rest of IETF for network instancing. I'd like to encourage the working group to provide feedback on this issue and also the changes in the module. As noted in another thread, we stil