Yaniv Bronhaim has uploaded a new change for review.
Change subject: Removing unused logErr parameter from execCmd
......................................................................
Removing unused logErr parameter from execCmd
Change-Id: Ic07b3524246d8795df8accf35c9a9a82e42d8497
Signed-off-by: Yaniv Bronhaim <ybronhei(a)redhat.com>
---
M lib/vdsm/utils.py
M vdsm/storage/misc.py
2 files changed, 5 insertions(+), 5 deletions(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/50/28050/1
diff --git a/lib/vdsm/utils.py b/lib/vdsm/utils.py
index 600fc7d..092ef2f 100644
--- a/lib/vdsm/utils.py
+++ b/lib/vdsm/utils.py
@@ -644,7 +644,7 @@
self._poller.close()
-def execCmd(command, sudo=False, cwd=None, data=None, raw=False, logErr=True,
+def execCmd(command, sudo=False, cwd=None, data=None, raw=False,
printable=None, env=None, sync=True, nice=None, ioclass=None,
ioclassdata=None, setsid=False, execCmdLogger=logging.root,
deathSignal=0, childUmask=None):
diff --git a/vdsm/storage/misc.py b/vdsm/storage/misc.py
index 4eb56e7..6dd2679 100644
--- a/vdsm/storage/misc.py
+++ b/vdsm/storage/misc.py
@@ -506,9 +506,9 @@
newName = os.path.join(directory, fd[key]['new'])
if isOvirtNode() and persist and not cp:
try:
- execCmd([constants.EXT_UNPERSIST, oldName], logErr=False,
+ execCmd([constants.EXT_UNPERSIST, oldName],
sudo=True)
- execCmd([constants.EXT_UNPERSIST, newName], logErr=False,
+ execCmd([constants.EXT_UNPERSIST, newName],
sudo=True)
except:
pass
@@ -516,7 +516,7 @@
if cp:
execCmd([constants.EXT_CP, oldName, newName], sudo=True)
if isOvirtNode() and persist and not os.path.exists(newName):
- execCmd([constants.EXT_PERSIST, newName], logErr=False,
+ execCmd([constants.EXT_PERSIST, newName],
sudo=True)
else:
@@ -525,7 +525,7 @@
pass
if isOvirtNode() and persist and not cp:
try:
- execCmd([constants.EXT_PERSIST, newName], logErr=False,
+ execCmd([constants.EXT_PERSIST, newName],
sudo=True)
except:
pass
--
To view, visit
http://gerrit.ovirt.org/28050
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic07b3524246d8795df8accf35c9a9a82e42d8497
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <ybronhei(a)redhat.com>