discovery server not load balancing newly added publishers

Bug #1459473 reported by Deepinder Setia
12
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R2.20
Fix Committed
High
Deepinder Setia
Trunk
Fix Committed
High
Deepinder Setia

Bug Description

If new publishers get added, discovery server does not start assigning them to existing subscribers. The fix is two fold - discovery server should start assigning new servers to existing subscribers as existing subscriptions expire. Also the subscribers should accept the new assignment even though existing assignments are fully functional and healthy.

Tags: discovery
Deepinder Setia (dsetia)
Changed in opencontrail:
assignee: nobody → Deepinder Setia (dsetia)
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/12333
Submitter: Deepinder Setia (<email address hidden>)

Revision history for this message
Nipa (nipak) wrote :

On client side, for phase-1 of quench we will implement the following -

1) Client will honor list of services returned by the server.
2) Clients will apply the services returned by discovery gracefully.
3) If Discovery returns an empty list, the client will continue using the older connections.
4) If Discovery returns a smaller list of services, client will use the shorter list and continue using
     services from older list - i,e client will not CLEANUP the additional existing connections.
5) Note, clients on discovering services DOWN, will wait fot atleast heartbeat detect DOWN
     time i.e 3 keepalives (= 15 secs) before re-subscribing to get new set of services.

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

Review in progress for https://review.opencontrail.org/12333
Submitter: Deepinder Setia (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/12557
Submitter: Deepinder Setia (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/12333
Submitter: Deepinder Setia (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/12600
Submitter: Deepinder Setia (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/12609
Submitter: Deepinder Setia (<email address hidden>)

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

Reviewed: https://review.opencontrail.org/12609
Committed: http://github.org/Juniper/contrail-controller/commit/07e421a4d7ac419dc2cc64cf7ee6137cc9249dc5
Submitter: Zuul
Branch: R2.20

commit 07e421a4d7ac419dc2cc64cf7ee6137cc9249dc5
Author: Deepinder Setia <email address hidden>
Date: Fri Jul 24 02:12:58 2015 -0700

bug 1459473: Add off-line command to load-balance

Change-Id: Iee2b8d420e04125819375cfb3336b913311f533f

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

Review in progress for https://review.opencontrail.org/12633
Submitter: Deepinder Setia (<email address hidden>)

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

Reviewed: https://review.opencontrail.org/12633
Committed: http://github.org/Juniper/contrail-controller/commit/5fb301aa6e968201408f35b615772be77861b80b
Submitter: Zuul
Branch: R2.20

commit 5fb301aa6e968201408f35b615772be77861b80b
Author: Deepinder Setia <email address hidden>
Date: Sun Jul 26 15:22:56 2015 -0700

Related-Bug: #1459473 - add unit test cases

Change-Id: I15968ffc1c2d7959f77e5008a111bfc6ae202634

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

Review in progress for https://review.opencontrail.org/12789
Submitter: Nipa Kumar (<email address hidden>)

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

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

commit f4d3d30ef36a434b5052db626a46a2eca3b919b8
Author: Nipa Kumar <email address hidden>
Date: Fri Jul 31 16:18:14 2015 -0700

Vrouter changes to honor the new set of services sent by discovery irrespective of the
current state of connection to control-node or dns-server.

a) Do not send update connection info after deleting the connection info, this results in the
library creating a new connection info ds. (not_ready event is triggered after connection
info is deleted)

b) Do not remove stale entries as we want to keep working with older entries in case the
discovery server returns a smaller list of servers or empty server list.

Change-Id: I1fdc0a4fd1fbafea9b225ca94699060f9e866e00
Partial-Bug: #1459473

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

Review in progress for https://review.opencontrail.org/12835
Submitter: Deepinder Setia (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/13044
Submitter: Deepinder Setia (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/12835
Submitter: Deepinder Setia (<email address hidden>)

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

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

commit 296e1a86c9facd79ec23d7100de7e4af2fb69996
Author: Deepinder Setia <email address hidden>
Date: Fri Jul 24 02:12:58 2015 -0700

Closes-Bug: #1459473: Add off-line command to load-balance

Change-Id: Iee2b8d420e04125819375cfb3336b913311f533f

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

Review in progress for https://review.opencontrail.org/13044
Submitter: Deepinder Setia (<email address hidden>)

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

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

commit bd7be496e0a262b1e3d48024e69b6a16b6b984aa
Author: Deepinder Setia <email address hidden>
Date: Sun Jul 26 15:22:56 2015 -0700

Related-Bug: #1459473 - add unit test cases

Change-Id: I15968ffc1c2d7959f77e5008a111bfc6ae202634

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

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

Changed in opencontrail:
importance: Undecided → High
tags: added: discovery
Nipa (nipak)
no longer affects: opencontrail
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.