Hi Murray, Thanks for your review and comments. Please see my responses inline marked with "AS>".
Cheers, Ali On 7/13/20, 10:50 PM, "Murray Kucherawy via Datatracker" <nore...@ietf.org> wrote: Murray Kucherawy has entered the following ballot position for draft-ietf-bess-evpn-inter-subnet-forwarding-09: No Objection When responding, please keep the subject line intact and reply to all email addresses included in the To and CC lines. (Feel free to cut this introductory paragraph, however.) Please refer to https://www.ietf.org/iesg/statement/discuss-criteria.html for more information about IESG DISCUSS and COMMENT positions. The document, along with other ballot positions, can be found here: https://datatracker.ietf.org/doc/draft-ietf-bess-evpn-inter-subnet-forwarding/ ---------------------------------------------------------------------- COMMENT: ---------------------------------------------------------------------- Bigger stuff: I'll see Benjamin's DISCUSS (meaning I support it) and raise him the two seemingly normative references to [TUNNEL-ENCAP], which is also undefined. AS> Added the reference to tunnel-encap. As someone not from the Routing Area, I found this a little hard to read because it becomes quite dense with acronyms in some places. Is Section 13 intended to remain in the final published version? Is it needed? AS> removed this one-line section since other RFCs don't have such an explicit IPR section. In the Abstract, please expand all acronyms on first use in the abstract, per the I-D Guidelines. AS> Done. Lesser stuff: Thanks for the glossary in Section 1. I note that, although defined in the glossary here, the terms "DGW", "Ethernet A-D route", "GRE", "GW IP", "IPL", "ML", and "VXLAN" are not used anywhere else in this document. AS> removed the unused terms. There are lots of places where a single hyphen is used to break up a sentence, such as to introduce an example. These should be em dashes, or commas, or maybe semicolons, but not simply hyphens. AS> done. A few nits: Section 2: * "... all the inter-subnet forwarding are performed ..." -- s/are/is/ AS> done. * "... connected to the same PE, wanted to communicate ..." -- remove the comma AS> done. Section 3: * "A MAC-VRF can consists of one ..." -- either drop "can", or use "consist" AS> done. _______________________________________________ BESS mailing list BESS@ietf.org https://www.ietf.org/mailman/listinfo/bess