drop rpc functionality

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

Bug Description

the collector supports udp, rpc and notifier publishing. i'm not sure rpc offers anything more than notifier. it is significantly slower... do we need to keep it around?

Revision history for this message
Chris Dent (cdent) wrote :

I had a vague impression that rpc had a strong reliability guarantee? If not then yes we should totally kill it but I guess we'll need a deprecation plan.

Revision history for this message
Julien Danjou (jdanjou) wrote :

We need to remove it, but we need to deprecate it because I know people are using it with their own collector (yeah even if it was a bad idea).

Changed in ceilometer:
status: New → Confirmed
importance: Undecided → Medium
Revision history for this message
gordon chung (chungg) wrote :

i'm not sure it offers any more reliability vs queues. it's really only intended when publisher expects a response, which we don't need.

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

Related fix proposed to branch: master
Review: https://review.openstack.org/211304

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to ceilometer (master)

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

commit a27643a75ff985c0d61f3c10ad6cdce109845bf9
Author: gordon chung <email address hidden>
Date: Mon Aug 10 15:28:29 2015 -0400

    start rpc deprecation

    we switch to work queues in Juno and made it the default rather
    than rpc. the main advantage of using RPC is it offers callback
    functionality at the expense of performance. we don't need callback
    for our use cases so it's best to stop supporting rpc and leave just
    udp and queues in our collector.

    Change-Id: Ia02421ae09cdf27998ce76a99170100f36595f71
    Related-Bug: #1479820

Lianhao Lu (lianhao-lu)
Changed in ceilometer:
assignee: nobody → gordon chung (chungg)
status: Confirmed → Fix Committed
milestone: none → liberty-3
Thierry Carrez (ttx)
Changed in ceilometer:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in ceilometer:
milestone: liberty-3 → 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.