discovery fixed policy breaks if service stays down for extended period

Bug #1401304 reported by Deepinder Setia
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
Trunk
In Progress
Undecided
Deepinder Setia

Bug Description

Discovery has fixed policy for service assignment in which services are assigned to consumers in a fixed, static or constant manner. For example if there are "n" publishers of a service and there are "m" consumers that are interested in "k" instances (say 2) of service, then all "m" consumers will get <n0, n1, n2 ... nk> service instances. This is akin to priority order. If an instance, say "ni" such that 0 <= i <= k went down for an extended period (> 15 seconds) and comes back up, it should no longer be assigned to a new consumer because it should go to the bottom of the prioritized list. It should not retain its position. Current behavior of discovery does not match this expectation and "ni" service continues to get assigned to new consumers.

Deepinder Setia (dsetia)
affects: opencontrail → juniperopenstack
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

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

commit 323c3ba37bd1ff2d68a0da11ccee7c6dc39d189c
Author: Deepinder Setia <email address hidden>
Date: Wed Dec 10 15:09:27 2014 -0800

Bug 1401304: Reset sequence number when service is republished. This should push service instance to bottom of priority list

Change-Id: I499334d55db44eb274bb7e1f7b69dfd38ca83ec3

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

Review in progress for https://review.opencontrail.org/18879
Submitter: Pulkit Tandon (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/18880
Submitter: Pulkit Tandon (<email address hidden>)

Revision history for this message
Pulkit Tandon (pulkitt) wrote :

A check to verify this bug is introduced in following Automation test case:
verify_policy_fixed

File path:
/contrail-test/serail_script/discovery-resgression/test_discovery_serial.py

tags: added: automation
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.