March 2013 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: webadmin: UnitVmModel subscribe GetSPClusterService
by derez＠redhat.com 04 Mar '13

04 Mar '13

Daniel Erez has submitted this change and it was merged. Change subject: webadmin: UnitVmModel subscribe GetSPClusterService ...................................................................... webadmin: UnitVmModel subscribe GetSPClusterService UnitVmModel - Subscribe VdcQueryType.GetStoragePoolsByClusterService; introduced in http://gerrit.ovirt.org/#/c/11395/ The query subscription is needed for progress mechanism: prevents hourglass 'blinking'. Change-Id: I7c2324a8569cc4c96b89a6a62fa4764c1518adb1 Signed-off-by: Daniel Erez <derez(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/UnitVmModel.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Daniel Erez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/12603 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7c2324a8569cc4c96b89a6a62fa4764c1518adb1 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Libor Spevak <lspevak(a)redhat.com>

1 0

Change in ovirt-engine[master]: webadmin: Optimize for virt store while creating volume
by sanjal＠redhat.com 04 Mar '13

04 Mar '13

Shireesh Anjal has submitted this change and it was merged. Change subject: webadmin: Optimize for virt store while creating volume ...................................................................... webadmin: Optimize for virt store while creating volume 'Optimize for Virt Store' checkbox is added to the Create Volume dialog. Change-Id: I98e2e636a67e0b7f204e45a3e3d39bdec96eeeb0 Bug-Url: https://bugzilla.redhat.com/891493 Signed-off-by: Kanagaraj M <kmayilsa(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/gluster/VolumeModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/volumes/VolumeListModel.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/ApplicationConstants.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/gluster/VolumePopupView.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/gluster/VolumePopupView.ui.xml 5 files changed, 53 insertions(+), 13 deletions(-) Approvals: Shireesh Anjal: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/12490 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I98e2e636a67e0b7f204e45a3e3d39bdec96eeeb0 Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Shireesh Anjal <sanjal(a)redhat.com>

1 0

Change in ovirt-engine[master]: engine: Added missing port for Gluster operations.
by sanjal＠redhat.com 04 Mar '13

04 Mar '13

Shireesh Anjal has submitted this change and it was merged. Change subject: engine: Added missing port for Gluster operations. ...................................................................... engine: Added missing port for Gluster operations. Opening port 111 for accessing gluster volume using nfs. Gluster IP table configuration has been overridden to open port 111. Change-Id: Ibae2d3ebda0e35649c38cd8cd168fc79ba05ba9f Bug-Url: https://bugzilla.redhat.com/906925 Signed-off-by: Selvasundaram <sesubram(a)redhat.com> --- M backend/manager/dbscripts/upgrade/pre_upgrade/0000_config.sql 1 file changed, 33 insertions(+), 0 deletions(-) Approvals: Shireesh Anjal: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/11804 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibae2d3ebda0e35649c38cd8cd168fc79ba05ba9f Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Selvasundaram <sesubram(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Selvasundaram <sesubram(a)redhat.com> Gerrit-Reviewer: Shireesh Anjal <sanjal(a)redhat.com>

1 0

Change in ovirt-engine[master]: webadmin: check UP server before adding bricks
by sanjal＠redhat.com 04 Mar '13

04 Mar '13

Shireesh Anjal has submitted this change and it was merged. Change subject: webadmin: check UP server before adding bricks ...................................................................... webadmin: check UP server before adding bricks While adding the bricks after creating a volume, Add Bricks dialog will be shown only if the cluster has a host in UP status. Otherwise an error message denoting no up server found in the cluster will be shown. Change-Id: I54b3c0eacbe4c94929d20738196324569ed8f3b6 Bug-Url: https://bugzilla.redhat.com/907338 Signed-off-by: Kanagaraj M <kmayilsa(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/gluster/VolumeBrickListModel.java M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/Constants.java 2 files changed, 35 insertions(+), 7 deletions(-) Approvals: Shireesh Anjal: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/11927 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I54b3c0eacbe4c94929d20738196324569ed8f3b6 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Shireesh Anjal <sanjal(a)redhat.com>

1 0

Change in ovirt-engine[master]: engine: Adding vds group name field to GlusterVolumeEntity
by sanjal＠redhat.com 04 Mar '13

04 Mar '13

Shireesh Anjal has submitted this change and it was merged. Change subject: engine: Adding vds group name field to GlusterVolumeEntity ...................................................................... engine: Adding vds group name field to GlusterVolumeEntity A new view named 'gluster_volumes_view' is added by joining 'gluster_volumes' and 'vds_groups' tables to get the cluster name of the volumes. gluster volume functions are modified to use the 'gluster_volumes_view' instead of 'gluster_volumes'. 'vdsGroupName' field is added to the GlusterVolumeEntity and it will shown in the 'Cluster' column of Volumes tab in the UI. Change-Id: I6b98e7b46e312d0dc0b9b29b7cb8a0b0a7ecd3b7 Bug-Url: https://bugzilla.redhat.com/875800 Signed-off-by: Kanagaraj M <kmayilsa(a)redhat.com> --- M backend/manager/dbscripts/create_views.sql M backend/manager/dbscripts/gluster_volumes_sp.sql M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/gluster/GlusterVolumeEntity.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/gluster/GlusterVolumeDaoDbFacadeImpl.java M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/gluster/GlusterVolumeDaoTest.java M backend/manager/modules/searchbackend/src/main/java/org/ovirt/engine/core/searchbackend/SearchObjectAutoCompleter.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/ApplicationConstants.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/MainTabVolumeView.java 8 files changed, 43 insertions(+), 14 deletions(-) Approvals: Shireesh Anjal: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/12461 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6b98e7b46e312d0dc0b9b29b7cb8a0b0a7ecd3b7 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Shireesh Anjal <sanjal(a)redhat.com>

1 0

Change in ovirt-engine[master]: engine: Removing flood message from engine.log
by mkublin＠redhat.com 04 Mar '13

04 Mar '13

Michael Kublin has submitted this change and it was merged. Change subject: engine: Removing flood message from engine.log ...................................................................... engine: Removing flood message from engine.log The following message is written to engine.log when we are trying to add some audit log message and it is illegal - meaning it should be skipped, the message is misleading and also for case of very common event we just flooding engine.log Change-Id: I0ec84555c0473a2e051d68f27cbbbbbd84fa6c9a Signed-off-by: Michael Kublin <mkublin(a)redhat.com> --- M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/dbbroker/auditloghandling/AuditLogDirector.java 1 file changed, 0 insertions(+), 2 deletions(-) Approvals: Michael Kublin: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/12521 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0ec84555c0473a2e051d68f27cbbbbbd84fa6c9a Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Michael Kublin <mkublin(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Michael Kublin <mkublin(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine-sdk[master]: sdk: support 3+ lengh URI methods gen
by mpastern＠redhat.com 04 Mar '13

04 Mar '13

Michael Pasternak has submitted this change and it was merged. Change subject: sdk: support 3+ lengh URI methods gen ...................................................................... sdk: support 3+ lengh URI methods gen Change-Id: I61cfc092d7cde139949f31890e1446468b459434 Signed-off-by: Michael Pasternak <mpastern(a)redhat.com> --- M src/codegen/rsdl/rsdlcodegen.py M src/ovirtsdk/infrastructure/brokers.py 2 files changed, 73 insertions(+), 67 deletions(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/12673 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I61cfc092d7cde139949f31890e1446468b459434 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine-sdk Gerrit-Branch: master Gerrit-Owner: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com>

1 0

Change in ovirt-engine[master]: core:VM pool snapshot should not be stateless.
by mlipchuk＠redhat.com 04 Mar '13

04 Mar '13

Maor Lipchuk has submitted this change and it was merged. Change subject: core:VM pool snapshot should not be stateless. ...................................................................... core:VM pool snapshot should not be stateless. After upgrading from version 3.0, the snapshots of VMs that are part of a pool are marked as stateless although they aren't. The proposed fix is to add a new script which fixes the already upgraded VMs. Change-Id: I9aaccd3c624c2ab6bcc6f4a0510b26d37f2c1eee Bug-Url: https://bugzilla.redhat.com/882621 Signed-off-by: Maor Lipchuk <mlipchuk(a)redhat.com> --- A backend/manager/dbscripts/upgrade/03_02_0480_fix_stateless_snapshot.sql 1 file changed, 7 insertions(+), 0 deletions(-) Approvals: Maor Lipchuk: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9794 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9aaccd3c624c2ab6bcc6f4a0510b26d37f2c1eee Gerrit-PatchSet: 7 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Alissa Bonas <abonas(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Ayal Baron <abaron(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Vered Volansky <vvolansk(a)redhat.com>

1 0

Change in ovirt-engine-sdk[master]: sdk: regenerate against latest api
by mpastern＠redhat.com 04 Mar '13

04 Mar '13

Michael Pasternak has submitted this change and it was merged. Change subject: sdk: regenerate against latest api ...................................................................... sdk: regenerate against latest api - to cluster added tunnel_migration - to display added keyboard_layout - to display added smartcard_enabled - to DataCenter added Clusters sub-collection - to StorageDomain added Disks sub-collection - to Disks.list() added 'unregistered' argument - to Host added display.address - to Snapshot added commit/preview/undo actions - to vms.add() added overload for creating vm from snapshot Change-Id: I58f893d447663d26aaf15931b71d29ea52f05c5e Signed-off-by: Michael Pasternak <mpastern(a)redhat.com> --- M src/ovirtsdk/infrastructure/brokers.py M src/ovirtsdk/xml/params.py 2 files changed, 2,690 insertions(+), 74 deletions(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/12672 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I58f893d447663d26aaf15931b71d29ea52f05c5e Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine-sdk Gerrit-Branch: master Gerrit-Owner: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com>

1 0

Change in ovirt-engine[master]: packaging: Move passwd encryption script to tools
by juan.hernandez＠redhat.com 04 Mar '13

04 Mar '13

Juan Hernandez has submitted this change and it was merged. Change subject: packaging: Move passwd encryption script to tools ...................................................................... packaging: Move passwd encryption script to tools This script used to be in the PKI directory, but it is a tool used outside of the PKI environment, so it is better placed in the bin directory and owned by the tools package. Change-Id: Ie0732b58043925d7931daa53b51d90310a6964f2 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M Makefile D backend/manager/conf/ca/encryptpasswd.sh A backend/manager/tools/src/main/shell/engine-encrypt-passwd.sh M packaging/fedora/setup/basedefs.py M packaging/fedora/spec/ovirt-engine.spec.in 5 files changed, 35 insertions(+), 45 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/12450 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie0732b58043925d7931daa53b51d90310a6964f2 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com>

1 0