skip bad meter while loading the meter definitions

Bug #1503433 reported by Pradeep Kilambi
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ceilometer
Fix Released
Undecided
Pradeep Kilambi

Bug Description

Currently we raise an exception if we have a bad meter definition, but this will abort loading rest of the definitions. Instead we should just skip bad ones and continue with rest

Changed in ceilometer:
status: New → In Progress
tags: added: liberty-backport
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ceilometer (master)

Reviewed: https://review.openstack.org/231671
Committed: https://git.openstack.org/cgit/openstack/ceilometer/commit/?id=5d5aaf58c8876370ff9729e08b3b41833f6e6efc
Submitter: Jenkins
Branch: master

commit 5d5aaf58c8876370ff9729e08b3b41833f6e6efc
Author: Pradeep Kilambi <email address hidden>
Date: Tue Oct 6 15:35:21 2015 -0400

    Skip bad meter definitions instead of erroring out

    Currently we raise an exception once we hit a bad
    meter definition in the yaml. Instead, skip the
    bad ones and proceed with rest of the yaml.

    Closes-Bug: #1503433

    Change-Id: I1dd5b69bf9a92b5af5692ea75010f11a863b0afc

Changed in ceilometer:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to ceilometer (stable/liberty)

Fix proposed to branch: stable/liberty
Review: https://review.openstack.org/232509

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

Reviewed: https://review.openstack.org/232509
Committed: https://git.openstack.org/cgit/openstack/ceilometer/commit/?id=5cdeeacafb835c1d395663a4f74270e03fc06a35
Submitter: Jenkins
Branch: stable/liberty

commit 5cdeeacafb835c1d395663a4f74270e03fc06a35
Author: Pradeep Kilambi <email address hidden>
Date: Tue Oct 6 15:35:21 2015 -0400

    Skip bad meter definitions instead of erroring out

    Currently we raise an exception once we hit a bad
    meter definition in the yaml. Instead, skip the
    bad ones and proceed with rest of the yaml.

    Closes-Bug: #1503433

    Change-Id: I1dd5b69bf9a92b5af5692ea75010f11a863b0afc
    (cherry picked from commit 5d5aaf58c8876370ff9729e08b3b41833f6e6efc)

tags: added: in-stable-liberty
Revision history for this message
Thierry Carrez (ttx) wrote : Fix included in openstack/ceilometer 6.0.0.0b1

This issue was fixed in the openstack/ceilometer 6.0.0.0b1 development milestone.

Thierry Carrez (ttx)
Changed in ceilometer:
status: Fix Committed → Fix Released
Revision history for this message
Doug Hellmann (doug-hellmann) wrote : Fix included in openstack/ceilometer 5.0.1

This issue was fixed in the openstack/ceilometer 5.0.1 release.

Liusheng (liusheng)
Changed in ceilometer:
milestone: none → mitaka-1
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.