kill only listeners we know we don't need anymore

Bug #1496459 reported by gordon chung
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ceilometer
Fix Released
Medium
gordon chung

Bug Description

when re-balancing queues across agents, we blindly kill all existing listeners and then recreate the appropriate new set of listeners. it's entirely possible that some of the existing listeners will be consumed by current agent so we should be smarter when deciding what to kill

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to ceilometer (master)

Fix proposed to branch: master
Review: https://review.openstack.org/225270

Changed in ceilometer:
assignee: nobody → gordon chung (chungg)
status: Triaged → In Progress
gordon chung (chungg)
Changed in ceilometer:
milestone: none → liberty-rc1
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ceilometer (master)

Reviewed: https://review.openstack.org/225270
Committed: https://git.openstack.org/cgit/openstack/ceilometer/commit/?id=338e7a655d5d422dc9db878d2f0952549e54e711
Submitter: Jenkins
Branch: master

commit 338e7a655d5d422dc9db878d2f0952549e54e711
Author: gordon chung <email address hidden>
Date: Fri Sep 18 14:39:25 2015 -0400

    retain existing listeners on refresh

    there is an overhead to killing and recreating listeners. this patch
    keeps the pipeline listeners we already have after rebalancing
    rather than blindly killing and recreating all of them.

    Change-Id: Ic7c23fd7649ca0b828cc4266582163bd326c2b80
    Closes-Bug: #1496459

Changed in ceilometer:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in ceilometer:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in ceilometer:
milestone: liberty-rc1 → 5.0.0
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.