update consumer (sql) not actually updating

Bug #1215483 reported by Steve Martinelli on 2013-08-22
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Identity (keystone)
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) on 2013-08-24
Changed in keystone:
milestone: none → havana-3
importance: Undecided → Medium

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) on 2013-09-05
Changed in keystone:
status: Fix Committed → Fix Released
Thierry Carrez (ttx) on 2013-10-17
Changed in keystone:
milestone: havana-3 → 2013.2
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers