Make limit=0 has same behaviour in redis

Bug #1590182 reported by Feilong Wang
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
zaqar
Fix Released
Medium
wanghao

Bug Description

limit=0 in Mongo means unlimit, but in redis when we doing list, it has different behaviour. We need to fix this to make sure all the drivers are consistent.

Feilong Wang (flwang)
tags: added: low-hanging-fruit
Changed in zaqar:
importance: Undecided → Medium
Changed in zaqar:
assignee: nobody → Annapoornima Koppad (annakoppad)
Changed in zaqar:
assignee: Annapoornima Koppad (annakoppad) → nobody
gaojie (gaojie)
Changed in zaqar:
assignee: nobody → gaojie (gaojie)
Revision history for this message
wanghao (wanghao749) wrote :
Changed in zaqar:
assignee: gaojie (gaojie) → wanghao (wanghao749)
Changed in zaqar:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to zaqar (master)

Reviewed: https://review.openstack.org/423023
Committed: https://git.openstack.org/cgit/openstack/zaqar/commit/?id=8477f95d59037b00a8197f8d443c1f5b28d26560
Submitter: Jenkins
Branch: master

commit 8477f95d59037b00a8197f8d443c1f5b28d26560
Author: wanghao <email address hidden>
Date: Fri Jan 20 11:25:04 2017 +0800

    Add validation of limit value in querying flavors and poolss

    Now queues and subscriptions have a constraint that
    limit arg in API must be bigger than 0.

    But flavor and pool have missed it, so added this validation.

    Change-Id: I9cc5146b7149adb0f7592eea15ff21e7fb3549f2
    Closes-Bug: #1590182

Changed in zaqar:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/zaqar 4.0.0.0b3

This issue was fixed in the openstack/zaqar 4.0.0.0b3 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.