[R4.1-7]: Flow is created when policy is disabled for ecmp case

[alok kumar]

Sanity test case: DisablePolicyEcmpIpv6.test_ecmp_with_static_routes
Sanity report: http://10.204.216.50/Docs/logs/4.1.0.0-7_jenkins-SMLite_ubuntu-14-04_mitaka_Openstack_HA_Sanity-367_1512024480.27/junit-noframes.html

This case is to test no flow created when policy is disabled on all the VMIs with ecmp destination but on the source node flow is getting created for reverse traffic.

source VM: 131.234.239.3,
ECMP destinations: 131.234.239.4, and 131.234.239.5
ECMP destinations are on nodem8 where static IP 211.153.168.75 is configured on the VMs.

on nodem10(source VM's node):
    Index Source:Port/Destination:Port Proto(V)
-----------------------------------------------------------------------------------
    60160<=>357472 131.234.239.3:57242 17 (2)
                         211.153.168.75:11000
(Gen: 6, K(nh):29, Action:F, Flags:, QOS:-1, S(nh):30, Stats:0/0, SPort 63479,
 TTL 0, Sinfo 0.0.0.0)

   357472<=>60160 211.153.168.75:11000 17 (2)
                         131.234.239.3:57242
(Gen: 4, K(nh):29, Action:F, Flags:, QOS:-1, S(nh):21, Stats:1/170, SPort 55866,
 TTL 0, Sinfo 10.10.10.8)

vrouter team has debugged the issue and found the problem.

[OpenContrail Admin (ci-admin-f)]

Review in progress for https://review.opencontrail.org/38014
Submitter: Naveen N (<email address hidden>)

[OpenContrail Admin (ci-admin-f)]

Review in progress for https://review.opencontrail.org/38015
Submitter: Naveen N (<email address hidden>)

[Hari Prasad Killi (haripk)]

This issue was present in earlier builds & releases as well. When policy is disabled or enabled with encapsulation priority having VxLAN as the highest, the bridge route entry wasnt being updated with policy disabled NH, resulting in flows being setup. As the sanity test case was with ECMP, this issue was hidden and is showing up now as ARP handling was modified with https://bugs.launchpad.net/juniperopenstack/+bug/1733811.

[OpenContrail Admin (ci-admin-f)]

Reviewed: https://review.opencontrail.org/38014
Committed: http://github.com/Juniper/contrail-controller/commit/12bad88cba1c8daa3b27a84dffc51cabbfcc7512
Submitter: Zuul (<email address hidden>)
Branch: R4.1

commit 12bad88cba1c8daa3b27a84dffc51cabbfcc7512
Author: Naveen N <email address hidden>
Date: Thu Nov 30 18:01:45 2017 +0530

* Handle policy change flag in vxlan mode

In case of MPLS encap label change triggers in revaluation of
bridge route, in case of vxlan encap nexthop change should be
treated as indication to update bridge route.
Test case for same.

Change-Id: I53692f83b1c6d898e54902a81579215aef745de4
Closes-bug: #1735385

[OpenContrail Admin (ci-admin-f)]

Reviewed: https://review.opencontrail.org/38015
Committed: http://github.com/Juniper/contrail-controller/commit/673bce4bb324e5411f8d1434d6c4284636bdd4f4
Submitter: Zuul (<email address hidden>)
Branch: master

commit 673bce4bb324e5411f8d1434d6c4284636bdd4f4
Author: Naveen N <email address hidden>
Date: Thu Nov 30 18:01:45 2017 +0530

* Handle policy change flag in vxlan mode

In case of MPLS encap label change triggers in revaluation of
bridge route, in case of vxlan encap nexthop change should be
treated as indication to update bridge route.
Test case for same.

Change-Id: I53692f83b1c6d898e54902a81579215aef745de4
Closes-bug: #1735385