Hi Authors, I have the following comments on draft-ietf-bess-evpn-ipvpn-interworking-03 version of the draft. Please see below.
Thanks, Suresh 4.1. No-Propagation-Mode This is the default mode of operation. Comment: We must not have default mode of operation which could result in loops. So the default mode of operation must be Uniform-Propagation-mode, and No-Propagation-Mode should be made optional. -- 4.3. Aggregation of Routes and Path Attribute Propagation - ISF routes that have different attributes of the following type codes MUST NOT be aggregated: D-PATH, LOCAL_PREF, ORIGINATOR_ID, CLUSTER_ID, MED or AIGP. Comment: The draft can add a rule on D-PATH asking for not aggregating routes if they have a different D-PATH value. However, the draft must not talk about other attributes (this is out of scope of the work). -- 5. Route Selection Process between EVPN and other ISF SAFIs For a given prefix advertised in one or more non-EVPN ISF routes, the BGP best path selection procedure will produce a set of "non-EVPN best paths". For a given prefix advertised in one or more EVPN ISF routes, the BGP best path selection procedure will produce a set of "EVPN best paths". To support IP/EVPN interworking, it is then necessary to run a tie-breaking selection algorithm on the union of these two sets. Comment: The above sentence does not convey the context in which the best path is calculated. Do you mean the following. For a given prefix present in one or more ISF routes, the BGP best path is first run in the respective ISF SAFI and then again on the union of ISF SAFI best paths in the IP-VRF context. -- 5. Route Selection Process between EVPN and other ISF SAFIs 4- Steps 1-3 could possibly leave Equal Cost Multi-Path (ECMP) between IP and EVPN paths. By default, the EVPN path is considered (and the IP path removed from consideration). However, if ECMP across ISF SAFIs is enabled by policy, and an "IP path" and an "EVPN path" remain at the end of step 3, both path types will be used. Comment: Referring to non-EVPN paths as IP paths is confusing since EVPN path is also an IP path -- 7. Gateway PE Procedures The gateway PE procedures are described as follows: o A gateway PE that imports an ISF SAFI-x route to prefix P in an IP-VRF, MUST export P in ISF SAFI-y if: 1. P is installed in the IP-VRF (hence the SAFI-x route is the best one for P) and 2. PE has a BGP peer for SAFI-y (enabled for the same IP-VRF) and 3. Either x or y is EVPN Comment: This above text is unclear and possibly wrong on a couple of points: - 1. SAFI-x route is the BGP best path for P and - The last point brings some ambiguity. As SAFI 1 is considered as an ISF SAFI, you have to consider all combinations of ISF at the gateway, not only EVPN <-> something else. The current text could be interpreted in a way that SAFI 1 <-> SAFI 128 route export is broken and it must not. --
_______________________________________________ BESS mailing list BESS@ietf.org https://www.ietf.org/mailman/listinfo/bess