diff options
| author | Jenkins <jenkins@review.openstack.org> | 2013-05-23 17:48:19 +0000 |
|---|---|---|
| committer | Gerrit Code Review <review@openstack.org> | 2013-05-23 17:48:19 +0000 |
| commit | ef9b8b0e9e6ab98bd046424138826adc1934d914 (patch) | |
| tree | 581ddd8c031c3a2f4d87af087f7e1ac93bac66d5 /nova/virt | |
| parent | 6dd1709bcea69ab47a695d42cb1c3d0e7f221ab2 (diff) | |
| parent | 6ea7df2cbcdca6c9be0c44d2436e24549c499297 (diff) | |
Merge "Fix UnboundLocalError in powervm lvm cleanup code"
Diffstat (limited to 'nova/virt')
| -rw-r--r-- | nova/virt/powervm/blockdev.py | 14 |
1 files changed, 8 insertions, 6 deletions
diff --git a/nova/virt/powervm/blockdev.py b/nova/virt/powervm/blockdev.py index 5b15c14bb..5e207e1f2 100644 --- a/nova/virt/powervm/blockdev.py +++ b/nova/virt/powervm/blockdev.py @@ -181,6 +181,7 @@ class PowerVMLocalVolumeAdapter(PowerVMDiskAdapter): size_gb = max(instance_type['root_gb'], constants.POWERVM_MIN_ROOT_GB) size = size_gb * 1024 * 1024 * 1024 + disk_name = None try: LOG.debug(_("Creating logical volume of size %s bytes") % size) disk_name = self._create_logical_volume(size) @@ -192,12 +193,13 @@ class PowerVMLocalVolumeAdapter(PowerVMDiskAdapter): "Will attempt cleanup.")) # attempt cleanup of logical volume before re-raising exception with excutils.save_and_reraise_exception(): - try: - self.delete_volume(disk_name) - except Exception: - msg = _('Error while attempting cleanup of failed ' - 'deploy to logical volume.') - LOG.exception(msg) + if disk_name is not None: + try: + self.delete_volume(disk_name) + except Exception: + msg = _('Error while attempting cleanup of failed ' + 'deploy to logical volume.') + LOG.exception(msg) return {'device_name': disk_name} |
