Samples from net-pollster don't have all instances metadata

Bug #1496778 reported by Mehdi Abaakouk
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ceilometer
Fix Released
Undecided
Mehdi Abaakouk

Bug Description

Hi,

Samples from net-pollster don't have all instances metadata like ohter instance pollster.

Also the instance_type is the flavor id instead of the flavor name.

Cheers.

Mehdi Abaakouk (sileht)
Changed in ceilometer:
assignee: nobody → Mehdi Abaakouk (sileht)
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/224550

Changed in ceilometer:
status: New → In Progress
gordon chung (chungg)
Changed in ceilometer:
milestone: none → liberty-rc1
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ceilometer (master)

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

commit 6c1be9b93153de6ad9e2a008d94c5d5ed6087946
Author: Mehdi Abaakouk <email address hidden>
Date: Thu Sep 17 11:57:51 2015 +0200

    Use make_sample_from_instance for net-pollster

    The net-pollster doesn't have the same metadata than other pollster.

    This change fixes that.

    Also the test is modified because if the metering metadata is already
    present in instance metadata they are filtered out.

    Change-Id: I50676067636ad01599f9dd5f0c31071b86f1466c
    Closes-bug: #1496778

Changed in ceilometer:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in ceilometer:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in ceilometer:
milestone: liberty-rc1 → 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.