Limit number of reach/unreach routes in xmpp message

Bug #1457679 reported by Nischal Sheth
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R2.0
Fix Committed
Wishlist
Nischal Sheth
R2.1
Fix Released
Wishlist
Nischal Sheth
R2.20
Fix Released
Wishlist
Nischal Sheth
Trunk
Fix Committed
Wishlist
Nischal Sheth

Bug Description

If there are a large number of routes with the same attribute set
CN packs all of them into a single, potentially very large update
message. This doesn't happen for bgp since the packing process is
limited by the maximum bgp message size.

Will implement following artificial limits for BgpXmppMessage:

kMaxReachCount = 32
kMaxUnreachCount = 256

Nischal Sheth (nsheth)
description: updated
Nischal Sheth (nsheth)
description: updated
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : R2.0

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

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

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

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

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

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

Review in progress for https://review.opencontrail.org/10718
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/10716
Committed: http://github.org/Juniper/contrail-controller/commit/29fe4ab64b9f59f49b0bc348fb1f4c244d4e197d
Submitter: Zuul
Branch: R2.1

commit 29fe4ab64b9f59f49b0bc348fb1f4c244d4e197d
Author: Nischal Sheth <email address hidden>
Date: Fri May 22 09:13:19 2015 -0700

Limit number of reach/unreach routes in xmpp message

Change-Id: I824e888f9554b77c543d540de12fa7c53276c51e
Closes-Bug: 1457679

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

Reviewed: https://review.opencontrail.org/10717
Committed: http://github.org/Juniper/contrail-controller/commit/4edcccd80fdf410091d89ae442b06e752436fcf7
Submitter: Zuul
Branch: R2.20

commit 4edcccd80fdf410091d89ae442b06e752436fcf7
Author: Nischal Sheth <email address hidden>
Date: Fri May 22 09:13:19 2015 -0700

Limit number of reach/unreach routes in xmpp message

Change-Id: I824e888f9554b77c543d540de12fa7c53276c51e
Closes-Bug: 1457679

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

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

commit a1b1a9815c31c64e4adebcd7ff9d8b6bb7a4dce1
Author: Nischal Sheth <email address hidden>
Date: Fri May 22 09:13:19 2015 -0700

Limit number of reach/unreach routes in xmpp message

Change-Id: I824e888f9554b77c543d540de12fa7c53276c51e
Closes-Bug: 1457679

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

Review in progress for https://review.opencontrail.org/10718
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/10718
Committed: http://github.org/Juniper/contrail-controller/commit/26fc5488de39a705e0aa99f62aab6f8111399668
Submitter: Zuul
Branch: master

commit 26fc5488de39a705e0aa99f62aab6f8111399668
Author: Nischal Sheth <email address hidden>
Date: Tue May 19 22:36:03 2015 -0700

Limit number of reach/unreach routes in xmpp message

Change-Id: Ib0cfa49585b6aa6ae3afa8052de6085a9a8bc0f4
Closes-Bug: 1457679

To post a comment you must log in.
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.