August 2013 - Engine-commits - oVirt List Archives

Change in ovirt-hosted-engine-setup[master]: vdsm: support latest VDSM API connection changes
by sbonazzo＠redhat.com 07 Aug '13

07 Aug '13

Sandro Bonazzola has submitted this change and it was merged. Change subject: vdsm: support latest VDSM API connection changes ...................................................................... vdsm: support latest VDSM API connection changes vdsClient has changed the API for connecting to the daemon with the following patch: http://gerrit.ovirt.org/#/c/16225/2 Adding support for the new API keeping compatibility with previous version. Change-Id: I1f08dc344d8dd1c46de79fbd746b9d89d89b70d6 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M src/plugins/ovirt-hosted-engine-setup/system/vdsmenv.py 1 file changed, 9 insertions(+), 4 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17530 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1f08dc344d8dd1c46de79fbd746b9d89d89b70d6 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Hunt Xu <mhuntxu(a)gmail.com> Gerrit-Reviewer: Lev Veyde <lveyde(a)gmail.com> Gerrit-Reviewer: Moran Goldboim <mgoldboi(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>

1 0

Change in ovirt-hosted-engine-setup[master]: packaging: setup: support OVF import
by sbonazzo＠redhat.com 07 Aug '13

07 Aug '13

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: support OVF import ...................................................................... packaging: setup: support OVF import At customization stage allow to select disk as boot media. With disk boot media will be asked a path to a valid OVF archive. The archive will be validated and needed VM configuration parameters will be retrieved from the OVF XML file. The VM configuration will be validated and the image will be imported. Change-Id: I04b1de72c19e558b20d46e9fe6a435c594b0005d Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M src/ovirt_hosted_engine_setup/constants.py M src/plugins/ovirt-hosted-engine-setup/engine/os_install.py M src/plugins/ovirt-hosted-engine-setup/vm/Makefile.am M src/plugins/ovirt-hosted-engine-setup/vm/__init__.py A src/plugins/ovirt-hosted-engine-setup/vm/boot_disk.py M src/plugins/ovirt-hosted-engine-setup/vm/configurevm.py M src/plugins/ovirt-hosted-engine-setup/vm/cpu.py M src/plugins/ovirt-hosted-engine-setup/vm/image.py M src/plugins/ovirt-hosted-engine-setup/vm/memory.py M src/plugins/ovirt-hosted-engine-setup/vm/runvm.py 10 files changed, 438 insertions(+), 9 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17521 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I04b1de72c19e558b20d46e9fe6a435c594b0005d Gerrit-PatchSet: 6 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Federico Simoncelli <fsimonce(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Lev Veyde <lveyde(a)gmail.com> Gerrit-Reviewer: Moran Goldboim <mgoldboi(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>

1 0

Change in ovirt-engine[ovirt-engine-3.3]: packaging: setup: Do not enable Gluster in Default cluster
by oschreib＠redhat.com 07 Aug '13

07 Aug '13

Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: setup: Do not enable Gluster in Default cluster ...................................................................... packaging: setup: Do not enable Gluster in Default cluster DEFAULT cluster is created during installation of engine. If Application mode is selected as BOTH during installation, DEFAULT cluster will be created with both Gluster and Virt enabled by default. By default only virt will be enabled in DEFAULT cluster. (cherry picked from commit 62d8878e7d3233bfb2dac861262efba24b428ccf) Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=988453 Change-Id: Ic8b4d15689c730428aad09dd0c981faf5732731d Signed-off-by: Aravinda VK <avishwan(a)redhat.com> Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/fedora/setup/engine-setup.py M packaging/setup/plugins/ovirt-engine-setup/config/appmode.py 2 files changed, 27 insertions(+), 12 deletions(-) Approvals: Aravinda VK: Verified Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17736 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic8b4d15689c730428aad09dd0c981faf5732731d Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3 Gerrit-Owner: Aravinda VK <avishwan(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Aravinda VK <avishwan(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-hosted-engine-setup[master]: Added OVF definition schema files
by sbonazzo＠redhat.com 07 Aug '13

07 Aug '13

Sandro Bonazzola has submitted this change and it was merged. Change subject: Added OVF definition schema files ...................................................................... Added OVF definition schema files Added OVF definition schema files and python interface generated using generateDS. Change-Id: I7f9f2b1cd5999c6252cd45bd3363eed9b9db7e13 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M configure.ac M ovirt-hosted-engine-setup.spec.in M src/ovirt_hosted_engine_setup/Makefile.am A src/ovirt_hosted_engine_setup/ovf/CIM_ResourceAllocationSettingData.xsd A src/ovirt_hosted_engine_setup/ovf/CIM_VirtualSystemSettingData.xsd A src/ovirt_hosted_engine_setup/ovf/Makefile.am A src/ovirt_hosted_engine_setup/ovf/__init__.py A src/ovirt_hosted_engine_setup/ovf/common.xsd A src/ovirt_hosted_engine_setup/ovf/dsp8023_1.1.0.xsd A src/ovirt_hosted_engine_setup/ovf/dsp8027_1.1.0.xsd A src/ovirt_hosted_engine_setup/ovf/ovfenvelope.py 11 files changed, 10,802 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Yedidyah Bar David: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/17520 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7f9f2b1cd5999c6252cd45bd3363eed9b9db7e13 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Lev Veyde <lveyde(a)gmail.com> Gerrit-Reviewer: Moran Goldboim <mgoldboi(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: no repeated steps for task handlers
by amureini＠redhat.com 06 Aug '13

06 Aug '13

Allon Mureinik has submitted this change and it was merged. Change subject: core: no repeated steps for task handlers ...................................................................... core: no repeated steps for task handlers Change-Id: I0b68bcefcb23fcbe7a4b01e32d21a846b7d16d7c Signed-off-by: Federico Simoncelli <fsimonce(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CommandBase.java 1 file changed, 6 insertions(+), 2 deletions(-) Approvals: Allon Mureinik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17673 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0b68bcefcb23fcbe7a4b01e32d21a846b7d16d7c Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Federico Simoncelli <fsimonce(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Federico Simoncelli <fsimonce(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: backend: pass context from ImportRepoImage
by amureini＠redhat.com 06 Aug '13

06 Aug '13

Allon Mureinik has submitted this change and it was merged. Change subject: backend: pass context from ImportRepoImage ...................................................................... backend: pass context from ImportRepoImage Change-Id: Id91fd1dab1102a64b348f8a203b4413f624e75ad Signed-off-by: Federico Simoncelli <fsimonce(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ImportRepoImageCreateTaskHandler.java 1 file changed, 2 insertions(+), 1 deletion(-) Approvals: Allon Mureinik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17674 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id91fd1dab1102a64b348f8a203b4413f624e75ad Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Federico Simoncelli <fsimonce(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Federico Simoncelli <fsimonce(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: restapi: add storage domain with existing conn id
by amureini＠redhat.com 06 Aug '13

06 Aug '13

Allon Mureinik has submitted this change and it was merged. Change subject: restapi: add storage domain with existing conn id ...................................................................... restapi: add storage domain with existing conn id Add ability to create a new storage domain for file domains with reference to an existing storage connection that was previously created. Also, added a backend validation that the given connection is not used by other storage domains. Before this patch, the storage connection was created along with storage domain creation. See also: http://www.ovirt.org/Features/Manage_Storage_Connections#Add_new_file_.28nf… Change-Id: Icb0495be711f80d71ad5334080228faee03d03dc Signed-off-by: Alissa Bonas <abonas(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/AddStorageDomainCommon.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/StorageDomainCommandBase.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendStorageDomainsResource.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendStorageDomainsResourceTest.java 4 files changed, 149 insertions(+), 25 deletions(-) Approvals: Allon Mureinik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17177 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icb0495be711f80d71ad5334080228faee03d03dc Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alissa Bonas <abonas(a)redhat.com> Gerrit-Reviewer: Alissa Bonas <abonas(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: restapi: allow storage conn deletion with no host
by amureini＠redhat.com 06 Aug '13

06 Aug '13

Allon Mureinik has submitted this change and it was merged. Change subject: restapi: allow storage conn deletion with no host ...................................................................... restapi: allow storage conn deletion with no host Allow deletion of storage connection without specifying the host. Before this change, deletion of storage connection required host id/name in order to perform a disconnect operation as part of the connection deletion. This change allows deleting the connection from db without the need to disconnect the host from it, since the connection could have been created without connecting the host to it. The option to delete connection and disconnect it via the host remains supported as well. Feature Page: http://www.ovirt.org/Features/Manage_Storage_Connections Change-Id: I7cd9dcea95028e12872beb2496c4338654cfbdb4 Signed-off-by: Alissa Bonas <abonas(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/StorageServerConnectionsResource.java 1 file changed, 15 insertions(+), 0 deletions(-) Approvals: Allon Mureinik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17568 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7cd9dcea95028e12872beb2496c4338654cfbdb4 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alissa Bonas <abonas(a)redhat.com> Gerrit-Reviewer: Alissa Bonas <abonas(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: remove unnecessary allocation of Guid
by amureini＠redhat.com 06 Aug '13

06 Aug '13

Allon Mureinik has submitted this change and it was merged. Change subject: core: remove unnecessary allocation of Guid ...................................................................... core: remove unnecessary allocation of Guid Remove unnecessary allocation and cloning of Guid when assigning id values to objects. Removed cloning in 2 places, in third place its usage was in unused method, so removed the method. Change-Id: I857b4fe7ad1f1dcff486e489e262441823a6cb3e Signed-off-by: Alissa Bonas <abonas(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AutoRecoveryManager.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/DiskImage.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/StorageDomainDynamic.java 3 files changed, 5 insertions(+), 12 deletions(-) Approvals: Allon Mureinik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17540 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I857b4fe7ad1f1dcff486e489e262441823a6cb3e Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alissa Bonas <abonas(a)redhat.com> Gerrit-Reviewer: Alissa Bonas <abonas(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Sergey Gotliv <sgotliv(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: use runVdsCommand method
by amureini＠redhat.com 06 Aug '13

06 Aug '13

Allon Mureinik has submitted this change and it was merged. Change subject: core: use runVdsCommand method ...................................................................... core: use runVdsCommand method Use the inherited runVdsCommand method (from CommandBase) for a shorter and more readable code in ConnectStorageToVdsCommand. Change-Id: Ib35bcfca36fa9d51c7de37a209bccd8e8e5303bb Signed-off-by: Alissa Bonas <abonas(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/ConnectStorageToVdsCommand.java 1 file changed, 1 insertion(+), 4 deletions(-) Approvals: Allon Mureinik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17670 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib35bcfca36fa9d51c7de37a209bccd8e8e5303bb Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alissa Bonas <abonas(a)redhat.com> Gerrit-Reviewer: Alissa Bonas <abonas(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Liron Ar <laravot(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Sergey Gotliv <sgotliv(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0