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é [http://www.cisco.com/c/dam/m/en_us/signaturetool/images/banners/standard/09_standard_graphic.png] Luc André Burdet lbur...@cisco.com<mailto: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<mailto:bess-boun...@ietf.org>> on behalf of Alexander Vainshtein <alexander.vainsht...@ecitele.com<mailto:alexander.vainsht...@ecitele.com>> Date: Sunday, March 3, 2019 at 05:46 To: Yu Tianpeng <yutianpeng.i...@gmail.com<mailto:yutianpeng.i...@gmail.com>>, "draft-ietf-bess-evpn-yang.auth...@ietf.org<mailto:draft-ietf-bess-evpn-yang.auth...@ietf.org>" <draft-ietf-bess-evpn-yang.auth...@ietf.org<mailto:draft-ietf-bess-evpn-yang.auth...@ietf.org>> Cc: "bess@ietf.org<mailto:bess@ietf.org>" <bess@ietf.org<mailto: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<mailto:alexander.vainsht...@ecitele.com> From: BESS <bess-boun...@ietf.org<mailto: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<mailto:draft-ietf-bess-evpn-yang.auth...@ietf.org> Cc: bess@ietf.org<mailto: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