Change in vdsm[ovirt-3.5]: tool: fix pyflakes errors[1].

mtayer at redhat.com mtayer at redhat.com
Wed Sep 3 14:33:59 UTC 2014


Hello Dan Kenigsberg,

I'd like you to do a code review.  Please visit

    http://gerrit.ovirt.org/32409

to review the following change.

Change subject: tool: fix pyflakes errors[1].
......................................................................

tool: fix pyflakes errors[1].

recently I broke master at change 31293.

[1]
./lib/vdsm/tool/configurators/libvirt.py:29: 'NotRootError' imported but unused
./lib/vdsm/tool/configurators/sanlock.py:24: 'NotRootError' imported but unused

Change-Id: I8e86dd38e3a871d55ffc5cb67d5f4596ec4dc3a6
Signed-off-by: Mooli Tayer <mtayer at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/32256
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
---
M lib/vdsm/tool/configurators/libvirt.py
M lib/vdsm/tool/configurators/sanlock.py
2 files changed, 0 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/09/32409/1

diff --git a/lib/vdsm/tool/configurators/libvirt.py b/lib/vdsm/tool/configurators/libvirt.py
index 0f88b32..1b7556b 100644
--- a/lib/vdsm/tool/configurators/libvirt.py
+++ b/lib/vdsm/tool/configurators/libvirt.py
@@ -27,7 +27,6 @@
 from vdsm.config import config
 
 from .. import \
-    NotRootError, \
     service, \
     validate_ovirt_certs
 from . import \
diff --git a/lib/vdsm/tool/configurators/sanlock.py b/lib/vdsm/tool/configurators/sanlock.py
index 0c847a3..1c2b3dd 100644
--- a/lib/vdsm/tool/configurators/sanlock.py
+++ b/lib/vdsm/tool/configurators/sanlock.py
@@ -21,7 +21,6 @@
 import grp
 import pwd
 
-from .. import NotRootError
 from .import \
     CONFIGURED, \
     InvalidConfig, \


-- 
To view, visit http://gerrit.ovirt.org/32409
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I8e86dd38e3a871d55ffc5cb67d5f4596ec4dc3a6
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.5
Gerrit-Owner: mooli tayer <mtayer at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>


More information about the vdsm-patches mailing list