Large value of period causes 500 error

Bug #1396223 reported by Srinivas Sakhamuri
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ceilometer
Fix Released
Medium
Srinivas Sakhamuri
Juno
Fix Released
Medium
Eoghan Glynn

Bug Description

The large value of period causes 500 error when querying statistics with large value for period. This is a bad user experience and misleading. Appropriate error message should be returned in this case.

Changed in ceilometer:
assignee: nobody → Srinivas Sakhamuri (srinivas-sakhamuri)
Changed in ceilometer:
status: New → In Progress
Eoghan Glynn (eglynn)
Changed in ceilometer:
milestone: none → kilo-1
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ceilometer (master)

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

commit 9f923ae928176ca36ad784fadeea56bfc806e7e1
Author: Srinivas Sakhamuri <email address hidden>
Date: Fri Nov 14 00:51:19 2014 +0000

    Internal error with period overflow

    When statistics queried with large period value the overflow
    causes error 500. This patch converts the overflow error to
    ClientSideError which is returned as 400 bad input error

    Change-Id: Iacf89c75428713a151fbe81aa1601df63785f942
    Closes-Bug: 1396223

Changed in ceilometer:
status: In Progress → Fix Committed
Eoghan Glynn (eglynn)
Changed in ceilometer:
importance: Undecided → Medium
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to ceilometer (stable/juno)

Fix proposed to branch: stable/juno
Review: https://review.openstack.org/138322

Eoghan Glynn (eglynn)
tags: added: juno-backport-potential
Alan Pevec (apevec)
tags: removed: juno-backport-potential
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ceilometer (stable/juno)

Reviewed: https://review.openstack.org/138322
Committed: https://git.openstack.org/cgit/openstack/ceilometer/commit/?id=43718e84b93bd15353e4cce8002a974553bd1bbc
Submitter: Jenkins
Branch: stable/juno

commit 43718e84b93bd15353e4cce8002a974553bd1bbc
Author: Srinivas Sakhamuri <email address hidden>
Date: Fri Nov 14 00:51:19 2014 +0000

    Internal error with period overflow

    When statistics queried with large period value the overflow
    causes error 500. This patch converts the overflow error to
    ClientSideError which is returned as 400 bad input error

    Change-Id: Iacf89c75428713a151fbe81aa1601df63785f942
    Closes-Bug: 1396223
    (cherry picked from commit 9f923ae928176ca36ad784fadeea56bfc806e7e1)

Thierry Carrez (ttx)
Changed in ceilometer:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in ceilometer:
milestone: kilo-1 → 2015.1.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.