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