May 2016 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: db: Add entity, tables and DAO implementation for Label
by rgolan＠redhat.com 22 May '16

22 May '16

Roy Golan has submitted this change and it was merged. Change subject: db: Add entity, tables and DAO implementation for Label ...................................................................... db: Add entity, tables and DAO implementation for Label This patch adds all the database pieces needed to support generic Label implementation starting with VM and Host assignment support. Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1254818 Change-Id: I2512e23c99553087aa4e6f64784686213cd0d0ba Signed-off-by: Martin Sivák <msivak(a)redhat.com> --- A backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/Label.java A backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/LabelBuilder.java A backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/LabelDao.java A backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/LabelDaoImpl.java A backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/LabelDaoTest.java M frontend/webadmin/modules/gwt-common/src/main/resources/org/ovirt/engine/core/Common.gwt.xml M packaging/dbscripts/create_views.sql A packaging/dbscripts/labels_sp.sql A packaging/dbscripts/upgrade/04_00_0700_create_label_tables.sql 9 files changed, 975 insertions(+), 1 deletion(-) Approvals: Martin Peřina: Looks good to me, but someone else must approve Martin Sivák: Verified Jenkins CI: Passed CI tests Roy Golan: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/55873 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2512e23c99553087aa4e6f64784686213cd0d0ba Gerrit-PatchSet: 20 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Jenny Tokar <jtokar(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Roman Mohr <rmohr(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: restapi: Integrate network filter id to vnic profile
by alkaplan＠redhat.com 22 May '16

22 May '16

Alona Kaplan has submitted this change and it was merged. Change subject: restapi: Integrate network filter id to vnic profile ...................................................................... restapi: Integrate network filter id to vnic profile Add get and update support. Enables the user to choose the network filter among several options: 1. Use default. 2. Use one of the supported network filters. 3. Have no filter at all. Change-Id: I636893e16dd19258a0e2d4fd4f17a3ec27cab5de Signed-off-by: Eliraz Levi <elevi(a)redhat.com> Signed-off-by: Alona Kaplan <alkaplan(a)redhat.com> --- M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/AbstractBackendVnicProfilesResource.java M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/VnicProfileMapper.java 2 files changed, 39 insertions(+), 1 deletion(-) Approvals: Juan Hernandez: Looks good to me, approved Alona Kaplan: Verified; Passed CI tests -- To view, visit https://gerrit.ovirt.org/52906 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I636893e16dd19258a0e2d4fd4f17a3ec27cab5de Gerrit-PatchSet: 17 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Eliraz Levi <elevi(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Marcin Mirecki <mmirecki(a)redhat.com> Gerrit-Reviewer: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: restapi: Integrate network filter
by alkaplan＠redhat.com 22 May '16

22 May '16

Alona Kaplan has submitted this change and it was merged. Change subject: restapi: Integrate network filter ...................................................................... restapi: Integrate network filter Integrate network filter under: http://localhost:8080/ovirt-engine/api/networkfilters The Get request will list all the supported network filters. Change-Id: Ic7271d14eb5a3fd29529899244224ef767ed71df Signed-off-by: Eliraz Levi <elevi(a)redhat.com> Signed-off-by: mirecki <mmirecki(a)redhat.com> Signed-off-by: Alona Kaplan <alkaplan(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/utils/ApiRootLinksCreator.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendApiResource.java A backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendNetworkFilterResource.java A backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendNetworkFiltersResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/util/LinkHelper.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendApiResourceTest.java A backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/NetworkFilterMapper.java 7 files changed, 134 insertions(+), 2 deletions(-) Approvals: Alona Kaplan: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/51450 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic7271d14eb5a3fd29529899244224ef767ed71df Gerrit-PatchSet: 31 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Eliraz Levi <elevi(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Eliraz Levi <elevi(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Marcin Mirecki <mmirecki(a)redhat.com> Gerrit-Reviewer: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: webadmin: show "Disk Allocation" only when there are disks
by mbetak＠redhat.com 20 May '16

20 May '16

Martin Betak has submitted this change and it was merged. Change subject: webadmin: show "Disk Allocation" only when there are disks ...................................................................... webadmin: show "Disk Allocation" only when there are disks It looked confusing to have label "Disk Allocation:" and nothing under it. Fixed by showing the label only if there is at least one disk. Change-Id: I0a5eb00496e63ab1ec6c81a26130c166ef7cf60c Bug-Url: https://bugzilla.redhat.com/1337619 Signed-off-by: Tomas Jelinek <tjelinek(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/AbstractVmPopupWidget.ui.xml M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/storage/DisksAllocationView.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/storage/DisksAllocationView.ui.xml 3 files changed, 15 insertions(+), 1 deletion(-) Approvals: Tomas Jelinek: Verified Jenkins CI: Passed CI tests Martin Betak: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/57801 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0a5eb00496e63ab1ec6c81a26130c166ef7cf60c Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Betak <mbetak(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: webadmin: Host dialog
by awels＠redhat.com 20 May '16

20 May '16

Alexander Wels has submitted this change and it was merged. Change subject: webadmin: Host dialog ...................................................................... webadmin: Host dialog - Update host dialog to match latest bootstrap standards. - Updated HostNetworkProviderWidget to match bootstrap standards. - Updated NeutronAgentWidget to match bootstrap standards. Change-Id: Ie36d0f350cb59c4020a6494858f51af90e96c51e Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1267573 Signed-off-by: Alexander Wels <awels(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/dialog/AdvancedParametersExpander.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/dialog/AdvancedParametersExpander.ui.xml M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/HostPopupView.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/HostPopupView.ui.xml M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/provider/ProviderPopupView.ui.xml M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/widget/host/HostProxySourceWidget.ui.xml M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/widget/provider/HostNetworkProviderWidget.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/widget/provider/HostNetworkProviderWidget.ui.xml M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/widget/provider/NeutronAgentWidget.ui.xml 9 files changed, 292 insertions(+), 327 deletions(-) Approvals: Alexander Wels: Verified Greg Sheremeta: Looks good to me, approved; Passed CI tests Objections: Jenkins CI: Failed CI tests -- To view, visit https://gerrit.ovirt.org/57536 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie36d0f350cb59c4020a6494858f51af90e96c51e Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Greg Sheremeta <gshereme(a)redhat.com> Gerrit-Reviewer: Jakub Niedermertl <jniederm(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Scott Dickerson <sdickers(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: Using lazy umount everywhere
by sbonazzo＠redhat.com 20 May '16

20 May '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: Using lazy umount everywhere ...................................................................... Using lazy umount everywhere Change-Id: Icb9549678f7716bba03bab765ce2fa347d1f6110 Signed-off-by: David Caro <dcaroest(a)redhat.com> --- M jobs/confs/groovy-scripts/cleanup_workspaces.groovy M jobs/confs/shell-scripts/build-local-ovirt-node-iso.sh M jobs/confs/shell-scripts/cleanup_slave.sh M jobs/confs/shell-scripts/mock_cleanup.sh M jobs/confs/shell-scripts/mock_setup.sh 5 files changed, 7 insertions(+), 7 deletions(-) Approvals: Sandro Bonazzola: Ready for merge Nir Soffer: Looks good to me Jenkins CI: Passed CI tests David Caro: Verified; Ready for review -- To view, visit https://gerrit.ovirt.org/57405 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icb9549678f7716bba03bab765ce2fa347d1f6110 Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: David Caro <dcaro(a)redhat.com> Gerrit-Reviewer: Anatoly Litovsky <tlitovsk(a)redhat.com> Gerrit-Reviewer: Anton Marchukov <amarchuk(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: David Caro <dcaro(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nir Soffer <nsoffer(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-hosted-engine-ha[ovirt-hosted-engine-ha-1.3]: storage: add the capability to optionally pass mount options
by msivak＠redhat.com 20 May '16

20 May '16

Martin Sivák has submitted this change and it was merged. Change subject: storage: add the capability to optionally pass mount options ...................................................................... storage: add the capability to optionally pass mount options Some scenario could require custom mount options calling connectStorageServer on the hosted-engine storage domain. Adding an optional parameter to store it in the conf file and, if present, pass it to connectStorageServer Change-Id: If7c0a0b5353ea55c6b0a1926db0866ba3a08c3a6 Bug-Url: https://bugzilla.redhat.com/1298693 Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M ovirt_hosted_engine_ha/env/config.py M ovirt_hosted_engine_ha/lib/storage_server.py 2 files changed, 11 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, but someone else must approve Martin Sivák: Looks good to me, approved Simone Tiraboschi: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/57787 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If7c0a0b5353ea55c6b0a1926db0866ba3a08c3a6 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: ovirt-hosted-engine-ha-1.3 Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-hosted-engine-ha[master]: storage: add the capability to optionally pass mount options
by msivak＠redhat.com 20 May '16

20 May '16

Martin Sivák has submitted this change and it was merged. Change subject: storage: add the capability to optionally pass mount options ...................................................................... storage: add the capability to optionally pass mount options Some scenario could require custom mount options calling connectStorageServer on the hosted-engine storage domain. Adding an optional parameter to store it in the conf file and, if present, pass it to connectStorageServer Change-Id: If7c0a0b5353ea55c6b0a1926db0866ba3a08c3a6 Bug-Url: https://bugzilla.redhat.com/1298693 Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M ovirt_hosted_engine_ha/env/config.py M ovirt_hosted_engine_ha/lib/storage_server.py 2 files changed, 11 insertions(+), 0 deletions(-) Approvals: Martin Sivák: Looks good to me, approved Simone Tiraboschi: Verified Jenkins CI: Passed CI tests Sahina Bose: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/57760 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If7c0a0b5353ea55c6b0a1926db0866ba3a08c3a6 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: master Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: Kernel cmdline - host deploy 2
by mperina＠redhat.com 20 May '16

20 May '16

Martin Peřina has submitted this change and it was merged. Change subject: core: Kernel cmdline - host deploy 2 ...................................................................... core: Kernel cmdline - host deploy 2 Second try after reverted aeb98b5, https://gerrit.ovirt.org/57052 This feature allows to customize kernel commandline (bootloader configuration) of hosts. New VdsDeployUnit setting two environment variables KernelEnv#CMDLINE_NEW and KernelEnv#CMDLINE_OLD to current and last applied kernel command line. After successful host deploy database column vds_static.last_stored_kernel_cmdline is updated. Change-Id: Ie396786ad97f9b62047b4cef7fc60daf8c1773e0 Bug-Url: https://bugzilla.redhat.com/1301104 Signed-off-by: Jakub Niedermertl <jniederm(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/hostdeploy/InstallVdsInternalCommand.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/hostdeploy/VdsDeployKernelUnit.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/hostdeploy/VdsDeployUnit.java 3 files changed, 78 insertions(+), 15 deletions(-) Approvals: Jakub Niedermertl: Verified Martin Peřina: Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/57756 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie396786ad97f9b62047b4cef7fc60daf8c1773e0 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Jakub Niedermertl <jniederm(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jakub Niedermertl <jniederm(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-3.6]: core: Fix permissions for profiles created for new domain
by piotr.kliczewski＠gmail.com 20 May '16

20 May '16

Piotr Kliczewski has submitted this change and it was merged. Change subject: core: Fix permissions for profiles created for new domain ...................................................................... core: Fix permissions for profiles created for new domain The AddDiskProfileCommand takes care of the permissions for newly added disk profiles and gives the users in Everyone group the DiskProfileUser role. AddStorageDomain command added the created disk profile directly to the DB without calling the command and so this role was not assigned. The fix was to call AddDiskProfileCommand directly from AddStorageDomain and also from AddStorageDomainToPoolCommand which creates a disk profile in case one doesn't exist for the storage domain. A new constructor was added to AddSTorageDomainCommand class, this constructor existed on the master branch and was required in order to enable calling that command internally. Change-Id: Ia6dd5483489b65285cbd5905865c07b9446f6459 Bug-Url: https://bugzilla.redhat.com/1209505 Signed-off-by: Jenny Tokar <jtokar(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/profiles/AddDiskProfileCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/profiles/AddProfileCommandBase.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/profiles/ProfileCommandBase.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/AddStorageDomainCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/AttachStorageDomainToPoolCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/StorageDomainCommandBase.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/DiskProfileParameters.java 7 files changed, 47 insertions(+), 30 deletions(-) Approvals: Jenkins CI: Passed CI tests Roy Golan: Looks good to me, approved Maor Lipchuk: Looks good to me, but someone else must approve Jenny Tokar: Verified -- To view, visit https://gerrit.ovirt.org/57526 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia6dd5483489b65285cbd5905865c07b9446f6459 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Jenny Tokar <jtokar(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Jenny Tokar <jtokar(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Phillip Bailey <phbailey(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Roman Mohr <rmohr(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Yanir Quinn <yquinn(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0