Change in vdsm[master]: Introducing configurator package in vdsm-tool

ybronhei at redhat.com ybronhei at redhat.com
Sat Nov 2 20:21:26 UTC 2013


Yaniv Bronhaim has posted comments on this change.

Change subject: Introducing configurator package in vdsm-tool
......................................................................


Patch Set 23:

not hard, just need that someone will explain me the way it is. why do we call this function configure sanlock and why don't we add qemu gid to sanlock group manually ? must be a reason for the way this function implemented . Federico, can you share what's behind that logic? It's quite new this part..

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I16bf5894e7e55a84b4c2a0caacde383ae7c19242
Gerrit-PatchSet: 23
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alonbl at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: Zhou Zheng Sheng <zhshzhou at linux.vnet.ibm.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No


More information about the vdsm-patches mailing list