Ceph: metric container object is useless

Bug #1614965 reported by Julien Danjou
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Gnocchi
Fix Released
Low
gordon chung

Bug Description

Each metric has a "gnocchi_<id>_container" object to indicate it has been created. Relying on the "gnocchi_<id>_none" object of the unaggregated timeserie for the same purpose would require one less object per metric.

Julien Danjou (jdanjou)
Changed in gnocchi:
status: New → Triaged
importance: Undecided → Low
Revision history for this message
xiaozhuangqing (xiaozhuangqing) wrote :

Hi,Jd__, what you mean Relying on the "gnocchi_<id>_none" object of the unaggregated timeserie for the same ?

Revision history for this message
xiaozhuangqing (xiaozhuangqing) wrote :

do you mean remove metric container object ? I hope I can do this job.^^

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

Yeah I have the feeling we don't need the "container" object that is been created by the driver to assess that a metric exists. The unaggregated timeseries (named "none) should be enough.

gordon chung (chungg)
Changed in gnocchi:
assignee: nobody → gordon chung (chungg)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to gnocchi (master)

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

Changed in gnocchi:
status: Triaged → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to gnocchi (master)

Reviewed: https://review.openstack.org/398574
Committed: https://git.openstack.org/cgit/openstack/gnocchi/commit/?id=ac7ae99f7ef325563875c26656cf63790b341f82
Submitter: Jenkins
Branch: master

commit ac7ae99f7ef325563875c26656cf63790b341f82
Author: gord chung <email address hidden>
Date: Wed Nov 16 20:36:25 2016 +0000

    drop gnocchi_<id>_container object

    we just use filestore properties in gnocchi_<id>_container object.
    the gnocchi_<id>_none object can handle this as it's only being used
    for object content.

    Change-Id: Ia26638a007be2df5641b9828ce0a60065ee8a54a
    Closes-Bug: #1614965

Changed in gnocchi:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/gnocchi 3.1.0

This issue was fixed in the openstack/gnocchi 3.1.0 release.

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.