--limit can collide with query string on some commands
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Ceilometer |
Fix Released
|
Medium
|
Chris Dent | ||
python-ceilometerclient |
Invalid
|
Medium
|
Rohit Jaiswal |
Bug Description
Using the new 1.5.0 there is some inconsisten behavior on some commands when trying to (naively) mix the use of the --limit command with a limit query. Some examples:
* ceilometer event-list --limit=1 # returns one item
* ceilometer event-list -q limit=1 # returns an empty list
* ceilometer resource-list --limit=1 # returns one item
* ceilometer resource-list -q limit=1 # errors as follows
get_resources() got multiple values for keyword argument 'limit' (HTTP 500) (Request-ID: req-79132b9a-
* ceilometer meter-list --limit =1 # returns one item
* ceilometer meter-list -q limit=1 # errors the same way as resource-list
* ceilometer sample-list --limit =1 # returns one item
* ceilometer sample-list -q limit=1 # errors as follows
Unknown argument: "limit": unrecognized field in query: [<Query u'limit' eq u'1' >], valid keys: ['message_id', 'meter', 'project', 'resource', 'search_offset', 'source', 'timestamp', 'user'] (HTTP 400) (Request-ID: req-9feebef4-
There seem to be at least two bugs here:
* the client is not constructing the right query based on its input, or at least producing an error locally before going to the server
* the server is producing a 500 when it should be producing a 400 when it gets multiple limits
Changed in ceilometer: | |
assignee: | nobody → Chris Dent (chdent) |
Changed in python-ceilometerclient: | |
importance: | Undecided → Medium |
Changed in python-ceilometerclient: | |
assignee: | nobody → Rohit Jaiswal (rohit-jaiswal-3) |
Changed in python-ceilometerclient: | |
status: | New → Incomplete |
Changed in python-ceilometerclient: | |
status: | Incomplete → Invalid |
Changed in ceilometer: | |
milestone: | none → liberty-rc1 |
Changed in ceilometer: | |
importance: | Undecided → Medium |
Changed in ceilometer: | |
status: | Fix Committed → Fix Released |
Changed in ceilometer: | |
milestone: | liberty-rc1 → 5.0.0 |
Fix proposed to branch: master /review. openstack. org/222169
Review: https:/