Service chain route leaking/re-origination enhancements

Bug #1500698 reported by Nischal Sheth
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R3.0
Fix Committed
Medium
Prakash Bailkeri
Trunk
Fix Committed
Medium
Prakash Bailkeri

Bug Description

This is a placeholder bug to track several enhancements to route leaking
and re-origination in the context of service chaining.

1. Ability to prevent re-origination of interface static routes - this is
typically required when routes are configured on an interface that belongs
to a service VM. Need to allow user to configure a NoReoriginate property
for interface static route.

Sample use case is where there's a right VN which is a public network and
the last VM in the SFC between the left VN and the right VN does NAT using
a NAT pool. The right interface of the VM needs to be configured with an
interface static route for the NAT pool so that destination in the right
VN know how to reach addresses in the NAT pool. However, the NAT pool
prefix clearly should not be re-orignated into the left VN.

2. Advertise default route (instead of all specifics) into access VN/VPN.

This is required for scaling reasons when PEs in an access VPN cannot be
burdened (in control and/or data plane) with all the more specific routes
from other access VPNs. It is sufficient to advertise a default route into
the access VPN. Ideal behavior is for the default to be an aggregate route
that gets activated when at least one more specific route is available.

3. Advertise aggregate (instead of more specifics) with service chaining

Consider a left VN/VPN and right VN/VPN with a SFC applied via a policy.
The left interface of the leftmost VM in the SFC could be configured with
an interface static route that represents an aggregate of all the routes
in the right VN/VPN. In this case, service chaining should only advertise
this static route into the left VN/VPN and suppress the more specifics.
The forwarding information (Nexthop + Label) for the route should direct
traffic going from left VPN to right VPN to the left interface of leftmost
VM in the SFC.

This is a generalization of item 2 above.
Functionality has to be configurable independently for both directions.
Should provide ability to configure multiple such aggregate static routes
to handle cases where all VPN routes cannot be aggregated to one prefix.

4. Nominal/Backup SFCs in different clusters/DCs

Two identical SFCs need to be created in different contrail clusters. The
SFCs are to be applied for the same pair of VPNs. There's a requirement to
steer traffic from the VPN PEs to the nominal/active SFC, while keeping
the other SFC as a hot standby.

In order to achieve this goal contrail needs to provide the ability to set
the local preference for re-originated routes. Since local preference does
not apply for eBGP, it's required that 1 of 2 user-defined (or predefined)
community values can be attached to re-orignated routes. The VPN PEs can
use policy to match these communities and influence path selection.

Both the local preference and community mechanisms need to be supported to
handle iBGP and eBGP scenarios.

Nischal Sheth (nsheth)
description: updated
information type: Proprietary → Public
description: updated
Nischal Sheth (nsheth)
description: updated
summary: - Service chain route leaking/re-orignation enhancements
+ Service chain route leaking/re-origination enhancements
Nischal Sheth (nsheth)
description: updated
Nischal Sheth (nsheth)
description: updated
tags: added: config
Nischal Sheth (nsheth)
description: updated
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/14643
Submitter: Prakash Bailkeri (<email address hidden>)

Nischal Sheth (nsheth)
description: updated
Nischal Sheth (nsheth)
description: updated
3 comments hidden view all 139 comments
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/14818
Submitter: Prakash Bailkeri (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/14643
Submitter: Prakash Bailkeri (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/14900
Submitter: Prakash Bailkeri (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/14643
Submitter: Prakash Bailkeri (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/14976
Submitter: Prakash Bailkeri (<email address hidden>)

1 comments hidden view all 139 comments
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

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

commit a2298b79102e26a6f47ac00c4bdbfa729690eb00
Author: Prakash M Bailkeri <email address hidden>
Date: Mon Oct 26 10:04:04 2015 -0700

Community attribute to avoid re-orignating the routes

Refer to https://bugs.launchpad.net/juniperopenstack/+bug/1500698

Changes made:
1. Schema changes to add new property for community list and
add this property to static routes
2. XMPP message schema changes to add community to route entry
3. Handling community in Rx/Tx on bgp_xmpp_channel
4. Introduce new type for Community.
5. Avoid service chaining routes which are tagged with "no-reoriginate" community
6. UT for CommunityType, Rx/Tx of community in Xmpp messages and service chain

Change-Id: If57c005fd0e6c0b9705491db2f1da81bb0920000
Related-bug:#1500698

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/15053
Submitter: Prakash Bailkeri (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/14976
Submitter: Prakash Bailkeri (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/15053
Submitter: Prakash Bailkeri (<email address hidden>)

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

Reviewed: https://review.opencontrail.org/14976
Committed: http://github.org/Juniper/contrail-controller/commit/14b8f30b7c56b718b8d3f16e89276db6ae51db1d
Submitter: Zuul
Branch: master

commit 14b8f30b7c56b718b8d3f16e89276db6ae51db1d
Author: Prakash M Bailkeri <email address hidden>
Date: Thu Nov 5 10:17:32 2015 -0800

Routing Policy schema

Refer to https://bugs.launchpad.net/juniperopenstack/+bug/1500698

1. Trim routing_policy.xsd to meet requirements in above bug
a. Prefix match and community match
b. Action to update community and local_pref
2. Link routing instance and routing policy

Change-Id: I3c7df32732f09ae603998bfffee4d2aba8ea10fa
Related-bug:#1500698

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/15053
Submitter: Prakash Bailkeri (<email address hidden>)

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

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

commit bcefae63626115b1b2d5b81813dc44f2d2cba70f
Author: Prakash M Bailkeri <email address hidden>
Date: Wed Nov 11 21:08:32 2015 -0800

Agent support for Community attribute to avoid re-orignating the routes

Refer to https://bugs.launchpad.net/juniperopenstack/+bug/1500698

1. Add community attribute to AgentPath
2. Static routes added with community attribute as per RouteType config object
3. send community in xmpp message for ip v4 & v6 routes
4. UT for the same

Change-Id: Ibb928f535e8fbc0c6b26db1ff6ee6e3f5087e53f
Related-bug:#1500698

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/15277
Submitter: Prakash Bailkeri (<email address hidden>)

3 comments hidden view all 139 comments
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/15410
Submitter: Prakash Bailkeri (<email address hidden>)

1 comments hidden view all 139 comments
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/15277
Committed: http://github.org/Juniper/contrail-controller/commit/3c7f0fcc082b09c01928258476724878e22c7807
Submitter: Zuul
Branch: master

commit 3c7f0fcc082b09c01928258476724878e22c7807
Author: Prakash M Bailkeri <email address hidden>
Date: Thu Nov 19 22:30:37 2015 +0530

Routing policy code changes

Initial version of the config handling code for Routing Policy

Change-Id: I65ae7e7ea8f4a3b6af42202ddc267874733a13af
Closes-bug: #1500698

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/15427
Submitter: Prakash Bailkeri (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/15429
Submitter: Prakash Bailkeri (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/15427
Submitter: Prakash Bailkeri (<email address hidden>)

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

Reviewed: https://review.opencontrail.org/15410
Committed: http://github.org/Juniper/contrail-controller/commit/0337efb4ccd45d828d5cbcc38353e301ec8ec2e6
Submitter: Zuul
Branch: master

commit 0337efb4ccd45d828d5cbcc38353e301ec8ec2e6
Author: Prakash M Bailkeri <email address hidden>
Date: Wed Nov 25 10:21:11 2015 +0530

Schema changes to route aggregation and routing policy

1. Introduce route aggregate identifier.
2. Link Routing instance and route aggregation
3. Link Service instance and routing policy
3. Link Service instance and route aggregation
Related-bug: #1500698

Change-Id: I6c0b5fdd137a4be620098a45d017a9be7f069f16

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Reviewed: https://review.opencontrail.org/15429
Committed: http://github.org/Juniper/contrail-controller/commit/126e8489ae908c7a3f9fba3773898d41da5d63c1
Submitter: Zuul
Branch: master

commit 126e8489ae908c7a3f9fba3773898d41da5d63c1
Author: Prakash M Bailkeri <email address hidden>
Date: Wed Nov 25 22:22:42 2015 +0530

Typo for routing policy lib in rtarget/test/SConscript

Fix the library name

Change-Id: I1ffc7ad55e74e96b0b995e52dad225a7a5f1a0fd
Related-bug: #1500698

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Reviewed: https://review.opencontrail.org/15427
Committed: http://github.org/Juniper/contrail-controller/commit/687268253f8c48a6f10bf91e81c3638fd0f56506
Submitter: Zuul
Branch: master

commit 687268253f8c48a6f10bf91e81c3638fd0f56506
Author: Prakash M Bailkeri <email address hidden>
Date: Wed Nov 25 19:18:42 2015 +0530

UT cases for routing policy config code

1. Added config listener tests
2. Added config ifmap tests

Related-bug: #1500698

Change-Id: I3dc8c72d7f0df67d343bdab1985994593749a5c9

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/15496
Submitter: Prakash Bailkeri (<email address hidden>)

2 comments hidden view all 139 comments
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/15548
Submitter: Prakash Bailkeri (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/15496
Submitter: Prakash Bailkeri (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/15548
Submitter: Prakash Bailkeri (<email address hidden>)

1 comments hidden view all 139 comments
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/15548
Committed: http://github.org/Juniper/contrail-controller/commit/8e1ccb6afabc7c96f1b716bd395a06154e9e7ff5
Submitter: Zuul
Branch: master

commit 8e1ccb6afabc7c96f1b716bd395a06154e9e7ff5
Author: Prakash M Bailkeri <email address hidden>
Date: Mon Nov 30 23:53:26 2015 +0530

Routing policy operational code changes

TODOs:
1. handle update of policy
2. Handle update of policies on routing instance
3. UT coverage
4. Comments and formatting

Routing policy Unit test cases

Bug fix for inet6 prefix match

Change-Id: I83de4a24f52c9781d61f4325246b10e115ad8af6
Related-bug: #1500698

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/15641
Submitter: Prakash Bailkeri (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/15654
Submitter: Prakash Bailkeri (<email address hidden>)

59 comments hidden view all 139 comments
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/16456
Committed: http://github.org/Juniper/contrail-controller/commit/73211b7ea687c4d22706e5e78d5c2c66d1832c69
Submitter: Zuul
Branch: master

commit 73211b7ea687c4d22706e5e78d5c2c66d1832c69
Author: Prakash M Bailkeri <email address hidden>
Date: Fri Jan 22 15:47:05 2016 +0530

Introspect support for route aggregation

Enhanced UT cases to invoke route-aggregation introspect

Change-Id: Ib244faf3b30323c5b032fd4f079410e94da3ce81
Related-bug: #1500698

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/16479
Submitter: Prakash Bailkeri (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/16510
Submitter: Nischal Sheth (<email address hidden>)

1 comments hidden view all 139 comments
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/16479
Committed: http://github.org/Juniper/contrail-controller/commit/793eeb529b8bdf84461e12df82f1de0645f88ff2
Submitter: Zuul
Branch: master

commit 793eeb529b8bdf84461e12df82f1de0645f88ff2
Author: Prakash M Bailkeri <email address hidden>
Date: Mon Jan 25 09:40:00 2016 +0530

Introspect support for routing policy

Enhanced UT cases to invoke routing-policy introspect

Change-Id: I6809c0cfe7ab3a093afd40e97ab1c364bbd025fe
Related-bug: #1500698

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/16538
Submitter: Prakash Bailkeri (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/16558
Submitter: Prakash Bailkeri (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/16609
Submitter: Prakash Bailkeri (<email address hidden>)

1 comments hidden view all 139 comments
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/16558
Committed: http://github.org/Juniper/contrail-controller/commit/4f6bd6877186955cecf025709870442f21896e75
Submitter: Zuul
Branch: master

commit 4f6bd6877186955cecf025709870442f21896e75
Author: Prakash Bailkeri <email address hidden>
Date: Wed Jan 27 22:43:49 2016 +0530

Support matching multiple prefixes for routing policy match

Allow specifying multiple prefixes for routing policy match
Test case to validate it

Change-Id: Ib79fde031aa1c3a56a87288186afdb06757728b4
Related-bug: #1500698

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/16694
Submitter: Prakash Bailkeri (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/16609
Submitter: Prakash Bailkeri (<email address hidden>)

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

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

commit b1b4ea3120f176c35ca28d5e04c01bfc936c5965
Author: Prakash M Bailkeri <email address hidden>
Date: Thu Jan 28 16:18:07 2016 +0530

Add path source as match condition

Schema changes

Change-Id: I228aa80676d3e14b8849fa9962066cf13e32cd75
Related-bug: #1500698

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Reviewed: https://review.opencontrail.org/16510
Committed: http://github.org/Juniper/contrail-controller/commit/6321725832436851c7da14c99e5276a8913ec0b8
Submitter: Zuul
Branch: master

commit 6321725832436851c7da14c99e5276a8913ec0b8
Author: Nischal Sheth <email address hidden>
Date: Mon Jan 25 13:51:58 2016 -0800

Templatize code to sync aggregate config and oper state

Change-Id: Id7035f87aad9c3317cb6b2dd94653660be64fbac
Partial-Bug: 1500698

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/16734
Submitter: Nischal Sheth (<email address hidden>)

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

Reviewed: https://review.opencontrail.org/16734
Committed: http://github.org/Juniper/contrail-controller/commit/04bfcf2b9dc1babf9e9e7ca44f18d439078de2f1
Submitter: Zuul
Branch: master

commit 04bfcf2b9dc1babf9e9e7ca44f18d439078de2f1
Author: Nischal Sheth <email address hidden>
Date: Tue Jan 26 11:32:11 2016 -0800

Make cosmetic changes to use consistent file names

Change-Id: If7dbf8e4fc4ab5f2f17ad6ea547e6f20318a3ae9
Partial-Bug: 1500698

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/16538
Submitter: Prakash Bailkeri (<email address hidden>)

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

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

commit e8c489d1586673922744dd261e8c629472588abe
Author: Prakash M Bailkeri <email address hidden>
Date: Fri Jan 29 23:18:23 2016 +0530

Add path source as match condition

1. control-node changes
2. Unit tests for matching protocols

Change-Id: I406f20762498be79c43ec2c8d2dde46af1c55467
Related-bug: #1500698

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Reviewed: https://review.opencontrail.org/16538
Committed: http://github.org/Juniper/contrail-controller/commit/56295b15ac19ad7b1b0e4b08125a4fe53ac819b5
Submitter: Zuul
Branch: master

commit 56295b15ac19ad7b1b0e4b08125a4fe53ac819b5
Author: Prakash Bailkeri <email address hidden>
Date: Mon Feb 1 08:50:28 2016 +0530

Support setting med in routing policy action

Add support for updating the MED in routing policy action.
Test case to validate it

Change-Id: I23ac0570673329a571c48606e3fc80cdf057081a
Related-bug: #1500698

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/16808
Submitter: Prakash Bailkeri (<email address hidden>)

2 comments hidden view all 139 comments
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/16808
Committed: http://github.org/Juniper/contrail-controller/commit/553e78124fff57cbe8404445de759091ae92424b
Submitter: Zuul
Branch: master

commit 553e78124fff57cbe8404445de759091ae92424b
Author: Prakash Bailkeri <email address hidden>
Date: Wed Feb 3 11:07:35 2016 +0530

Routing policy schema update

Remove usage of keywords from schema
Suffix Type to each type def

Change-Id: If5a2ffda5cebbbd49a3bbf4be296083e0e117ee2
Related-bug: #1500698

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/16816
Submitter: Prakash Bailkeri (<email address hidden>)

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

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

commit e5056fa885f8abb3098a6b6703f28950f6ed5d03
Author: Prakash Bailkeri <email address hidden>
Date: Wed Feb 3 14:38:37 2016 +0530

Fix introspect output for route-aggregate

also fix display of path source
Fixed similar issue in static route introspect

Change-Id: I1665b29562d8cbce1e7a99f89c307248b471ce1a
Related-bug: #1500698

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/16877
Submitter: Prakash Bailkeri (<email address hidden>)

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

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

commit a8991e07c0bc3a5453e9704706146ad738ee4544
Author: Prakash Bailkeri <email address hidden>
Date: Thu Feb 4 23:55:22 2016 +0530

Fix issue in Adding/Setting community list in policy action

If the route doesn't have community attribute and policy action sets/Adds a
new set of community, update is skipped.

Fix this issue and added Unit test cases to validate the case where route has empty
community before applying policy for all scenario(add/set/remove of community)

Change-Id: I58f12a23406bb86edfd8a9ea62c4bbda5cdfd66a
Related-bug: #1500698

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/16963
Submitter: Prakash Bailkeri (<email address hidden>)

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

Reviewed: https://review.opencontrail.org/16963
Committed: http://github.org/Juniper/contrail-controller/commit/7d96ea8b660bca6f9f2b7a7096b5d1d995b670d1
Submitter: Zuul
Branch: master

commit 7d96ea8b660bca6f9f2b7a7096b5d1d995b670d1
Author: Prakash Bailkeri <email address hidden>
Date: Sun Feb 7 23:49:19 2016 +0530

OriginVn for aggregate route

Currently origin vn is not set for aggregate route.

Consider the following example:
1. left-vn -- Service Chain -- Right-vn
2. Internal routing instance created for service chain has route aggregate config
3. Aggregate route for right-vn subnet is added to left-vn

Since the aggregate route doesn't have origin vn, it gets leaked backed to right
network as service chain route.

This can cause black-holing of traffic in right due to wrong next hop for right-vn
subnet route

Fix:
Add origin VN for aggregate route.
a. If the routing instance has service chain config, origin vn is the
dest-routing-instance
b. Else origin-vn is routing instance's virtual network index
Re-order origin-vn calculation in service chain code to consider origin-vn
present in the BgpAttr(extCommunity)
Added test in route aggregate to validate service chain scenario and origin vn

Change-Id: I9f1ffb6fedb816eac7b5f070a965cf202b96cb49
Related-bug: #1500698

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/16988
Submitter: Nischal Sheth (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/17005
Submitter: Prakash Bailkeri (<email address hidden>)

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

Reviewed: https://review.opencontrail.org/16988
Committed: http://github.org/Juniper/contrail-controller/commit/3ccd103208ab52f7689aae2a658d06d60cce74a9
Submitter: Zuul
Branch: master

commit 3ccd103208ab52f7689aae2a658d06d60cce74a9
Author: Nischal Sheth <email address hidden>
Date: Mon Feb 8 12:25:17 2016 -0800

Trigger pending chain resolution on routing instance change (and add)

Recent change to treat a service chain as pending if the dest routing
instance doesn't have a VN index causes problems if the instance does
not have an index when it's created.

Also fix issue in service_chain_test that causes occasional failures.
Check if DBEnry returned by DBTable::Find is NULL before dynamic_cast.

Change-Id: I971443345699a7909e5247967916ddff0818f78a
Related-bug: 1500698

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

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

commit e6033719926f79ac0b50f7336366b54573ebe6d2
Author: Prakash Bailkeri <email address hidden>
Date: Mon Feb 8 21:07:51 2016 +0530

OriginVn in show route

Skip deriving origin vn if already calculated from extCommunity

Change-Id: I617d35d1cf7e027ce62c914597e7c858a6a77efb
Related-bug: #1500698

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/17341
Submitter: Nischal Sheth (<email address hidden>)

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

Reviewed: https://review.opencontrail.org/17341
Committed: http://github.org/Juniper/contrail-controller/commit/74f86d3a786556099d86ff54a69b7a8bdd88ea86
Submitter: Zuul
Branch: master

commit 74f86d3a786556099d86ff54a69b7a8bdd88ea86
Author: Nischal Sheth <email address hidden>
Date: Wed Feb 17 11:49:54 2016 -0800

Add ShowRouteAggregateSummaryReq introspect command

Change-Id: Ib633b018798e19c7fc73f4bebaef10283aef1a89
Related-bug: 1500698

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R3.0

Review in progress for https://review.opencontrail.org/17391
Submitter: Nischal Sheth (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/17424
Submitter: Vinay Vithal Mahuli (<email address hidden>)

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

Reviewed: https://review.opencontrail.org/17424
Committed: http://github.org/Juniper/contrail-controller/commit/a6be817f369ca6f2e30f2c3a39079e8a9133e5e9
Submitter: Zuul
Branch: R3.0

commit a6be817f369ca6f2e30f2c3a39079e8a9133e5e9
Author: Divakar <email address hidden>
Date: Wed Feb 17 15:22:01 2016 +0530

Correct the metadata string of fip-pool to vn

The metadata mentioned in xsd for the link between floating-ip-pool to
VN is "virtual-network-floating-ip-pool" and Agent is adding a rule
in dependency manager for VMI as "floating-ip-pool-virtual-network".
This is resulting in no InterestedEvent in dependency tracker if the
link added between VN-FIP is the last link in that VMI.

As a fix, metadata is corrected in Agent.

Change-Id: I484099d8c843493cfcdb618a3165819f9eab5d08
closes-bug: #1544788

Handle vxlan change for local_vm peer in l2 flood.

Change-Id: I3eb43c9541eec6ef3d343a8aa041b527033c24fe
Closes-bug: 1529665

Add ShowRouteAggregateSummaryReq introspect command

Change-Id: Ib633b018798e19c7fc73f4bebaef10283aef1a89
Related-bug: 1500698

database-purge request fails if analytics-api starts before cassandra

analytics-api creates cql session only on creation of AnalyticsDb
object. Therefore, if analytics-api starts before cassandra becomes
active, then database-purge request fails.

This patch ensures that the cql session is created upon purge request if
cql session is None.

Change-Id: Ic38d82cd1d614cd80af917ff205c3a63b95d6b92
Closes-Bug: #1546435

LBaaS scheduler should not use hardcoded tenant name

This fix removes the hardcoded 'admin' tenant name from
lbaas scheduler and reads the tenant from config.
It also fixes the default admin_tenant_name in svc_monitor.py
to 'admin'.
Closes-Bug: #1543296

Change-Id: If5e558e27ef6fd81d81cf56a317f39dbac2854ce

Fix error message

The VM uuid should be shown instead of the VN uuid

Change-Id: I56a65f5aabbc441139784e672babcf5e7508b880
Closes-Bug: #1546107

Adding UVE information for Health Check

Virtual Machine Interface UVE will also carry additional
information on the HealthCheck instances running on it,
its state and running status

Closes-Bug: 1546390
Change-Id: Ib6f2dac3e80544852359b8e7633343011b9f7a4e

Agent changes to accept port add requests without IP address.

This is done for both REST and thrift interface. Also add a flag to Flow info maintained
by FlowStatsCollector to indicate whether a delete was enqueued for that flow or not.

Change-Id: Ibe1cd81bcd71d42919995860e846383740bdfcdc
Closes-Bug: #1546939

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Reviewed: https://review.opencontrail.org/17391
Committed: http://github.org/Juniper/contrail-controller/commit/666d8685461e34db69407e4619aff1c47e4b48ce
Submitter: Zuul
Branch: R3.0

commit 666d8685461e34db69407e4619aff1c47e4b48ce
Author: Nischal Sheth <email address hidden>
Date: Wed Feb 17 11:49:54 2016 -0800

Add ShowRouteAggregateSummaryReq introspect command

Change-Id: Ib633b018798e19c7fc73f4bebaef10283aef1a89
Related-bug: 1500698

Revision history for this message
Nischal Sheth (nsheth) wrote :

Tracking item 5 using bug 1559811.

description: updated
Displaying first 40 and last 40 comments. View all 139 comments or add a comment.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.