The Windows volume driver fails when queried for a manually deleted volume

Bug #1089168 reported by Alessandro Pilotti
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
Undecided
Alessandro Pilotti

Bug Description

Here is the exception trace: http://paste.openstack.org/show/27866/

To reproduce it:

Provision DevStack or any other controller
Install Cinder Volume on a Windows Server
Create a volume on the Cinder server: cinder create 1
Wipe out the Windows server (e.g. by revertining to a previous snapshot) and reinstall Cinder Volume
When the service starts Cinder queries the status of the volumes
If the volumes are not found, you get the above error

To fix it the following WMI result length needs to be checked before taking the first element:

 wt_disk = self._conn_wmi.WT_Disk(Description=vol_name)[0]

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/17967

Changed in cinder:
assignee: nobody → Alessandro Pilotti (alexpilotti)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cinder (master)

Reviewed: https://review.openstack.org/17967
Committed: http://github.com/openstack/cinder/commit/147e30b4a8a9ce8361b048239a0187ac04679db1
Submitter: Jenkins
Branch: master

commit 147e30b4a8a9ce8361b048239a0187ac04679db1
Author: Alessandro Pilotti <email address hidden>
Date: Wed Dec 12 22:10:35 2012 +0200

    Fixes a Windows volume driver bug on disk export

    Fixes Bug #1089168

    When a VHD created by Cinder using the iSCSI Target Server is deleted
    manually, at the next startup the cinder-volume service terminates with an
    exception.
    This fix adds a check to verify if the disk exists.

    Change-Id: I830cb8332dcef845591bd88cccef1e0e9781ffcc

Changed in cinder:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in cinder:
milestone: none → grizzly-2
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in cinder:
milestone: grizzly-2 → 2013.1
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.