- Engine-commits - oVirt List Archives

Change in ovirt-engine[ovirt-engine-3.4]: webadmin: Fix allocation policy and storage domain when edit...
by iheim＠redhat.com 19 Feb '14

19 Feb '14

Itamar Heim has submitted this change and it was merged. Change subject: webadmin: Fix allocation policy and storage domain when editing VM disk ...................................................................... webadmin: Fix allocation policy and storage domain when editing VM disk Edit Vm disk window always displayed disk allocation policy based on the storage type instead of displaying the actual allocation policy of that disk. In case of the storage domain, first domain from the list of available storage domains was displayed instead of the storage domain of that disk. Change-Id: Ifae8052b4e37a7ad7f39eb38081268a7b6bd1a43 Bug-Url: https://bugzilla.redhat.com/1057622 Signed-off-by: Sergey Gotliv <sgotliv(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/AbstractDiskModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/EditDiskModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/NewDiskModel.java 3 files changed, 20 insertions(+), 7 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Sergey Gotliv: Verified -- To view, visit http://gerrit.ovirt.org/24686 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifae8052b4e37a7ad7f39eb38081268a7b6bd1a43 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.4 Gerrit-Owner: Sergey Gotliv <sgotliv(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Sergey Gotliv <sgotliv(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.4]: engine: Only iSCSI storage connections are viable for iSCSI ...
by iheim＠redhat.com 19 Feb '14

19 Feb '14

Itamar Heim has submitted this change and it was merged. Change subject: engine: Only iSCSI storage connections are viable for iSCSI Bond ...................................................................... engine: Only iSCSI storage connections are viable for iSCSI Bond iSCSI Bond is the new managed entity introduced in 3.4 in order to configure iSCSI multipathing. This entity is configured based on iSCSI storage connections therefore only connections of that type should be presented to the user when it configures the iSCSI Bond in UI. Change-Id: Iae30db6aa0c5d2989e9e31566d6555d70259799e Signed-off-by: Sergey Gotliv <sgotliv(a)redhat.com> --- A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/GetConnectionsByDataCenterAndStorageTypeQuery.java A backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/GetConnectionsByDataCenterAndStorageTypeParameters.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/VdcQueryType.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/datacenters/DataCenterListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/IscsiBondModel.java 6 files changed, 54 insertions(+), 9 deletions(-) Approvals: Daniel Erez: Looks good to me, approved Sergey Gotliv: Verified -- To view, visit http://gerrit.ovirt.org/24682 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iae30db6aa0c5d2989e9e31566d6555d70259799e Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.4 Gerrit-Owner: Sergey Gotliv <sgotliv(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(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[ovirt-engine-3.4]: engine: Introduce getConnectableStorageConnectionsByStorageT...
by iheim＠redhat.com 19 Feb '14

19 Feb '14

Itamar Heim has submitted this change and it was merged. Change subject: engine: Introduce getConnectableStorageConnectionsByStorageType API ...................................................................... engine: Introduce getConnectableStorageConnectionsByStorageType API This API allows to retrieve all connectable storage connections by storage type. If storage type is not specified then this API returns the same result as getAllConnectableStorageSeverConnection - all connectable storage connections. Change-Id: I9cea901ff2707bb4caa52e8f8eff06d533d5ddb6 Signed-off-by: Sergey Gotliv <sgotliv(a)redhat.com> --- M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/StorageServerConnectionDAO.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/StorageServerConnectionDAODbFacadeImpl.java M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/StorageServerConnectionDAOTest.java M packaging/dbscripts/storages_san_sp.sql 4 files changed, 64 insertions(+), 6 deletions(-) Approvals: Sergey Gotliv: Verified Liron Ar: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/24681 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9cea901ff2707bb4caa52e8f8eff06d533d5ddb6 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.4 Gerrit-Owner: Sergey Gotliv <sgotliv(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(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

Change in ovirt-engine[master]: restapi: add template versions support
by ofrenkel＠redhat.com 19 Feb '14

19 Feb '14

Omer Frenkel has submitted this change and it was merged. Change subject: restapi: add template versions support ...................................................................... restapi: add template versions support Add to templates REST resource a new section that describes template version details. The user will be able to add this in POST (add template), update version name in PUT (update) and see version details in GET requests. Feature page: http://www.ovirt.org/Features/Template_Versions Change-Id: Id8e4c498c4a1856ac341237837db40db8375c6da Signed-off-by: Alissa Bonas <abonas(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/resources/api.xsd M backend/manager/modules/restapi/interface/definition/src/main/resources/rsdl_metadata.yaml M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendTemplatesResource.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendTemplatesResourceTest.java M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/TemplateMapper.java M backend/manager/modules/restapi/types/src/test/java/org/ovirt/engine/api/restapi/types/TemplateMapperTest.java 6 files changed, 155 insertions(+), 13 deletions(-) Approvals: Omer Frenkel: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/23453 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id8e4c498c4a1856ac341237837db40db8375c6da Gerrit-PatchSet: 17 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Alissa Bonas <abonas(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: Engine: RFE: add hostname attribute for windows sysprep
by shavivi＠redhat.com 19 Feb '14

19 Feb '14

Shahar Havivi has submitted this change and it was merged. Change subject: Engine: RFE: add hostname attribute for windows sysprep ...................................................................... Engine: RFE: add hostname attribute for windows sysprep Change-Id: I3284b45f032c9c1ec3c976481351b7e81e66f0f6 Signed-off-by: Shahar Havivi <shaharh(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/CreateVmVDSCommand.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/vm/VmInitWidget.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/vm/VmInitWidget.ui.xml M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmInitModel.java 4 files changed, 42 insertions(+), 9 deletions(-) Approvals: Shahar Havivi: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/24619 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3284b45f032c9c1ec3c976481351b7e81e66f0f6 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in jenkins[master]: Adding proper database setup
by eedri＠redhat.com 19 Feb '14

19 Feb '14

Eyal Edri has submitted this change and it was merged. Change subject: Adding proper database setup ...................................................................... Adding proper database setup Jobs were failing on database setup when trying to connect to it as the pg_hba file was being removed each time. Change-Id: I1d9daebdb557880a63d8eeca5c838d28d3808b1e Signed-off-by: David Caro <dcaroest(a)redhat.com> --- M jobs/ovirt_engine_upgrade_params/update_engine_params.sh 1 file changed, 126 insertions(+), 103 deletions(-) Approvals: Eyal Edri: Looks good to me, approved David Caro: Verified -- To view, visit http://gerrit.ovirt.org/24781 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1d9daebdb557880a63d8eeca5c838d28d3808b1e Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Ewoud Kohl van Wijngaarden <ewoud(a)kohlvanwijngaarden.nl> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Kiril Nesenko <knesenko(a)redhat.com> Gerrit-Reviewer: Ohad Basan <obasan(a)redhat.com>

1 0

Change in ovirt-engine[master]: restapi: RSDL - Throw Exception For Dir Creation Error
by oliel＠redhat.com 19 Feb '14

19 Feb '14

Ori Liel has submitted this change and it was merged. Change subject: restapi: RSDL - Throw Exception For Dir Creation Error ...................................................................... restapi: RSDL - Throw Exception For Dir Creation Error Change-Id: I9c242d3ca052f6ac596cd69b03eb3f1744ead38e Signed-off-by: Ori Liel <oliel(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/rsdl/RsdlIOManager.java 1 file changed, 6 insertions(+), 2 deletions(-) Approvals: Ori Liel: Verified; Looks good to me, approved Juan Hernandez: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/24785 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9c242d3ca052f6ac596cd69b03eb3f1744ead38e Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com>

1 0

Change in ovirt-reports[master]: packaging: setup: support package customization
by ydary＠redhat.com 19 Feb '14

19 Feb '14

Yaniv Dary has submitted this change and it was merged. Change subject: packaging: setup: support package customization ...................................................................... packaging: setup: support package customization Change-Id: Iee60cede8158effd6846d8f0f559dc06d6fc787b Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- R packaging/conf/reports.xml.in M packaging/setup/ovirt_engine_setup/reportsconstants.py M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-reports/config/engine.py M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-reports/jasper/deploy.py 4 files changed, 115 insertions(+), 43 deletions(-) Approvals: Yaniv Dary: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/24600 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iee60cede8158effd6846d8f0f559dc06d6fc787b Gerrit-PatchSet: 4 Gerrit-Project: ovirt-reports Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com>

1 0

Change in ovirt-reports[master]: packaging: setup: allow setting package names via environment
by ydary＠redhat.com 19 Feb '14

19 Feb '14

Yaniv Dary has submitted this change and it was merged. Change subject: packaging: setup: allow setting package names via environment ...................................................................... packaging: setup: allow setting package names via environment Change-Id: I52c842bd319aa0a0f9f2031772e5213bc502a301 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/setup/ovirt_engine_setup/reportsconstants.py M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-reports/distro-rpm/packages.py 2 files changed, 30 insertions(+), 4 deletions(-) Approvals: Yaniv Dary: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/24599 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I52c842bd319aa0a0f9f2031772e5213bc502a301 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-reports Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com>

1 0

Change in ovirt-reports[master]: packaging: setup: admin password -> power users password
by ydary＠redhat.com 19 Feb '14

19 Feb '14

Yaniv Dary has submitted this change and it was merged. Change subject: packaging: setup: admin password -> power users password ...................................................................... packaging: setup: admin password -> power users password Change-Id: I2e16259087d0aea26997ba3a1c837fd43fb1f962 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-reports/config/admin.py 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Yaniv Dary: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/24598 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2e16259087d0aea26997ba3a1c837fd43fb1f962 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-reports Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com>

1 0