Dan Kenigsberg has submitted this change and it was merged. Change subject: tool: autoload modules. ......................................................................
tool: autoload modules. Configurator modules are loaded once in the beginning of vdsm-tool's operation. Once vdsm-tool uses autoloading adding new configurators does not requires changing configurators.py but only module logic and packaging. This also enables Manual installation of modules. If desired, we could have a /etc/vdsm/configurators.d in the future. The import of validate_ovirt_certs needed a change due to autoload. Change-Id: I6cc14a1c409ee607e18d42adcd8a54759d8dbf6d Signed-off-by: Mooli Tayer <mta...@redhat.com> Reviewed-on: http://gerrit.ovirt.org/34049 Reviewed-by: Yaniv Bronhaim <ybron...@redhat.com> Reviewed-by: Dan Kenigsberg <dan...@redhat.com> --- M lib/vdsm/tool/configurator.py M lib/vdsm/tool/configurators/certificates.py M lib/vdsm/tool/configurators/libvirt.py 3 files changed, 31 insertions(+), 20 deletions(-) Approvals: Yaniv Bronhaim: Looks good to me, but someone else must approve mooli tayer: Verified Dan Kenigsberg: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/34049 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6cc14a1c409ee607e18d42adcd8a54759d8dbf6d Gerrit-PatchSet: 14 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: mooli tayer <mta...@redhat.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Dima Kuznetsov <dkuzn...@redhat.com> Gerrit-Reviewer: Saggi Mizrahi <smizr...@redhat.com> Gerrit-Reviewer: Yaniv Bronhaim <ybron...@redhat.com> Gerrit-Reviewer: Yeela Kaplan <ykap...@redhat.com> Gerrit-Reviewer: automat...@ovirt.org Gerrit-Reviewer: mooli tayer <mta...@redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches