Can't get the unit of a metric

Bug #1530967 reported by Sergio Colinas
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Gnocchi
Fix Released
Low
Julien Danjou

Bug Description

When you call metric show, it did't show the metric unit.

nathan (nathan95-b)
Changed in gnocchi:
status: New → Opinion
status: Opinion → Incomplete
assignee: nobody → nathan (nathan95-b)
assignee: nathan (nathan95-b) → nobody
Revision history for this message
Sergio Colinas (sergio-n) wrote :

Each metric should have a unit where it measurements are expressed (for example MB, %, instance, B/s, ns, etc). The unit should be included in metric show method.

Julien Danjou (jdanjou)
Changed in gnocchi:
status: Incomplete → Triaged
importance: Undecided → Low
Revision history for this message
gordon chung (chungg) wrote :

i took a look at a few other tsdb, some support unit through tags (kairosdb, influxdb), others have it as native field (kdb+). speaking with jd, i'm inclined to say just make it a native field since it's easier. also, cadf expects unit always be present for metrics so it works on enterprise level.

Changed in gnocchi:
assignee: nobody → zhangguoqing (474751729-o)
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/312489

Changed in gnocchi:
status: Triaged → In Progress
Changed in gnocchi:
assignee: zhangguoqing (474751729-o) → Julien Danjou (jdanjou)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to gnocchi (master)

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

commit d74ea92ebe40e59b251b5c543cc7a2a73a8bb2ef
Author: zhangguoqing <zhang.guoqing@99cloud.net>
Date: Wed May 4 12:26:20 2016 +0000

    add unit column for metric

    Change-Id: I9cdf83f1ec51670e74a11125ac9f048bf9d70d84
    Closes-Bug: #1530967

Changed in gnocchi:
status: In Progress → Fix Committed
Revision history for this message
Doug Hellmann (doug-hellmann) wrote : Fix included in openstack/gnocchi 2.2.0

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

Julien Danjou (jdanjou)
Changed in gnocchi:
status: Fix Committed → Fix Released
milestone: none → 2.2.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.