Change in vdsm[master]: Separate MOM to an external process

ybronhei at redhat.com ybronhei at redhat.com
Thu Jun 11 11:53:13 UTC 2015


Yaniv Bronhaim has posted comments on this change.

Change subject: Separate MOM to an external process
......................................................................


Patch Set 14:

(1 comment)

https://gerrit.ovirt.org/#/c/41602/14/tests/momTests.py
File tests/momTests.py:

Line 6: from momIF import MomClient
Line 7: from caps import PAGE_SIZE_BYTES
Line 8: from vdsm.define import Mbytes
Line 9: 
Line 10: MOM_CONF = os.path.join(os.path.dirname(__file__), "../vdsm/mom.conf")
are sure this relative path works when vdsm-tests package is installed and run on from /usr/share/vdsm ?
Line 11: MOM_PORT = 54323
Line 12: 
Line 13: 
Line 14: class DummyMomApi(object):


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I99b81d21080834b7031824dab0a26f45e7eac7af
Gerrit-PatchSet: 14
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: Adam Litke <alitke at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Dima Kuznetsov <dkuznets at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list