Skip to content

Commit

Permalink
MPLS gre never get picked up for remote route.
Browse files Browse the repository at this point in the history
Since there is a change in interpretation of TunnelEncapType, agent failed to
identify GRE. Solution is to use the modified encap type name.
https://bugs.launchpad.net/juniperopenstack/+bug/1588554

Change-Id: I36540574d46f29387f72ff458e9d4b758e4efada
Closes-bug: #1605545
  • Loading branch information
manishsing committed Jul 22, 2016
1 parent 2e78fd4 commit 7b14dcf
Showing 1 changed file with 6 additions and 3 deletions.
9 changes: 6 additions & 3 deletions src/vnsw/agent/controller/controller_peer.cc
Expand Up @@ -298,7 +298,8 @@ GetEnetTypeBitmap(const EnetTunnelEncapsulationListType &encap) {
iter != encap.end(); iter++) {
TunnelEncapType::Encap encap =
TunnelEncapType::TunnelEncapFromString(*iter);
if (encap == TunnelEncapType::MPLS_O_GRE)
if ((encap == TunnelEncapType::GRE) ||
(encap == TunnelEncapType::MPLS_O_GRE))
bmap |= (1 << TunnelType::MPLS_GRE);
if (encap == TunnelEncapType::MPLS_O_UDP)
bmap |= (1 << TunnelType::MPLS_UDP);
Expand All @@ -315,7 +316,8 @@ GetTypeBitmap(const TunnelEncapsulationListType &encap) {
iter != encap.end(); iter++) {
TunnelEncapType::Encap encap =
TunnelEncapType::TunnelEncapFromString(*iter);
if (encap == TunnelEncapType::MPLS_O_GRE)
if ((encap == TunnelEncapType::GRE) ||
(encap == TunnelEncapType::MPLS_O_GRE))
bmap |= (1 << TunnelType::MPLS_GRE);
if (encap == TunnelEncapType::MPLS_O_UDP)
bmap |= (1 << TunnelType::MPLS_UDP);
Expand All @@ -329,7 +331,8 @@ GetMcastTypeBitmap(const McastTunnelEncapsulationListType &encap) {
iter != encap.end(); iter++) {
TunnelEncapType::Encap encap =
TunnelEncapType::TunnelEncapFromString(*iter);
if (encap == TunnelEncapType::MPLS_O_GRE)
if ((encap == TunnelEncapType::GRE) ||
(encap == TunnelEncapType::MPLS_O_GRE))
bmap |= (1 << TunnelType::MPLS_GRE);
if (encap == TunnelEncapType::MPLS_O_UDP)
bmap |= (1 << TunnelType::MPLS_UDP);
Expand Down

0 comments on commit 7b14dcf

Please sign in to comment.