Hi Luc and Sasha, Thanks both for the reply. Will wait for the new version. Regards, Tim
On Sun, 3 Mar 2019, 15:39 Alexander Vainshtein, < alexander.vainsht...@ecitele.com> wrote: > Luc hi, > > Lots of thanks for a prompt response. > > > > Will be waiting for the new version of the draft. > > > > Regards, > > Sasha > > > > Office: +972-39266302 > > Cell: +972-549266302 > > Email: alexander.vainsht...@ecitele.com > > > > *From:* Luc Andre Burdet (lburdet) <lbur...@cisco.com> > *Sent:* Sunday, March 3, 2019 3:02 PM > *To:* Alexander Vainshtein <alexander.vainsht...@ecitele.com>; Yu > Tianpeng <yutianpeng.i...@gmail.com>; > draft-ietf-bess-evpn-yang.auth...@ietf.org > *Cc:* bess@ietf.org > *Subject:* Re: [bess] Review on draft-ietf-bess-evpn-yang-06 > > > > Hi Sasha, Tim, > > > > Apologies for not sending a response to your prior email; Your comments > were noted and intend to update the draft this week. > > > > Thanks, > > Luc André > > > > > > [image: > http://www.cisco.com/c/dam/m/en_us/signaturetool/images/banners/standard/09_standard_graphic.png] > > *Luc André Burdet* > > lbur...@cisco.com > > Tel: *+1 613 254 4814* > > Cisco Systems Canada Co. / Les Systemes Cisco Canada CIE > > Cisco.com <http://www.cisco.com/web/CA/> > > > > > > *From: *BESS <bess-boun...@ietf.org> on behalf of Alexander Vainshtein < > alexander.vainsht...@ecitele.com> > *Date: *Sunday, March 3, 2019 at 05:46 > *To: *Yu Tianpeng <yutianpeng.i...@gmail.com>, " > draft-ietf-bess-evpn-yang.auth...@ietf.org" < > draft-ietf-bess-evpn-yang.auth...@ietf.org> > *Cc: *"bess@ietf.org" <bess@ietf.org> > *Subject: *Re: [bess] Review on draft-ietf-bess-evpn-yang-06 > > > > Dear all, > > I concur with Tim’s comment regarding representation of ESI in > draft-ietf-bess-evpn-yang-06. > > > > I have raised similar concerns in an email > <https://mailarchive.ietf.org/arch/msg/bess/HfPveilbdp5iufQYh5pu9KSf4Ug> > to the authors of the draft and to the WG last August (when the draft was > still at its -05 version). > > Unfortunately, I did not receive any response, nor were my concerns > addresses in the -06 version of the draft. > > > > My 2c, > > Sasha > > > > Office: +972-39266302 > > Cell: +972-549266302 > > Email: alexander.vainsht...@ecitele.com > > > > *From:* BESS <bess-boun...@ietf.org> *On Behalf Of *Yu Tianpeng > *Sent:* Saturday, March 2, 2019 9:31 PM > *To:* draft-ietf-bess-evpn-yang.auth...@ietf.org > *Cc:* bess@ietf.org > *Subject:* [bess] Review on draft-ietf-bess-evpn-yang-06 > > > > Dear authors and WG, > > I had a review on draft-ietf-bess-evpn-yang-06. > > This draft covers most of the evpn/vpws scenarios, but there are some > points I would like to discuss. > > > > For small points that do not impact the architecture of the yang data > model, I have made changes in a new yang file directly (attached), which > will be easier to understand. > > Also, some comments related to the structure of the data model in the > final. > > Appreciate authors and WG can have a review on comments and proposed > changes below. > > Thanks in advance. > > Regards, > > Tim > > > ================================================================================= > > Changes: > > ietf-ethernet-segment.yang > > 1. esid-type defined. current uint32 cannot cover 10 octs ESI. We can > either use a string with a regex or uint64 with a range. in the attachment > is the regex. > > 2. change key to esi instead of "name", the name looks like a string or a > description to me, cannot be used as the key. > > 3. add new leaf "interface" to indicate which ESI applied to which > interface. > > 4. BGP parameters are deleted, I don't think RD RT are concepts related > with ES. I have put a new leaf es-list in the EVPN yang data model > providing links between ES and EVPN yang > > 5. change of VLAN type to unit16 with range limit > > > > ietf-evpn.yang > > 1. evpn label mode function added > > 2. re-write RD RT part as rt-types:vpn-route-targets is a list type > already, an extra level of list definition in EVPN is not needed anymore. > > 3. ES list leaf added. This leaf provides a list indicating ESs bound to > the EVPN instance. > > 4. control word and MTU added > > 5. statistics part re-structure. > > 6. change the counter type from uint32 to counter64 to avoid overflow. > > 7. interface type leaf added, the previous vpws-vlan-aware deleted. > reason: all interface type should be covered across EVPN and VPWS > > > > General comments: > > 1. In EVPN yang, I would suggest to re-structure current content as below > instead of one evpn container : > > - EVPN general: RD, RT, name, etc. > > - EVPN ELAN/VPWS/E-TREE specifics: BUM, leaf indication, proxy.. and use > the generals to get yang data model for each scenario. > > > > 2. EVPN parameters are not registered under /ni:network-instances and /pw: > pseudowires. Now only very few info registered > > > > > > > > > > > > > ___________________________________________________________________________ > > This e-mail message is intended for the recipient only and contains > information which is > CONFIDENTIAL and which may be proprietary to ECI Telecom. If you have > received this > transmission in error, please inform us by e-mail, phone or fax, and then > delete the original > and all copies thereof. > ___________________________________________________________________________ > > ___________________________________________________________________________ > > This e-mail message is intended for the recipient only and contains > information which is > CONFIDENTIAL and which may be proprietary to ECI Telecom. If you have > received this > transmission in error, please inform us by e-mail, phone or fax, and then > delete the original > and all copies thereof. > ___________________________________________________________________________ >
_______________________________________________ BESS mailing list BESS@ietf.org https://www.ietf.org/mailman/listinfo/bess