Dan Kenigsberg has submitted this change and it was merged.

Change subject: tool: configurators package should only contain configurators.
......................................................................


tool: configurators package should only contain configurators.

in preperation for autoloading modules. Moving configfile.py out.

Change-Id: I862469b19265ce7034d65b06c9d7b60e4b65a70f
Signed-off-by: Mooli Tayer <mta...@redhat.com>
Reviewed-on: http://gerrit.ovirt.org/34048
Reviewed-by: Yeela Kaplan <ykap...@redhat.com>
Reviewed-by: Yaniv Bronhaim <ybron...@redhat.com>
Reviewed-by: Dan Kenigsberg <dan...@redhat.com>
---
M lib/vdsm/tool/Makefile.am
R lib/vdsm/tool/configfile.py
M lib/vdsm/tool/configurators/Makefile.am
M lib/vdsm/tool/configurators/libvirt.py
M tests/toolTests.py
M vdsm.spec.in
6 files changed, 17 insertions(+), 14 deletions(-)

Approvals:
  Yeela Kaplan: Looks good to me, but someone else must approve
  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/34048
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I862469b19265ce7034d65b06c9d7b60e4b65a70f
Gerrit-PatchSet: 10
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

Reply via email to