Change in vdsm[master]: coverage: Add vdsm-coverage sub package

nsoffer at redhat.com nsoffer at redhat.com
Sat Dec 5 13:42:26 UTC 2015


Nir Soffer has posted comments on this change.

Change subject: coverage: Add vdsm-coverage sub package
......................................................................


Patch Set 3:

(1 comment)

https://gerrit.ovirt.org/#/c/49807/3/coverage/README.md
File coverage/README.md:

Line 4: 
Line 5: ## Files
Line 6: 
Line 7: - /etc/vdsm/coverage.conf - coverage configuration file
Line 8: - /etc/sysconfig/vdsm - environment file for enabling coverage
This file is useful if we like to collect coverage data for sub processes. Currently we do not, so we can eliminate this file by using coverage api
directly:
http://coverage.readthedocs.org/en/latest/api_coverage.html#the-coverage-class
Line 9: 
Line 10: ## Recording code coverage
Line 11: 
Line 12: 1.  Enable coverage in vdsm.conf:


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I95454731ca1c19e6f01417d2c26023e7c748365a
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Adam Litke <alitke at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Petr Balogh <pbalogh at redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski at gmail.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list