GET /queues returns duplicated items when pooling enabled

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

Bug Description

control plane: sqlalchemy
data plane: redis
zaqar code base: master branch
How to recreate:
1. Create a queue
2. Get /queues

{
    "queues": [
        {
            "href": "/v2/queues/fake",
            "name": "fake"
        },
        {
            "href": "/v2/queues/fake",
            "name": "fake"
        },
        {
            "href": "/v2/queues/test",
            "name": "test"
        },
        {
            "href": "/v2/queues/test",
            "name": "test"
        }
    ],
    "links": [
        {
            "href": "/v2/queues?marker=test",
            "rel": "next"
        }
    ]
}

Feilong Wang (flwang)
Changed in zaqar:
assignee: nobody → Fei Long Wang (flwang)
Feilong Wang (flwang)
Changed in zaqar:
importance: Undecided → High
Revision history for this message
Feilong Wang (flwang) wrote :

The root cause is we're saving the queues data on management plane. So it's no necessary to iterate all the pools to list queues. So if there are X pools, the queues count will be multipuled X times.

Changed in zaqar:
status: New → Triaged
status: Triaged → Confirmed
status: Confirmed → Triaged
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to zaqar (master)

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

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

Reviewed: https://review.openstack.org/221142
Committed: https://git.openstack.org/cgit/openstack/zaqar/commit/?id=3b7888c84a556466ccfcf08f4360203311a46bea
Submitter: Jenkins
Branch: master

commit 3b7888c84a556466ccfcf08f4360203311a46bea
Author: Fei Long Wang <email address hidden>
Date: Tue Sep 8 15:11:04 2015 +1200

    Fix duplicated queues on multi pools

    Now we're saving queues on management plane, so it's not necessary
    to interate all the pools to list queues. This patch fixes the issue
    and adds a test for that.

    Closes-Bug: #1490807

    Change-Id: I595f240d2944c55c2f4883bbc3070aaf25c19763

Changed in zaqar:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in zaqar:
milestone: none → liberty-rc1
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in zaqar:
milestone: liberty-rc1 → 1.0.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.