Can't get the unit of a metric

Bug #1530967 reported by Sergio Colinas on 2016-01-04
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) on 2016-01-04
Changed in gnocchi:
status: New → Opinion
status: Opinion → Incomplete
assignee: nobody → nathan (nathan95-b)
assignee: nathan (nathan95-b) → nobody
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) on 2016-01-05
Changed in gnocchi:
status: Incomplete → Triaged
importance: Undecided → Low
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)

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)

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

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

Julien Danjou (jdanjou) on 2016-08-02
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  Edit
Everyone can see this information.

Other bug subscribers