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

nsoffer at redhat.com nsoffer at redhat.com
Thu Nov 26 17:22:28 UTC 2015


Nir Soffer has posted comments on this change.

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


Patch Set 3:

Edward, running vmdsm via coverage is not good enough for QE, they want an easy way to enable or disable coverage on a system.

Adding configuration is the minimal change to make our coverage support safe for production.

I think the next step would be to move coverage to daemonAdapter.py, so it will read vdsm conf and execve "coverage run vdsm". This way we can measure coverage also for supervdsm.

-- 
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: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Petr Balogh <pbalogh at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Edward Haas <edwardh 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: No


More information about the vdsm-patches mailing list