No type check for groupby in statistics

Bug #1291935 reported by Stéphane Albert
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
python-ceilometerclient
Fix Released
Undecided
Stéphane Albert

Bug Description

The groupby parameter is not checked for its type. As a matter of fact, if it's a string, it gets expanded as multiple groupby parameters for every characters.

Changed in python-ceilometerclient:
assignee: nobody → Stéphane Albert (sheeprine)
description: updated
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to python-ceilometerclient (master)

Fix proposed to branch: master
Review: https://review.openstack.org/80256

Changed in python-ceilometerclient:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to python-ceilometerclient (master)

Reviewed: https://review.openstack.org/80256
Committed: https://git.openstack.org/cgit/openstack/python-ceilometerclient/commit/?id=f032b7ef848d248f5917ea30a2caaa7f6d4e6822
Submitter: Jenkins
Branch: master

commit f032b7ef848d248f5917ea30a2caaa7f6d4e6822
Author: Stéphane Albert <email address hidden>
Date: Thu Mar 13 12:42:06 2014 +0100

    Statistics groupby handling improvement

    The groupby parameter for statistics now handle single string as a
    parameter. It fixes the bug where every characters of the string where
    interpreted as a groupby value.

    Change-Id: Ia763741d7ad5717be381b0ac2f0acb57e9f46cac
    Closes-Bug: #1291935

Changed in python-ceilometerclient:
status: In Progress → Fix Committed
Changed in python-ceilometerclient:
milestone: none → 1.3.0
status: Fix Committed → Fix Released
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.