update consumer (sql) not actually updating

Bug #1215483 reported by Steve Martinelli
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Identity (keystone)
Fix Released
Medium
Steve Martinelli

Bug Description

Noticed this wile developing command line support:

$ openstack consumer list
| ID | Description
| 5387ef1e888e4c1bb903ec1c7cfd500c | con1

$ openstack consumer set 5387ef1e888e4c1bb903ec1c7cfd500c --description myCon

$ openstack consumer list
| ID | Description
| 5387ef1e888e4c1bb903ec1c7cfd500c | con1

The description is never updated for the consumer. I dug around and noticed that within update_consumer(), a session was being started immediately; then another session would be started when calling _get_consumer(). This was causing issues when saving the values and closing the sessions.

Changed in keystone:
assignee: nobody → Steve Martinelli (stevemar)
Changed in keystone:
status: New → In Progress
Dolph Mathews (dolph)
Changed in keystone:
milestone: none → havana-3
importance: Undecided → Medium
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to keystone (master)

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to keystone (master)

Reviewed: https://review.openstack.org/43600
Committed: http://github.com/openstack/keystone/commit/a746527c411952bc6b8acb604e2f8fca26d8854f
Submitter: Jenkins
Branch: master

commit a746527c411952bc6b8acb604e2f8fca26d8854f
Author: Steve Martinelli <email address hidden>
Date: Sat Aug 24 20:18:49 2013 -0500

    Enable SQL tests for oauth

    Enables testing of the SQL backend for oauth.
    As a result of running the tests, I noticed that this also
    fixes an update_consumer test cases too.

    fixes bug: #1215483
    fixes bug: #1216447

    Change-Id: I206d164caa66c3211cfc216d13e3d0bab0e7d54a

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