RPC publisher shouldn't block if transport layer is down

Bug #1189488 reported by Julien Danjou
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ceilometer
Fix Released
Medium
Mehdi Abaakouk

Bug Description

The current RPC publisher might block any program trying to publisher to an AMQP server if it fails to publisher.

It should handle its inability to publish with either dropping the counter or storing in a local queue until it's possible to publish.

Julien Danjou (jdanjou)
Changed in ceilometer:
importance: Undecided → Medium
status: New → Triaged
Julien Danjou (jdanjou)
summary: - RPC publisher shouldn't block if transport layer id own
+ RPC publisher shouldn't block if transport layer is down
Mehdi Abaakouk (sileht)
Changed in ceilometer:
assignee: nobody → Mehdi Abaakouk (sileht)
Changed in ceilometer:
status: Triaged → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ceilometer (master)

Reviewed: https://review.openstack.org/35604
Committed: http://github.com/openstack/ceilometer/commit/3f044dd1ace2da3dba65fdb9017b878ce902a554
Submitter: Jenkins
Branch: master

commit 3f044dd1ace2da3dba65fdb9017b878ce902a554
Author: Mehdi Abaakouk <email address hidden>
Date: Thu Jul 4 09:30:43 2013 +0200

    publisher.rpc: queing policies

    This change allow to configure the behavior of rpc publisher
    when rabbitmq is down.

    3 policies are available:
    - default: wait until it comes back if rabbit_max_retries <= 0
               raise a exception if rabbit_max_retries > 0
    - drop: don't publish the samples
    - queue: create a local queue of 'max_queue_length' samples

    Configuration of the policy is done via the publisher url
    example: rpc://?policy=queue&max_queue_length=100

    Change-Id: I41c9be9e1c760db1155711325434a1877e6dd7b2
    Fixes: bug#1192918 bug#1189488

Changed in ceilometer:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in ceilometer:
milestone: none → havana-2
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in ceilometer:
milestone: havana-2 → 2013.2
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.