replace cast() with notification for metering data

Bug #1005933 reported by Doug Hellmann
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ceilometer
Fix Released
Wishlist
Mehdi Abaakouk

Bug Description

Passing data to the collector should not use a cast() call, since that implies that the sender knows what the receiver is going to do with the data. We should use a model more like notifications, which send the data without any expectations on the receiver beyond agreeing to what the message format is.

Changed in ceilometer:
assignee: nobody → Doug Hellmann (doug-hellmann)
tags: added: agent collector effort-l
Revision history for this message
Doug Hellmann (doug-hellmann) wrote :

This ticket depends on bug #1005935

Changed in ceilometer:
status: New → Triaged
importance: Undecided → Wishlist
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/80225

Changed in ceilometer:
assignee: Doug Hellmann (doug-hellmann) → Mehdi Abaakouk (sileht)
status: Triaged → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ceilometer (master)

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

commit fec77dbc85fed701fdbc3a96bae57e5ae3a705cb
Author: Mehdi Abaakouk <email address hidden>
Date: Tue Jul 29 12:12:06 2014 +0200

    Permit usage of notifications for metering

    This patch introduces a new publisher that uses notification instead
    of RPC.

    And set it by default.

    Closes bug: #1005933
    Implements blueprint replace-rpc-cast-with-notifications
    Co-Authored-By: Ala Rezmerita <email address hidden>

    Change-Id: Idc40c148ef60d5e1349d30c66ba85691d93c5675

Changed in ceilometer:
status: In Progress → Fix Committed
Eoghan Glynn (eglynn)
Changed in ceilometer:
milestone: none → juno-3
Thierry Carrez (ttx)
Changed in ceilometer:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in ceilometer:
milestone: juno-3 → 2014.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.