[Build CentOS release 6.5 2.10-37~icehouse.el6] Multi-Interface Sanity - vrouter crashed at NhDecode(NextHop const*, PktInfo const*, PktFlowInfo*, PktControlInfo*, PktControlInfo*, bool)

Though there was only one vgw in setup exporting 0/0 route, agent got a ecmp route for 0/0 route,
thought both next hop and label are same,there encap types differ.

<message from=\"<email address hidden>\" to=\"nodei5/bgp-peer\">
    <event xmlns=\"http://jabber.org/protocol/pubsub\">
        <items node=\"1/1/default-domain:admin:vgwvn:vgwvn\">
            <item id=\"0.0.0.0/0\">
                <entry>
                    <nlri>
                        <af>1</af>
                        <safi>1</safi>
                        <address>0.0.0.0/0</address>
                    </nlri>
                    <next-hops>
                        <next-hop>
                            <af>1</af>
                            <address>192.168.192.5</address>
                            <label>17</label>
                            <tunnel-encapsulation-list>
                                <tunnel-encapsulation>gre</tunnel-encapsulation>
                            </tunnel-encapsulation-list>
                        </next-hop>
                        <next-hop>
                            <af>1</af>
                            <address>192.168.192.5</address>
                            <label>17</label>
                            <tunnel-encapsulation-list>
                                <tunnel-encapsulation>gre</tunnel-encapsulation>
                                <tunnel-encapsulation>udp</tunnel-encapsulation>
                            </tunnel-encapsulation-list>
                        </next-hop>
                    </next-hops>
                    <version>1</version>
                    <virtual-network>default-domain:admin:vgwvn</virtual-network>
                    <sequence-number>0</sequence-number>
                    <security-group-list />
                    <local-preference>100</local-preference>
                </entry>
            </item>
        </items>

If there is a ecmp of vgw and traffic originates first from outside to contrail-domain VM,
agent was trying to get policy enabled next hop for reverse flow setup which would not
be created for vgw.
Fix for to take policy enabled next hop for reverse flow only destination is vm interface.

Reviewed: https://review.opencontrail.org/7875
Committed: http://github.org/Juniper/contrail-controller/commit/f000a7fcd9c99fff809c87d3f30ed7e162ace1a8
Submitter: Zuul
Branch: R2.1

commit f000a7fcd9c99fff809c87d3f30ed7e162ace1a8
Author: Naveen N <email address hidden>
Date: Thu Feb 26 05:57:20 2015 -0800

* In a scenario where there are 2 vgw exporting same subnet route,
and traffic session gets initiated from external domain to
VM, then for setting reverse flow agent was trying to get policy
enabled interface nexthop for vgw, and for vgw we would only
create policy disabled nh. Fixing the same to pick
policy disabled nexthop for vgw destination
Closes-bug:#1425856

Change-Id: Ic0c61be47997a2c9ebe3196394cef21468fce8ca

Reviewed: https://review.opencontrail.org/7905
Committed: http://github.org/Juniper/contrail-controller/commit/d0414d2e6eeae350ecf4189f894c475e0ef00d21
Submitter: Zuul
Branch: R2.0

commit d0414d2e6eeae350ecf4189f894c475e0ef00d21
Author: Naveen N <email address hidden>
Date: Fri Feb 27 01:58:02 2015 -0800

* In a scenario where there are 2 vgw exporting same subnet route,
and traffic session gets initiated from external domain to
VM, then for setting reverse flow agent was trying to get policy
enabled interface nexthop for vgw, and for vgw we would only
create policy disabled nh. Fixing the same to pick
policy disabled nexthop for vgw destination
Closes-bug:#1425856

Change-Id: I23462aec80cc7fc351c376705f2fb2dc062acf16

Review in progress for https://review.opencontrail.org/8032

Reviewed: https://review.opencontrail.org/8032
Committed: http://github.org/Juniper/contrail-controller/commit/9d44b8f521cf02f70ecb7cd628754c64b8de3b9a
Submitter: Zuul
Branch: master

commit 9d44b8f521cf02f70ecb7cd628754c64b8de3b9a
Author: Hari <email address hidden>
Date: Tue Mar 3 21:29:48 2015 -0800

Merge the following commits from R2.1 branch (final).

Perf improvements for Logical Interface delete handling

Follow up on commit b4af4d67cef56ac85da5bb0d50bb55f882c70538 for delete
handling

When a link delete is notified to config listener, the IFMapNode in link
can be NULL. We used to ignore links with left/right node deleted.

Modified the link handler to pass the node-type when left/right node is
deleted. This way, the IFLinkToReq handler can take action based on the
type of node being deleted
(cherry picked from commit f1d7816dbdc886859caec5152d2b436ad4fb2a96)

Conflicts:
src/vnsw/agent/test-xml/test_xml_oper.cc

Holding the UUID from ID_PERMS in config DBState in DB Add and using it
for Config Delete
closes-bug: #1420209
(cherry picked from commit 6a9b4be6a0dcfb3980bd072a57f92761a548eb9c)

* Send ARP request packet on VMI interface instead of parent interface
such that vlan gets added, based on transmit vlan id
* If vlan-id is 0, then its treated as untagged interface, in agent
while framing the packet we were setting the tag in packet 0
avoiding the same same setting vlan id as invalid and programming
it as vlan-id 0 only in vrouter
Closes-bug:#1423122
(cherry picked from commit 38f1405b27e05e5d31a2ae6297ca42d915b368bd)

* If arp route is deleted twice, then we arp nexthop was not
getting deleted, since upon each arp route delete we enqueue
a request to modify nexthop to invalid and this might
result in arp entry creation, hence use operation RESYNC
instead of add/change
* Check for vrf while parsing on subnet address change, since
we might modify next vrf subnet routes if prefixx matches
Test case for same
Closes-bug:#1423502
(cherry picked from commit a65d557c27d15de5d7d0bf30fdc5a20fa93acf42)

Conflicts:
src/vnsw/agent/test/test_route.cc

reverting the unwated checkin of modifying the notify_count_ to bool

(cherry picked from commit 975f507d81ce89106814f4fced1d14b454dcdd5a)

* Whenever ICMP error packet is trapped for flow setup,
parse the inner payload and frame the flow key
Closes-bug:#1423848
(cherry picked from commit 81ee64728201b1d8234d790772ab3035874c1f5d)

* ARP route behind gateway should have proxy flag set
such that arp request on VM would be proxied by vrouter
Closes-bug:#1423122
(cherry picked from commit e72443e1a73d3372944981f15941e97a5a9f439f)

Remove unused UVEs

Remove UVE generated on add/delete of a link
Remove UVE generated on change of physical-device-vn entry.
Added a UT for physical-device-vn to validate add/delete of VMI to VN link

(cherry picked from commit 452b9341668f0a89455cc5e1847c0c7d70f0d3b2)

Holding the metadata of link while storing it in defer list. While
adding the link to Graph from defer list, the metadata also added back
to graph which makes the dependency tracker to work correctly

(cherry picked from commit 43d7a71e1c486a3c326d16e16134c25312b4953f)

Optimize config handling for SG and...