Got 503 when get queue with sqlalchemy

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

Bug Description

When using sql as the management plane and get queue, user will run into 503 error since we're using self.driver.get, but unfortunately, there is no get() method in sqlalchemy's ControlDriver.

[1] https://github.com/openstack/zaqar/blob/master/zaqar/storage/sqlalchemy/queues.py#L66
[2] https://github.com/openstack/zaqar/blob/master/zaqar/storage/sqlalchemy/driver.py

Feilong Wang (flwang)
Changed in zaqar:
importance: Undecided → Low
assignee: nobody → Fei Long Wang (flwang)
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/218083

Changed in zaqar:
status: New → In Progress
tags: added: liberty-test-day
tags: added: zaqar-liberty-test-day
removed: liberty-test-day
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to zaqar (master)

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

commit b03cceec2f8231218986e5de2f77e72f5e448f05
Author: Fei Long Wang <email address hidden>
Date: Fri Aug 28 16:13:21 2015 +1200

    Fix GET queue metadata for sqlalchemy

    When using sql as the management plane and get queue, user
    will run into 503 error since we're using self.driver.get,
    but unfortunately, there is no get() method in sqlalchemy's
    ControlDriver. This patch fixes the issue by using run()
    instead of get().

    Closes-Bug: #1489701

    Change-Id: I064e123eae3617861804092f4f7a1b194a5ec136

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