Change in vdsm[master]: Added code coverage plugin

lbednar at redhat.com lbednar at redhat.com
Wed Jun 12 09:26:21 UTC 2013


Lukas Bednar has posted comments on this change.

Change subject: Added code coverage plugin
......................................................................


Patch Set 2: (1 inline comment)

....................................................
File vdsm/codeCoveragePlugin.py
Line 37:                  "%s variable to enable it", OPTION_NAME)
Line 38:         return
Line 39: 
Line 40:     config = None
Line 41:     enabled = os.environ[OPTION_NAME].strip()
see several lines bellow.
Line 42: 
Line 43:     if enabled.lower() not in POSITIVE_ANSWERS:
Line 44:         if not os.path.exists(enabled):
Line 45:             log.warn("Found %s=%s; but expected is one from %s options "


-- 
To view, visit http://gerrit.ovirt.org/15545
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I6cb465f7723abad9698aa86bedf2ffc25df55713
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Lukas Bednar <lbednar at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Lukas Bednar <lbednar at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list