Change in vdsm[master]: code coverage: change approach how to enable it

nsoffer at redhat.com nsoffer at redhat.com
Thu Nov 26 11:26:33 UTC 2015


Nir Soffer has posted comments on this change.

Change subject: code coverage: change approach how to enable it
......................................................................


Patch Set 2:

(1 comment)

https://gerrit.ovirt.org/#/c/49168/2/lib/vdsm/config.py.in
File lib/vdsm/config.py.in:

Line 406: 
Line 407:         ('coverage_enable', 'false',
Line 408:             'Enable code coverage (requires python-coverage package). '
Line 409:             'false by default. Use environment file /etc/sysconfig/vdsm to '
Line 410:             'set COVERAGE_PROCESS_START and COVERAGE_FILE variables.'),
> Yes, I will need put two files on each host.. one is /etc/sysconfig/vdsm wh
These files seems to be useful to anyone that want to do code coverage for vdsm, so I like to have them in vdsm.

This will also help developers to understand how the code work and decrease the chance that we break this in the future.

How about adding a package - vdsm-coverage that will install all the file in the correct place and enable coverage in the configuration?

Then it will be trivial for ovirt-host-deploy to install this package if instructed by the engine.

Piotr, Dan, what do you think?
Line 411: 
Line 412:     ]),
Line 413: 
Line 414:     # Section: [gluster]


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I49d8de0fd7c329d19a80827a4c2fd26eb70e04be
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Petr Balogh <pbalogh at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken 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: Yaniv Bronhaim
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list