Change in vdsm[ovirt-3.5]: vdsm-tool: reorgenize module configurers.

ybronhei at redhat.com ybronhei at redhat.com
Sun Aug 31 21:18:15 UTC 2014


Yaniv Bronhaim has submitted this change and it was merged.

Change subject: vdsm-tool: reorgenize module configurers.
......................................................................


vdsm-tool: reorgenize module configurers.

configurators are different from other vdsm tool exposed
entities since they have their own lifecycle (defined
by ModuleConfigure) and they deserve their own directory.

this happens now since configurator is getting messy
and it is time to split it into different
ModuleConfigure file. In addition more are expected
to join soon.

Change-Id: Ic79d400760e425f6f4863b6a33c69aa14fb0c623
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1127877
Signed-off-by: Mooli Tayer <mtayer at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/31400
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/32047
Reviewed-by: Yaniv Bronhaim <ybronhei at redhat.com>
---
M configure.ac
M lib/vdsm/tool/Makefile.am
M lib/vdsm/tool/configurator.py
A lib/vdsm/tool/configurators/Makefile.am
A lib/vdsm/tool/configurators/__init__.py
R lib/vdsm/tool/configurators/configfile.py
A lib/vdsm/tool/configurators/libvirt.py
A lib/vdsm/tool/configurators/sanlock.py
M tests/toolTests.py
M vdsm.spec.in
10 files changed, 787 insertions(+), 664 deletions(-)

Approvals:
  Yaniv Bronhaim: Looks good to me, approved
  mooli tayer: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ic79d400760e425f6f4863b6a33c69aa14fb0c623
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.5
Gerrit-Owner: mooli tayer <mtayer at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: Yaniv Eylon <yaniv.eylon at gmail.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: mooli tayer <mtayer at redhat.com>


More information about the vdsm-patches mailing list