openstack
/
cinder
Code Issues Proposed changes

pylint: Fix vmdk driver use before definition 

Define "attached" earlier so that it is always
defined in the except: block.

Change-Id: Id07a2e713ebf667432e9bfceac0972c92185d3c5
This commit is contained in:
Eric Harney 2022-05-17 10:35:42 -04:00
parent 66b44b8f34
commit 970ac584d6
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
cinder/volume/drivers/vmware/vmdk.py
View File

@ -1272,6 +1272,7 @@ class VMwareVcVmdkDriver(driver.VolumeDriver):
'param': create_params})
backing = self._create_backing(volume, create_params=create_params)
attached = False
try:
# Find the backing's datacenter, host, datastore and folder.
(ds_name, folder_path) = self._get_ds_name_folder_path(backing)
@ -1279,7 +1280,6 @@ class VMwareVcVmdkDriver(driver.VolumeDriver):
dc_ref = self.volumeops.get_dc(host)
vmdk_path = None
attached = False
# Create flat extent virtual disk from the image.
if image_disk_type == ImageDiskType.SPARSE: