Comment 2 for bug 1381978

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

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

commit f1370e28e6e1cb8e9bc6b4bdac7b4c4c1442aefc
Author: Naveen N <email address hidden>
Date: Thu Nov 20 22:18:14 2014 -0800

* All the component nh member of a composite NH are policy disabled
nexthop, since composite NH itself is policy enabled and there is
no need for component member to policy disabled. When a transition
happens from non-ecmp to ecmp, local interface nexthop could be
added with policy enabled flag, and finding of the nexthop inside
composite NH would fail since we always search for policy disabled
nh inside composite NH. Since nexthop lookup fails flow ecmp index
would not be set, resulting in drop
Unit test added to verify the case where transition happens from
non-ecmp to ecmp with local nexthop pre-existing
Closes-bug:#1381978

Change-Id: Iba53ae69c9bbdf8e796ee56d77b0a961b2f4ebf4