openstack/cinder
Code Issues Proposed changes

Fixes "is not" usage

Browse Source 
Fixes bug #1110987

Change-Id: Iafa9fc548deaf03c229cc9144d85e6c68b3101be
This commit is contained in:
Zhongyue Luo 2013-01-31 10:31:51 +08:00
parent 342e11b912
commit 8960e4d33c
2 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
cinder/volume/drivers/netapp.py
View File

@ -548,7 +548,7 @@ class NetAppISCSIDriver(driver.ISCSIDriver):
is empty. This is necessary because the API proxy produces nasty
looking XML.
"""
if not type(elem) is list:
if type(elem) is not list:
return True
if 0 == len(elem):
return True

4
cinder/volume/drivers/san/hp_lefthand.py
View File

@ -159,13 +159,13 @@ class HpSanISCSIDriver(SanISCSIDriver):
volume_attributes["volume." + k] = v
status_node = volume_node.find("status")
if not status_node is None:
if status_node is not None:
for k, v in status_node.attrib.items():
volume_attributes["status." + k] = v
# We only consider the first permission node
permission_node = volume_node.find("permission")
if not permission_node is None:
if permission_node is not None:
for k, v in status_node.attrib.items():
volume_attributes["permission." + k] = v