Comment 20 for bug 1664301

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

Reviewed: https://review.opencontrail.org/29092
Committed: http://github.org/Juniper/contrail-controller/commit/99fbd41081d09e2a2186a92bd57b7d4c696ea27e
Submitter: Zuul (<email address hidden>)
Branch: R3.0.3.x

commit 99fbd41081d09e2a2186a92bd57b7d4c696ea27e
Author: Manish <email address hidden>
Date: Tue Feb 21 07:18:04 2017 +0530

Dont use link local flag in bgp-aas flow encode.

Presence of BGP service flag is indication of bgp-aas flows. Link local is not
needed to identify candidate for relaxed policy. Bgp flags also indicates
relaxed policy on fabric.
Presence of this flag used to cause port to be freed on flow delete even though
there will be a second flow using this port. Vrouter uses link local flag to
release port. Removing this flag ensures that vrouter deos not release the port
used in bgp-aas.
In case flow is not present and port is still part of relaxed policy then packet
will be sent to host, hence achieving the functionailty when port would have
been released.

Conflicts:
 src/vnsw/agent/vrouter/ksync/flowtable_ksync.cc

Change-Id: Id6cfd2d0b0ff3dc0724715bc126ccee882068486
Closes-bug: #1664301