Comment 17 for bug 1491047

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/14446
Committed: http://github.org/Juniper/contrail-controller/commit/40c8571b783cfe10d260607d0a91f11819f2b4d0
Submitter: Zuul
Branch: master

commit 40c8571b783cfe10d260607d0a91f11819f2b4d0
Author: Manish <email address hidden>
Date: Wed Jun 29 12:25:25 2016 +0530

Use Inet component from Evpn routes for IP mac binding.

Whenever EVPN route is received from CN, IP was not being used.
With the current changes IP will be installed with a new peer known as
inet_evpn_peer. This path will be lesser in priority w.r.t. local_vm path and
BGP peer received inet path. For nexthop and label this path will use
parent subnet rt. Nexthop and label and other path parameters will be same as of
subnet route. (Note: subnet route can be many and the best match will be
selected).
On subnet route going off it will fallback to next best available subnet and so
on. If nothing is available then its discard. Similar thing happens on addition
of better subnets.

Some more notes w.r.t. these changes:
- For IPAM gateway if EVPN route is received with MAC from remote peer,
stitching is done only on TSN and not on any other type of compute node.
This forces non TSN compute to do proxy always for gateway.
- If VN does not have layer3 forwarding enabled then stitching is done for
whatever is available. It breaks the above point as well.

Also from route_ksync.cc remove the check of ipam_subnet_route for marking arp
flags to flood. Now composite will always have proxy flags. Flood flag was
present to handle ECMP NH received for gateway route. Now when arp was issued
for baremetal behind gateway it will get flooded. With this fix all EVPN routes
from external router will be used to program IP and add mac stitching. So any
arp for baremetal will be resolved by vrouter itself using mac binding. There
will be no need to flood arp on ECMP NH received.

Closes-bug: 1491047

Conflicts:
src/vnsw/agent/oper/agent_path.cc
src/vnsw/agent/oper/inet_unicast_route.cc

Change-Id: I113e3c46d0e63441878a7e007dd5240bef7a339f