Cinder

NFS driver have some useless return values and commented out code to remove

Bug #1037622 reported by Ben Swartzlander
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
Medium
Ben Swartzlander
Cinder 2012.2 "folsom"

Bug Description

cinder.volume.NfsDriver.delete_volume() returns True when it should return nothing.
Also, that method has some commented out code, which should be deleted.

Ben Swartzlander (bswartz)
Changed in cinder:
assignee: nobody → Ben Swartzlander (bswartz)
John Griffith (john-griffith)
Changed in cinder:
milestone: none → folsom-rc1
importance: Undecided → Medium
John Griffith (john-griffith)
Changed in cinder:
status: New → Confirmed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to cinder (master)

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

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

Reviewed: https://review.openstack.org/12161
Committed: http://github.com/openstack/cinder/commit/198a91a5e447915beca9585546cb92ab956fb0e6
Submitter: Jenkins
Branch: master

commit 198a91a5e447915beca9585546cb92ab956fb0e6
Author: Ben Swartzlander <email address hidden>
Date: Wed Aug 29 19:10:11 2012 -0400

    Remove unused return values and commented out code from NFS driver

    bug 1037622

    Change-Id: Idea3005ad94fd6bfc76fd0f3969ee78e3031767e

Changed in cinder:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in cinder:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in cinder:
milestone: folsom-rc1 → 2012.2
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.