pollster/api publish to info queue

Bug #1484695 reported by gordon chung
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ceilometer
Fix Released
Medium
Rohit Jaiswal
oslo.messaging
Fix Released
Undecided
Unassigned

Bug Description

we probably shouldn't publish to notifications.info queue. we should publish to notifications.sample... that way it'll be easier for us to handle data between telemetry and other services.

Changed in ceilometer:
assignee: nobody → Rohit Jaiswal (rohit-jaiswal-3)
Revision history for this message
Julien Danjou (jdanjou) wrote :

If you go that road, it's likely we should publish to queue based on interesting routing information. Like the sample name, etc. And allowing subscribers to pick only the samples they want, e.g. if nobody subscribes to some topic then the message are just discarded.

That's a perfect usage of AMQP but I'm not sure oslo.messaging allows that for now.

Revision history for this message
gordon chung (chungg) wrote :

sileht, for the record, when you read this, i was not the one who gave you more work :P

Revision history for this message
Rohit Jaiswal (rohit-jaiswal-3) wrote :

Looks like oslo_messaging supports message publishing at sample level: https://github.com/openstack/oslo.messaging/blob/stable/kilo/oslo_messaging/notify/notifier.py#L272

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/223206

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

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

commit 4e8e3c0feb14a99dd331d47c6d388c4bfac1a21d
Author: Rohit Jaiswal <email address hidden>
Date: Mon Sep 14 15:35:20 2015 +0000

    pollster/api now publish to sample queue

    Pollsters and Post Sample implementation
    will publish to the notifications.sample
    queue with this change.

    Change-Id: Iccb9261ee805d41ed057bd0ac72306cd6a2ddaff
    Closes-Bug: #1484695
    Closes-Bug: #1489341

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
Andy Smith (ansmith-t)
Changed in oslo.messaging:
status: New → Fix Released
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.