Change in vdsm[master]: fixing pyflakes errors

Yaniv Bronhaim has uploaded a new change for review.

Change subject: fixing pyflakes errors
......................................................................

fixing pyflakes errors

Change-Id: I54dad2edc73c3922bdb8c97e6fa036a3dfcf683b
Signed-off-by: Yaniv Bronhaim <ybronhei(a)redhat.com&gt;
---
M contrib/profile-stats
M vdsm/vdsm
M vdsm/vdsmDebugPlugin.py
3 files changed, 2 insertions(+), 4 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/26/35026/1

diff --git a/contrib/profile-stats b/contrib/profile-stats
index 0666eac..4b02cd9 100755
--- a/contrib/profile-stats
+++ b/contrib/profile-stats
@@ -56,7 +56,6 @@
 
 import optparse
 import pstats
-import sys
 
 op = optparse.OptionParser(usage='%prog [options] file')
 op.add_option('-s', '--sort', dest='sort',
diff --git a/vdsm/vdsm b/vdsm/vdsm
index 9004c8c..5c1ab71 100755
--- a/vdsm/vdsm
+++ b/vdsm/vdsm
@@ -111,6 +111,7 @@
     # vdsmDebugPlugin.py should not exists
     try:
         import vdsmDebugPlugin
+        vdsmDebugPlugin.turnOnDebugPlugin()
     except ImportError:
         # This is OK, it just means the file isn't
         # there and we are not running in debug mode.
diff --git a/vdsm/vdsmDebugPlugin.py b/vdsm/vdsmDebugPlugin.py
index 3f4e7b7..af06dbe 100644
--- a/vdsm/vdsmDebugPlugin.py
+++ b/vdsm/vdsmDebugPlugin.py
@@ -36,7 +36,7 @@
         exec(code)
 
 
-def __turnOnDebugPlugin():
+def turnOnDebugPlugin():
     log.warn("Starting Debug Interpreter. Tread lightly!")
     try:
         if os.path.exists(ADDRESS):
@@ -50,5 +50,3 @@
         servThread.start()
     except:
         log.error("Could not start debug plugin", exc_info=True)
-
-__turnOnDebugPlugin()


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I54dad2edc73c3922bdb8c97e6fa036a3dfcf683b
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <ybronhei(a)redhat.com&gt;

[Dan Kenigsberg]

Dan Kenigsberg has submitted this change and it was merged.

Change subject: fixing pyflakes errors
......................................................................


fixing pyflakes errors

Change-Id: I54dad2edc73c3922bdb8c97e6fa036a3dfcf683b
Signed-off-by: Yaniv Bronhaim <ybronhei(a)redhat.com&gt;
Reviewed-on: http://gerrit.ovirt.org/35026
Reviewed-by: Dan Kenigsberg <danken(a)redhat.com&gt;
---
M contrib/profile-stats
M vdsm/vdsm
M vdsm/vdsmDebugPlugin.py
3 files changed, 2 insertions(+), 4 deletions(-)

Approvals:
  Yaniv Bronhaim: Verified
  Dan Kenigsberg: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I54dad2edc73c3922bdb8c97e6fa036a3dfcf683b
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <ybronhei(a)redhat.com&gt;
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com&gt;
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com&gt;
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server

[Dan Kenigsberg]

Dan Kenigsberg has posted comments on this change.

Change subject: fixing pyflakes errors
......................................................................


Patch Set 1: Code-Review+2

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I54dad2edc73c3922bdb8c97e6fa036a3dfcf683b
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <ybronhei(a)redhat.com&gt;
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com&gt;
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com&gt;
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No

Yaniv Bronhaim has posted comments on this change.

Change subject: fixing pyflakes errors
......................................................................


Patch Set 1: Verified+1

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I54dad2edc73c3922bdb8c97e6fa036a3dfcf683b
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <ybronhei(a)redhat.com&gt;
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com&gt;
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No

[Jenkins CI RO]

oVirt Jenkins CI Server has posted comments on this change.

Change subject: fixing pyflakes errors
......................................................................


Patch Set 1:

Build Failed 

http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/13481/ : FAILURE

http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/13319/ : SUCCESS

http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/12529/ : FAILURE

http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created_staging/92/ : FAILURE

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I54dad2edc73c3922bdb8c97e6fa036a3dfcf683b
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <ybronhei(a)redhat.com&gt;
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com&gt;
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No

[Jenkins CI RO]

oVirt Jenkins CI Server has posted comments on this change.

Change subject: fixing pyflakes errors
......................................................................


Patch Set 2:

Build Failed 

http://jenkins.ovirt.org/job/vdsm_master_create-rpms-fc21-x86_64_merged/181/ : FAILURE

http://jenkins.ovirt.org/job/vdsm_master_create-rpms-el6-x86_64_merged/204/ : SUCCESS

http://jenkins.ovirt.org/job/vdsm_master_create-rpms_merged_test_debug/397/ : FAILURE

http://jenkins.ovirt.org/job/vdsm_master_verify-error-codes_merged/6029/ : SUCCESS

http://jenkins.ovirt.org/job/vdsm_master_create-rpms-el7-x86_64_merged/202/ : FAILURE

http://jenkins.ovirt.org/job/vdsm_master_create-rpms-fc20-x86_64_merged/197/ : SUCCESS

http://jenkins.ovirt.org/job/vdsm_master_unit-tests_merged/4190/ : FAILURE

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I54dad2edc73c3922bdb8c97e6fa036a3dfcf683b
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <ybronhei(a)redhat.com&gt;
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com&gt;
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com&gt;
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No

[Jenkins CI RO]

oVirt Jenkins CI Server has posted comments on this change.

Change subject: fixing pyflakes errors
......................................................................


Patch Set 2:

Build Failed 

http://jenkins.ovirt.org/job/vdsm_master_create-rpms-el7-x86_64_merged/203/ : SUCCESS

http://jenkins.ovirt.org/job/vdsm_master_create-rpms-fc21-x86_64_merged/182/ : SUCCESS

http://jenkins.ovirt.org/job/vdsm_master_create-rpms-el6-x86_64_merged/205/ : SUCCESS

http://jenkins.ovirt.org/job/vdsm_master_create-rpms_merged_test_debug/398/ : FAILURE

http://jenkins.ovirt.org/job/vdsm_master_verify-error-codes_merged/6030/ : SUCCESS

http://jenkins.ovirt.org/job/vdsm_master_create-rpms-fc20-x86_64_merged/198/ : ABORTED

http://jenkins.ovirt.org/job/vdsm_master_unit-tests_merged/4191/ : FAILURE

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I54dad2edc73c3922bdb8c97e6fa036a3dfcf683b
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <ybronhei(a)redhat.com&gt;
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com&gt;
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com&gt;
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No