Change in vdsm[master]: debug: Remove unneeded coverage support

nsoffer at redhat.com nsoffer at redhat.com
Tue Aug 11 16:59:14 UTC 2015


Nir Soffer has uploaded a new change for review.

Change subject: debug: Remove unneeded coverage support
......................................................................

debug: Remove unneeded coverage support

For testing code coverage, we can use nosetests --with-coverage options.
There is no need to pollute vdsm production code with special imports
that may cause silent failures in production.

Change-Id: I5b7c5eba1202e3db6a5fe1679806ca82ac0b076b
Signed-off-by: Nir Soffer <nsoffer at redhat.com>
---
M vdsm/vdsm
1 file changed, 0 insertions(+), 9 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/25/44725/1

diff --git a/vdsm/vdsm b/vdsm/vdsm
index 0648562..9fe3af5 100755
--- a/vdsm/vdsm
+++ b/vdsm/vdsm
@@ -131,15 +131,6 @@
         # plugin and we would like to print that out.
         pass
 
-    # Used to enable code coverage. On production machines
-    # "coverage" should not exists and COVERAGE_PROCESS_START should not be
-    # set.
-    try:
-        import coverage
-        coverage.process_startup()
-    except ImportError:
-        pass
-
     log = logging.getLogger('vds')
     try:
         logging.root.handlers.append(logging.StreamHandler())


-- 
To view, visit https://gerrit.ovirt.org/44725
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I5b7c5eba1202e3db6a5fe1679806ca82ac0b076b
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer <nsoffer at redhat.com>


More information about the vdsm-patches mailing list