openstack
/
deb-python-os-brick
Code Issues Proposed changes

Remove useless info logging in check_valid_device 

In the method 'check_valid_device' of class BaseLinuxConnector(),the
actual return value of 'info' from the try block is not captured as a
ProcessExecutionError exception but is simply logged with the initial
assignment 'None'. It is not necessary to have 'info' logged as 'None'.

Change-Id: Ia67727c5741695130e487d4532b85b6206bc05b7
This commit is contained in:
Shilpa Jagannath 2016-07-18 19:51:14 +05:30
parent 3ee1b9a0aa
commit 9aee7c2972
1 changed files with 2 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
os_brick/initiator/connector.py
View File

@ -473,9 +473,8 @@ class BaseLinuxConnector(InitiatorConnector):
root_helper=self._root_helper)
except putils.ProcessExecutionError as e:
LOG.error(_LE("Failed to access the device on the path "
"%(path)s: %(error)s %(info)s."),
{"path": path, "error": e.stderr,
"info": info})
"%(path)s: %(error)s."),
{"path": path, "error": e.stderr})
return False
# If the info is none, the path does not exist.
if info is None: