Inability to remove name or description from volume type

Bug #1486730 reported by Joel Coffman
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
In Progress
Undecided
Tushar Trambak Gite

Bug Description

The SQLAlchemy volume_type_update function (http://git.openstack.org/cgit/openstack/cinder/tree/cinder/db/sqlalchemy/api.py#n2294) has several issues:
1. possible key errors if 'name' and 'description' are not in the values dictionary passed as a parameter,
2. the inability to remove an existing name or description, and
3. retrieving the updated data without first flushing the session.
In practice, the first issue is currently guarded against via checks in other modules so the values dictionary always contains 'name' and 'description' keys, but the second issue effectively precludes removing an existing name or description, which should be possible.

Changed in cinder:
assignee: nobody → Joel Coffman (joel-coffman)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to cinder (master)

Related fix proposed to branch: master
Review: https://review.openstack.org/219007

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on cinder (master)

Change abandoned by Sean McGinnis (<email address hidden>) on branch: master
Review: https://review.openstack.org/213829
Reason: This review is > 4 weeks without comment and currently blocked by a core reviewer with a -2. We are abandoning this for now. Feel free to reactivate the review by pressing the restore button and contacting the reviewer with the -2 on this review to ensure you address their concerns.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Change abandoned by Joel Coffman (<email address hidden>) on branch: master
Review: https://review.openstack.org/213829

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Change abandoned by Joel Coffman (<email address hidden>) on branch: master
Review: https://review.openstack.org/219007

Revision history for this message
Sean McGinnis (sean-mcginnis) wrote : Bug Assignee Expired

Unassigning due to no activity for > 6 months.

Changed in cinder:
assignee: Joel Coffman (joel-coffman) → nobody
Changed in cinder:
status: In Progress → New
Changed in cinder:
assignee: nobody → Tushar Trambak Gite (tushargite96)
Changed in cinder:
status: New → In Progress
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.