- Engine-commits - oVirt List Archives

Change in ovirt-engine[ovirt-engine-3.6]: engine: bump VM version upon VM nic add/update
by tnisan＠redhat.com 18 May '16

18 May '16

Tal Nisan has submitted this change and it was merged. Change subject: engine: bump VM version upon VM nic add/update ...................................................................... engine: bump VM version upon VM nic add/update Change-Id: I000fa834b4c393e6ab261e4ae52d1fd71acfcdd4 Bug-Url: https://bugzilla.redhat.com/1327267 Signed-off-by: Yevgeny Zaspitsky <yzaspits(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VmCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/vm/AbstractVmInterfaceCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/vm/AddVmInterfaceCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/vm/RemoveVmInterfaceCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/vm/UpdateVmInterfaceCommand.java 5 files changed, 16 insertions(+), 0 deletions(-) Approvals: Jenkins CI: Passed CI tests Roy Golan: Looks good to me, but someone else must approve Arik Hadas: Looks good to me, approved Yevgeny Zaspitsky: Verified -- To view, visit https://gerrit.ovirt.org/57453 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I000fa834b4c393e6ab261e4ae52d1fd71acfcdd4 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-hosted-engine-setup[master]: build: ovirt-hosted-engine-setup-2.0.0_beta1
by stirabos＠redhat.com 18 May '16

18 May '16

Simone Tiraboschi has submitted this change and it was merged. Change subject: build: ovirt-hosted-engine-setup-2.0.0_beta1 ...................................................................... build: ovirt-hosted-engine-setup-2.0.0_beta1 Change-Id: Id770c665020f4966a357cbb76ffbe0cab3f5d81e Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M automation/build-artifacts.sh M configure.ac M ovirt-hosted-engine-setup.spec.in 3 files changed, 3 insertions(+), 5 deletions(-) Approvals: Sandro Bonazzola: Verified Simone Tiraboschi: Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/57605 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id770c665020f4966a357cbb76ffbe0cab3f5d81e Gerrit-PatchSet: 2 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Rafael Martins <rmartins(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: webadmin: Kernel cmdline - frontend
by tjelinek＠redhat.com 18 May '16

18 May '16

Tomas Jelinek has submitted this change and it was merged. Change subject: webadmin: Kernel cmdline - frontend ...................................................................... webadmin: Kernel cmdline - frontend This feature allows to customize kernel commandline (bootloader configuration) of hosts. This patch adds 'Kernel' tab to 'Host' dialog allowing to edit kernel commandline parameters. User can either choose form 4 predefined params using checkboxes or use textbox to create fully custom kernel cmdline (the cmdline will still be merged with predefined parameters that are already in bootloader). Kernel command line is never parsed pack to checkboxes. Change-Id: I6794f5b8ac60257e037a1c6e745387ab7c56232a Signed-off-by: Jakub Niedermertl <jniederm(a)redhat.com> Bug-Url: https://bugzilla.redhat.com/1301104 --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/CommonApplicationConstants.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/CommonApplicationTemplates.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/editor/generic/AbstractLabelableEntityModelEditor.java A frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/editor/generic/TextEntityModelEditor.java M frontend/webadmin/modules/gwt-common/src/main/resources/org/ovirt/engine/core/Common.gwt.xml M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/Cloner.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/dataprovider/AsyncDataProvider.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/TabName.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/EditHostModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostModel.java A frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/KernelCmdlineUtil.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/NewHostModel.java M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/UIConstants.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/presenter/popup/host/HostPopupPresenterWidget.java 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 17 files changed, 685 insertions(+), 44 deletions(-) Approvals: Tomas Jelinek: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/57053 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6794f5b8ac60257e037a1c6e745387ab7c56232a Gerrit-PatchSet: 23 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Jakub Niedermertl <jniederm(a)redhat.com> Gerrit-Reviewer: Jakub Niedermertl <jniederm(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: Kernel cmdline - data layer
by tjelinek＠redhat.com 18 May '16

18 May '16

Tomas Jelinek has submitted this change and it was merged. Change subject: core: Kernel cmdline - data layer ...................................................................... core: Kernel cmdline - data layer This feature allows to customize kernel commandline (bootloader configuration) of hosts. The configuration is stored in vds_static table in columns: kernel_cmdline and last_stored_kernel_cmdline. 'kernel_cmdline' column is JSON structured and holds values of checkboxes for UI and generated line to be send to hosts. The line is never parsed to checkboxes again. 'last_stored_kernel_cmdline' column contains last successfully stored line in host. Line from 'kernel_cmdline' column will be copied to last_stored_kernel_cmdline after successful host deploy. VdsStaticDao#update method and underlying stored procedure don't update 'last_stored_kernel_cmdline' column to prevent accidental change. There is separate VdsStaticDao#updateLastStoredKernelCmdline that called that will only be called from host deploy. Host deploy procedure will send both current and last stored line to the host because the tool manipulating bootloader configuration - grubby - can't actually read the configuration, only write. Thus old configuration is necessary to know what to remove before applying new configuration. Bug-Url: https://bugzilla.redhat.com/1301104 Change-Id: I93c1d299fa9c86c2bf4dc958904f61aaccf38ec4 Signed-off-by: Jakub Niedermertl <jniederm(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/ServerCpu.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VDS.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VdsStatic.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VdsDaoImpl.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VdsStaticDao.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VdsStaticDaoImpl.java M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/VdsStaticDaoTest.java M backend/manager/modules/dal/src/test/resources/fixtures.xml M packaging/dbscripts/create_views.sql A packaging/dbscripts/upgrade/04_00_0680_vds_add_kernel_args_columns.sql M packaging/dbscripts/upgrade/post_upgrade/0010_add_object_column_white_list_table.sql M packaging/dbscripts/vds_sp.sql 12 files changed, 328 insertions(+), 14 deletions(-) Approvals: Tomas Jelinek: Verified; Passed CI tests Arik Hadas: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/57051 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I93c1d299fa9c86c2bf4dc958904f61aaccf38ec4 Gerrit-PatchSet: 14 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: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Jakub Niedermertl <jniederm(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: gluster: fix NPE in gluster volumes Tab.
by sabose＠redhat.com 18 May '16

18 May '16

Sahina Bose has submitted this change and it was merged. Change subject: gluster: fix NPE in gluster volumes Tab. ...................................................................... gluster: fix NPE in gluster volumes Tab. There is an NPE when we compare null > 0. This happens when unsyncedEntries is null in volume bricks. So adding a null check before using this for comparison. Change-Id: I28a12bd37e30e35bdb722fe65e8a428e1cc097df Signed-off-by: Ramesh Nachimuthu <rnachimu(a)redhat.com> --- M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/utils/GlusterVolumeUtils.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/ApplicationMessages.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/widget/table/cell/BrickStatusCell.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/widget/table/column/BrickStatusColumn.java 4 files changed, 6 insertions(+), 5 deletions(-) Approvals: Jenkins CI: Passed CI tests Sahina Bose: Looks good to me, approved Ramesh N: Verified -- To view, visit https://gerrit.ovirt.org/57609 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I28a12bd37e30e35bdb722fe65e8a428e1cc097df Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: webadmin: add deprecated to Legacy USB support
by tjelinek＠redhat.com 18 May '16

18 May '16

Tomas Jelinek has submitted this change and it was merged. Change subject: webadmin: add deprecated to Legacy USB support ...................................................................... webadmin: add deprecated to Legacy USB support Change-Id: Ibb307daec0aca4890363f866ca3be2734a7539b0 Bug-Url: https://bugzilla.redhat.com/1337055 Signed-off-by: Tomas Jelinek <tjelinek(a)redhat.com> --- M frontend/webadmin/modules/uicompat/src/main/resources/org/ovirt/engine/ui/uicompat/LocalizedEnums.properties 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Tomas Jelinek: Verified; Passed CI tests Martin Betak: Looks good to me, approved Michal Skrivanek: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/57616 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibb307daec0aca4890363f866ca3be2734a7539b0 Gerrit-PatchSet: 2 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: Michal Skrivanek <michal.skrivanek(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-hosted-engine-setup[master]: network: skip NetworkManager check/disable via answerfile
by stirabos＠redhat.com 18 May '16

18 May '16

Simone Tiraboschi has submitted this change and it was merged. Change subject: network: skip NetworkManager check/disable via answerfile ...................................................................... network: skip NetworkManager check/disable via answerfile Adding the capability to skip NM check and disabling by host-deploy via answerfile (dafault enforcing). In the normal flow, host-deploy will disable NetworkManager if active. When host-deploy runs, the management bridge has already been created and the engine VM is already running so disabling NetworkManager on the fly can be risky for the engine VM networking. To avoid that, ovirt-hosted-engine-setup will check NetworkManager status and eventually ask the user to stop: this behavior wasn't configurable. On the other side Next Generation Node is based on Cockpit, which requires NetworkManager to report the network status. Next Generation Node comes with NetworkManager-config-server so NetworkManager there should be passive enough. With this patch we make its check optionally skippable via answerfile. Change-Id: Iaec165cfb8bf93a3a17af20b23ce97ec2ad3223d Bug-Url: https://bugzilla.redhat.com/1337050 Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M src/ovirt_hosted_engine_setup/constants.py M src/plugins/gr-he-setup/engine/add_host.py M src/plugins/gr-he-setup/network/bridge.py 3 files changed, 21 insertions(+), 3 deletions(-) Approvals: Simone Tiraboschi: Verified; Looks good to me, approved Jenkins CI: Passed CI tests Yedidyah Bar David: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/56146 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iaec165cfb8bf93a3a17af20b23ce97ec2ad3223d Gerrit-PatchSet: 10 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: master Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Ryan Barry <rbarry(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: dedicated query for monitored network interfaces
by ahadas＠redhat.com 18 May '16

18 May '16

Arik Hadas has submitted this change and it was merged. Change subject: core: dedicated query for monitored network interfaces ...................................................................... core: dedicated query for monitored network interfaces Previously, we used GetVmNetworkInterfaceViewByVmId that uses the costly vm_interface_view in order to query the network interfaces whose statistics are being monitored. This patches adds a new procedure that uses a much lighter view than vm_interface_view that will be used by the monitoring instead. There is still a lot of place for improvement but this change seems to be the simpler and one that would probably be needed along with further improvements anyway as it reduces the time to fetch the data by more than four times. Change-Id: I475ffe644c7636e1e3a5ddea474d08f4b6ee80bd Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/network/VmNetworkInterfaceDao.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/network/VmNetworkInterfaceDaoImpl.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/monitoring/VmAnalyzerFactory.java M packaging/dbscripts/create_views.sql M packaging/dbscripts/network_sp.sql 5 files changed, 84 insertions(+), 1 deletion(-) Approvals: Eli Mesika: Looks good to me, but someone else must approve Shahar Havivi: Looks good to me, approved Alona Kaplan: Looks good to me, approved Arik Hadas: Verified; Passed CI tests -- To view, visit https://gerrit.ovirt.org/57474 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I475ffe644c7636e1e3a5ddea474d08f4b6ee80bd Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Michal Skrivanek <mskrivan(a)redhat.com> Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-hosted-engine-setup[master]: upgrade: improving upgrade flow
by stirabos＠redhat.com 18 May '16

18 May '16

Simone Tiraboschi has submitted this change and it was merged. Change subject: upgrade: improving upgrade flow ...................................................................... upgrade: improving upgrade flow improving the upgrade flow to replace the el6 appliance with an el7 one restoring a backup and configuring it Change-Id: I7fcdee34aad5dbf21bf00822f89c53f6359de283 Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M src/ovirt_hosted_engine_setup/constants.py M src/ovirt_hosted_engine_setup/vm_status.py M src/plugins/gr-he-common/core/misc.py M src/plugins/gr-he-common/core/remote_answerfile.py M src/plugins/gr-he-common/engine/health.py M src/plugins/gr-he-common/vm/boot_disk.py M src/plugins/gr-he-common/vm/cloud_init.py M src/plugins/gr-he-common/vm/misc.py M src/plugins/gr-he-upgradeappliance/core/misc.py M src/plugins/gr-he-upgradeappliance/vm/configurevm.py M src/plugins/gr-he-upgradeappliance/vm/runvm.py 11 files changed, 52 insertions(+), 16 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, but someone else must approve Simone Tiraboschi: Verified; Looks good to me, approved Jenkins CI: Passed CI tests Yedidyah Bar David: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/57521 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7fcdee34aad5dbf21bf00822f89c53f6359de283 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: master Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Rafael Martins <rmartins(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: Revert "ovirt-engine: build on bare metal"
by sbonazzo＠redhat.com 18 May '16

18 May '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: Revert "ovirt-engine: build on bare metal" ...................................................................... Revert "ovirt-engine: build on bare metal" This reverts commit 58849171dde763b09e5bc9b0d68a416df1ac94a5. Bare metal slaves are not compliant. Change-Id: I4866dc4606825e83983817aca2e73cad8c12f30c Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/confs/projects/ovirt-engine/ovirt-engine_standard.yaml 1 file changed, 0 insertions(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Looks good to me; Passed CI tests; Ready for merge -- To view, visit https://gerrit.ovirt.org/57619 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4866dc4606825e83983817aca2e73cad8c12f30c Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0