The command 'ceilometer-polling --pollster-list' doesn't fail with the wrong pollster

Bug #1553926 reported by Zi Lian Ji
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ceilometer
Fix Released
Low
Julien Danjou

Bug Description

When executing the command 'ceilometer-polling --pollster-list' with the wrong pollster, it still execute and no error will be shown. It occupies the system resources and shows no errors.

Zi Lian Ji (jizilian)
Changed in ceilometer:
assignee: nobody → Zi Lian Ji (jizilian)
Revision history for this message
gordon chung (chungg) wrote :

what do you mean wrong pollster?

Revision history for this message
Zi Lian Ji (jizilian) wrote :

Hello Gordon

If I run the command 'ceilometer-polling --pollster-list aaa', it has no valid pollster to be loaded. But the program is still executed.

Thanks

gordon chung (chungg)
Changed in ceilometer:
importance: Undecided → Low
status: New → Triaged
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/295011

Zi Lian Ji (jizilian)
Changed in ceilometer:
status: Triaged → In Progress
Changed in ceilometer:
assignee: Zi Lian Ji (jizilian) → Julien Danjou (jdanjou)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ceilometer (master)

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

commit bb6032fd1bdabf45035bdf36d3f8c08508088b96
Author: jizilian <email address hidden>
Date: Mon Mar 14 02:58:13 2016 -0400

    'ceilometer-polling' should fail with no valid pollsters

    The command 'ceilometer-polling --pollster-list' should fail with
    invalid pollsters.

    When executing the command 'ceilometer-polling --pollster-list' with
    a wrong pollster like aaa, it still executes and no error will
    be shown. It occupies the system resources and shows no errors.

    Remove the test case for
    "ceilometer-polling --polling-namespaces ipmi" in the upgrade.sh.
    In the test environment, the ipmi is not ready. It should fail.

    Change-Id: If1f9f9fdbba1832e66d89d6b61f749790df5c2b6
    Closes-Bug: #1553926

Changed in ceilometer:
status: In Progress → Fix Released
Revision history for this message
Davanum Srinivas (DIMS) (dims-v) wrote : Fix included in openstack/ceilometer 7.0.0.0b2

This issue was fixed in the openstack/ceilometer 7.0.0.0b2 development milestone.

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.