Hi Jeffrey, Thanks for addressing my comments. See some comments inline.
From: Jeffrey Zhang <zzh...@juniper.net> Date: Monday, November 28, 2022 at 9:09 AM To: Acee Lindem <a...@cisco.com>, "draft-ietf-bess-evpn-irb-mc...@ietf.org" <draft-ietf-bess-evpn-irb-mc...@ietf.org>, Routing ADs <rtg-...@ietf.org> Cc: Routing Directorate <rtg-...@ietf.org>, "bess@ietf.org" <bess@ietf.org> Subject: RE: Routing Directorate Last Call Review for draft-ietf-bess-evpn-irb-mcast-07.txt Hi Acee, Thanks a lot for your thorough review and comments. I have posted -08 revision to address most of your comments: https://www.ietf.org/rfcdiff?url2=draft-ietf-bess-evpn-irb-mcast-08.txt. Please see zzh> below. Juniper Business Use Only From: Acee Lindem (acee) <a...@cisco.com> Sent: Tuesday, November 15, 2022 1:49 PM To: draft-ietf-bess-evpn-irb-mc...@ietf.org; Routing ADs <rtg-...@ietf.org> Cc: Routing Directorate <rtg-...@ietf.org>; bess@ietf.org Subject: Routing Directorate Last Call Review for draft-ietf-bess-evpn-irb-mcast-07.txt [External Email. Be cautious of content] Hello, I have been selected as the Routing Directorate reviewer for this draft. The Routing Directorate seeks to review all routing or routing-related drafts as they pass through IETF last call and IESG review, and sometimes on special request. The purpose of the review is to provide assistance to the Routing ADs. For more information about the Routing Directorate, please see http://trac.tools.ietf.org/area/rtg/trac/wiki/RtgDir Although these comments are primarily for the use of the Routing ADs, it would be helpful if you could consider them along with any other IETF Early Review/Last Call comments that you receive, and strive to resolve them through discussion or by updating the draft. Document: draft-ietf-bess-evpn-irb-mcast-07.txt Reviewer: Acee Lindem Review Date: Nov 15th, 2022 IETF LC End Date: Nov 7, 2022 Intended Status: Standards Track Summary: I have some minor concerns about this document that I think should be resolved before publication. Comments: The draft is readable per se but the subject matter, Optimized Inter-Subnet Multicast, is quite complex. The draft covers the mechanisms and procedures for multicast advertisement and forwarding between tenant-BDs. Additionally, a single line in the abstract includes procedures to accommodate multicast traffic external to the tenant domain results in very dense specification of various interworking with other multicast domains. These interworking scenarios build on the OISM gateway functionality specified early in the document. The cascaded complexity probably explains the number of directorate members who declined the review request. Given the complexity, this is a document that could really benefit from implementation experience. Major Issues: None Minor Issues: 1. The concept of multicast packets and, in some cases, advertisements being sent "Up" or "Down" the IRB interface seemed confusing to me. I'd of thought the IRB interfaces would be described in terms of transmission or reception by the IRB L3 Routing instance. In any case, the usage must be described in the terminology section is not intuitive even though one can reverse engineer what is meant. Zzh> An IRB interface connects a bridge domain (L2) to an IP routing instance (L3). Therefore, we use the up/down to indicate if traffic is up towards the L3 or down towards L2. I’ve added the following paragraph in section “1.1.2. Inter-BD (Inter-Subnet) IP Traffic” where IRB interface was firs mentioned in the document: In this document, when traffic is routed out of an IRB interface, we say it is sent down the IRB interface to the BD that the IRB is for. In the other direction, traffic is sent up the IRB interface from the BD to the L3 routing instance. That’s better. I was wondering why the IRB interface wasn’t conceptually an L3 routing instance interface and “sent down” would be “transmitted” and “sent up” would be “received”. This would be a cleaner abstraction to me. However, I must admit I only read Appendix A and not RFC 9135. 2. The Section 6 interworking scenarios could benefit from some ASCII art for visual reference of the various gateway and domain scenarios. Zzh> I have added the following figure: src1 rcvr1 | | R1 RP R2 PIM/MVPN domain +---+ +---+ -----|GW1|----------------------|GW2|---- +---+ +---+ | \ \ / / | | \ \ / / | BD1 BD2 SBD SBD BD2 BD1 EVPN Domain SBD SBD / \ / \ +---+ +---+ |PE1| |PE2| +---+ +---+ | \ / | BD1 BD2 BD2 BD1 | | | | src2 rcvr2 src3 rcvr3 Thanks – This is instructive. The specification is very dense in terms of the interoperability (section 5) and external traffic (section 6). These use cases could also benefit from a picture as well. However, this would be a lot to add in a last call review. Perhaps, this this better addressed in vendor white papers and documentation when the various use cases are supported. 3. Since this is Last Call review, it seems references to topics that may be covered in future revisions of the draft should be removed. Zzh> I fixed two such topics to say “may be specified in separate documents”. Thanks. 4. In section 4.1.1, why are ACs that are not using IGMP/MLD automatically added to the OIF list for all flows? I'd think an administrator would have to run IGMP/MLD on ACs on which multicast traffic is desired. Zzh> For a L2 switch, by default multicast is handled as broadcast – flooded everywhere – unless IGMP/MLD snooping is used. I added “snooping” in the paragraph: An EVPN-PE may run IGMP/MLD snooping procedures on each of its ACs, in order to determine the set of flows of interest to each AC. Okay, this makes more sense. I inherently think more in terms of L3 than L2. 5. In section 4.2, how can one lookup S in the MAC-VRF(s) of a tenant domain? S is the IP address of the source - not a MAC address. This needs to be clarified. Zzh> EVPN does advertise IP address along with MAC address. It’s actually fine looking up either in the MAC VRF or IP VRF, so I removed the lookup details. Thanks, 6. In section 6.1.2.2.1, it seems a bit odd to have the MEG import and export unicast routes dependent on whether or not there are hosts in the EVPN transmitting multicast flows? What route should be exported – a host route to the source or the corresponding subnet route from the EVPN IP RIB? Why isn't the AC source route covered by a subnet route for the corresponding tenant BD? Zzh> Because two MEGs can be attached to the same subnet, while a source S is only attached to single MEG (if the S is local to the MEG at all), we want the L3VPN egress PE to direct its (s,g) joins towards the MEG that has S locally attached. That’s why the MEG SHOULD advertise the host route when there is traffic (and withdraw after the traffic stops), in addition to the subnet route that both MEGs do advertise: As a result, if S is attached to a MEG, the L3VPN nodes will direct their MVPN C-multicast Join routes to that MEG. … If S is not attached to a MEG, the L3VPN nodes will direct their C-multicast Join routes to whichever MEG appears to be on the best route to S's subnet. Upon receiving the C-multicast Join, that MEG will originate an EVPN SMET route for (S,G). As a result, the MEG will receive the (S,G) traffic at layer 2 via the OISM procedures. The (S,G) traffic will be sent up the appropriate IRB interface, and the layer 3 MVPN procedures will ensure that the traffic is delivered to the L3VPN nodes that have requested it. Thanks – So this must be the unicast host route of the source. Perhaps, “host” could be added to the text. 7. Section 6.2, I reworded some text that didn't parse at all. I rewrote as: Furthermore, even if a particular AC is part of that BD, the PE SHOULD NOT transmit an IGMP/MLD Join on that AC unless there is an external PIM route attached via that AC zzh> Thanks! That’s what the original text tried to say. I also changed “route” to “router”. It looks good. Nits: Zzh> I fixed the nits below. In particular, for nit #1 below I changed “about” to “for”. 1. Saying a route is "about" a BD is awkward. Please use "pertains to" or "associated with". 2. Avoid the usage of "we" and use the infinitive instead. For example, "It is RECOMMENDED", rather than "We RECOMMEND". I didn’t fix all these In the diff. 3. Avoid putting extra parentheses around single references - I've fixed this in the diffs. 4. The draft uses various terms for assuring reception of multicast traffic - "draw", "pull", and "must see". I'd use "receive" consistently as in the diff. 5. Use "sent on ..." rather than "sent out ...". See attached RFC diff for more suggested editorial changes. Zzh> Thanks! They show the extraordinary effort you’ve put in and I really appreciate it! I fixed most of them, though kept a few unchanged. Zzh> For example, I did not change “pull” or “draw” to “receive”, because “pull/draw” refers to the fact that a PE advertises SMET routes to draw traffic. I see you later changed “pull” to “attract”. “pull” or “draw” seems to be more explicit? Zzh> Thanks! Zzh> Jeffrey Thanks – it looks good. Acee Thanks, Acee
_______________________________________________ BESS mailing list BESS@ietf.org https://www.ietf.org/mailman/listinfo/bess