Return devicenodes as empty list if no LUN's
got connected. It is possible for an intiator to successfully connect to a target, whilst getting no LUN's back. If no devicenodes get detected, it makes more sense to return an empty list than plainly None. This potentially avoids further tasks to have to check if devicenodes is iterable.
This commit is contained in:
parent
3acb084902
commit
21720f6bee
1 changed files with 9 additions and 12 deletions
|
@ -206,18 +206,15 @@ def target_device_node(module, target):
|
|||
# a given target...
|
||||
|
||||
devices = glob.glob('/dev/disk/by-path/*%s*' % target)
|
||||
if len(devices) == 0:
|
||||
return None
|
||||
else:
|
||||
devdisks = []
|
||||
for dev in devices:
|
||||
# exclude partitions
|
||||
if "-part" not in dev:
|
||||
devdisk = os.path.realpath(dev)
|
||||
# only add once (multi-path?)
|
||||
if devdisk not in devdisks:
|
||||
devdisks.append(devdisk)
|
||||
return devdisks
|
||||
devdisks = []
|
||||
for dev in devices:
|
||||
# exclude partitions
|
||||
if "-part" not in dev:
|
||||
devdisk = os.path.realpath(dev)
|
||||
# only add once (multi-path?)
|
||||
if devdisk not in devdisks:
|
||||
devdisks.append(devdisk)
|
||||
return devdisks
|
||||
|
||||
|
||||
def target_isauto(module, target):
|
||||
|
|
Loading…
Reference in a new issue