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 <[email protected]> Reviewed-on: http://gerrit.ovirt.org/31400 Reviewed-by: Dan Kenigsberg <[email protected]> Reviewed-on: http://gerrit.ovirt.org/32047 Reviewed-by: Yaniv Bronhaim <[email protected]> --- 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 <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Nir Soffer <[email protected]> Gerrit-Reviewer: Yaniv Bronhaim <[email protected]> Gerrit-Reviewer: Yaniv Eylon <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: mooli tayer <[email protected]> _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
