Unit msimatch between "network.services.lb.health_monitor" pollster and notification

Bug #1460696 reported by Yurii Prokulevych
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ceilometer
Fix Released
Undecided
Pradeep Kilambi
Kilo
Fix Released
Undecided
Unassigned

Bug Description

Meter "network.services.lb.health_monitor" can be obtained via notification and vai pollster.
But those to methods are using different 'Unit' for the counter:

ceilometer meter-list -q 'resource_id=83c2dd22-7e7d-43be-a53b-c9bedc06db51'
+-------------------------------------------+-------+----------------+--------------------------------------+---------+----------------------------------+
| Name | Type | Unit | Resource ID | User ID | Project ID |
+-------------------------------------------+-------+----------------+--------------------------------------+---------+----------------------------------+
| network.services.lb.health_monitor | gauge | health_monitor | 83c2dd22-7e7d-43be-a53b-c9bedc06db51 | None | 881bb85f8f2f4fe59151b56064b1d6c4 |
| network.services.lb.health_monitor | gauge | monitor | 83c2dd22-7e7d-43be-a53b-c9bedc06db51 | None | 881bb85f8f2f4fe59151b56064b1d6c4 |
| network.services.lb.health_monitor.create | delta | health_monitor | 83c2dd22-7e7d-43be-a53b-c9bedc06db51 | None | 881bb85f8f2f4fe59151b56064b1d6c4 |
+-------------------------------------------+-------+----------------+--------------------------------------+---------+----------------------------------+

ceilometer sample-list -m network.services.lb.health_monitor -q 'resource_id=83c2dd22-7e7d-43be-a53b-c9bedc06db51'
+--------------------------------------+------------------------------------+-------+--------+----------------+----------------------------+
| Resource ID | Name | Type | Volume | Unit | Timestamp |
+--------------------------------------+------------------------------------+-------+--------+----------------+----------------------------+
| 83c2dd22-7e7d-43be-a53b-c9bedc06db51 | network.services.lb.health_monitor | gauge | 1.0 | monitor | 2015-06-01T13:59:17.093000 |
| 83c2dd22-7e7d-43be-a53b-c9bedc06db51 | network.services.lb.health_monitor | gauge | 1.0 | monitor | 2015-06-01T13:58:17.012000 |
| 83c2dd22-7e7d-43be-a53b-c9bedc06db51 | network.services.lb.health_monitor | gauge | 1.0 | monitor | 2015-06-01T13:57:17.009000 |
| 83c2dd22-7e7d-43be-a53b-c9bedc06db51 | network.services.lb.health_monitor | gauge | 1.0 | monitor | 2015-06-01T13:56:17.014000 |
| 83c2dd22-7e7d-43be-a53b-c9bedc06db51 | network.services.lb.health_monitor | gauge | 1.0 | monitor | 2015-06-01T13:55:17.083000 |
| 83c2dd22-7e7d-43be-a53b-c9bedc06db51 | network.services.lb.health_monitor | gauge | 1.0 | monitor | 2015-06-01T13:54:17.007000 |
| 83c2dd22-7e7d-43be-a53b-c9bedc06db51 | network.services.lb.health_monitor | gauge | 1.0 | monitor | 2015-06-01T13:53:17.078000 |
| 83c2dd22-7e7d-43be-a53b-c9bedc06db51 | network.services.lb.health_monitor | gauge | 1.0 | monitor | 2015-06-01T13:52:17.230000 |
| 83c2dd22-7e7d-43be-a53b-c9bedc06db51 | network.services.lb.health_monitor | gauge | 1.0 | health_monitor | 2015-06-01T13:51:50.446000 |
+--------------------------------------+------------------------------------+-------+--------+----------------+----------------------------+

For consistency they should have the same "Unit".

Packages:
openstack-ceilometer-alarm-2015.1.0-2.el7ost.noarch
openstack-ceilometer-compute-2015.1.0-2.el7ost.noarch
openstack-ceilometer-common-2015.1.0-2.el7ost.noarch
python-ceilometerclient-1.0.13-1.el7ost.noarch
openstack-ceilometer-notification-2015.1.0-2.el7ost.noarch
openstack-ceilometer-central-2015.1.0-2.el7ost.noarch
python-ceilometer-2015.1.0-2.el7ost.noarch
openstack-ceilometer-api-2015.1.0-2.el7ost.noarch
openstack-ceilometer-collector-2015.1.0-2.el7ost.noarch

Thanks,
Yurii

Changed in ceilometer:
assignee: nobody → Pradeep Kilambi (pkilambi)
Revision history for this message
Yurii Prokulevych (yprokule) wrote :

Hi Pradeep,

network.services.firewall.policy meter also has unit's mismatch between pollster and notification:
ceilometer sample-list -m network.services.firewall.policy -q 'resource_id=84dd2fff-c1f3-4ae0-a453-e1d189788ab7'
+--------------------------------------+----------------------------------+-------+--------+-----------------+----------------------------+
| Resource ID | Name | Type | Volume | Unit | Timestamp |
+--------------------------------------+----------------------------------+-------+--------+-----------------+----------------------------+
| 84dd2fff-c1f3-4ae0-a453-e1d189788ab7 | network.services.firewall.policy | gauge | 1.0 | firewall_policy | 2015-06-09T14:59:23.514000 |
| 84dd2fff-c1f3-4ae0-a453-e1d189788ab7 | network.services.firewall.policy | gauge | 1.0 | firewall_policy | 2015-06-09T14:59:23.461000 |
| 84dd2fff-c1f3-4ae0-a453-e1d189788ab7 | network.services.firewall.policy | gauge | 1.0 | policy | 2015-06-09T14:52:40.668000 |
| 84dd2fff-c1f3-4ae0-a453-e1d189788ab7 | network.services.firewall.policy | gauge | 1.0 | policy | 2015-06-09T14:51:40.747000 |
| 84dd2fff-c1f3-4ae0-a453-e1d189788ab7 | network.services.firewall.policy | gauge | 1.0 | policy | 2015-06-09T14:50:40.673000 |
| 84dd2fff-c1f3-4ae0-a453-e1d189788ab7 | network.services.firewall.policy | gauge | 1.0 | policy | 2015-06-09T14:49:40.980000 |
+--------------------------------------+----------------------------------+-------+--------+-----------------+----------------------------+

Thanks,
Yurii

Changed in ceilometer:
status: New → In Progress
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/189802

Changed in ceilometer:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in ceilometer:
milestone: none → liberty-1
status: Fix Committed → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to ceilometer (stable/kilo)

Fix proposed to branch: stable/kilo
Review: https://review.openstack.org/194732

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ceilometer (stable/kilo)

Reviewed: https://review.openstack.org/194732
Committed: https://git.openstack.org/cgit/openstack/ceilometer/commit/?id=b28a8bb5628d2bacdd494931b122d7310082e46c
Submitter: Jenkins
Branch: stable/kilo

commit b28a8bb5628d2bacdd494931b122d7310082e46c
Author: Pradeep Kilambi <email address hidden>
Date: Tue Jun 9 12:10:49 2015 -0400

    Fix the meter unit types to be consistent

    The unit types for certain meters generated from
    pollsters are not consistent with the once generated
    via notifications. This fix handles the mismatch.

    DocImpact
    Closes-Bug: #1460696

    Change-Id: Ibd82fa4a5a4e66abaf2a6d0bbe18df4fbfac3ffb
    (cherry picked from commit 7dfb37ea4014669f84451e6ddca3fb61872716d7)

tags: added: in-stable-kilo
Thierry Carrez (ttx)
Changed in ceilometer:
milestone: liberty-1 → 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.