March 2015 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: core: change other OS to 64bit
by ofrenkel＠redhat.com 05 Mar '15

05 Mar '15

Omer Frenkel has submitted this change and it was merged. Change subject: core: change other OS to 64bit ...................................................................... core: change other OS to 64bit Currently the default for the "other" OS is 32bit This patch uses the following OS heirarchy and changing accordingly the "other" OS to be 64bit other other linux SUSE RH3 RH4 RH5 RH6 RH3 64 RH4 64 RH5 64 RH6 64 RH7 64 Ubuntu 1204 Debian Ubuntu 1210 Ubuntu 1304 Ubuntu 1310 Ubuntu 1404 other PPC 64 Linux PPC 64 Ubuntu PPC 64 FreeBSD FreeBSD 64 XP Win 7 Win 7 64 Win 8 64 Win 8 Win 2003 Win 2008 Win 2003 64 Win 2008 64 Win 2008 R2 64 Win 2012 64 Change-Id: Ie2c3eb1d64145b9ee372deda01168f0b9732375b Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1133642 Signed-off-by: emesika <emesika(a)redhat.com> --- M packaging/conf/osinfo-defaults.properties 1 file changed, 4 insertions(+), 4 deletions(-) Approvals: Eli Mesika: Verified Omer Frenkel: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/38280 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie2c3eb1d64145b9ee372deda01168f0b9732375b Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: return locks state in vm search
by masayag＠redhat.com 05 Mar '15

05 Mar '15

Moti Asayag has submitted this change and it was merged. Change subject: core: return locks state in vm search ...................................................................... core: return locks state in vm search Changing the search for VMs to return VM objects that contain the locks state each VM. By "locks state" we mean whether the lock is exclusive or shared, and the messages that are attached to the locks. Change-Id: I693a223b47dadb13f38fefadadc0ff778fe5ab9b Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/SearchQuery.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VmHandler.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VM.java 3 files changed, 23 insertions(+), 0 deletions(-) Approvals: Omer Frenkel: Looks good to me, approved Arik Hadas: Verified -- To view, visit https://gerrit.ovirt.org/38247 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I693a223b47dadb13f38fefadadc0ff778fe5ab9b Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in jenkins[master]: Removing rhn registration plugin to maintain repos integrity.
by sbonazzo＠redhat.com 05 Mar '15

05 Mar '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: Removing rhn registration plugin to maintain repos integrity. ...................................................................... Removing rhn registration plugin to maintain repos integrity. The rhn plugin requires the subscription manager which is not present in the repositories Change-Id: I2729f0b324676c8d7b8d5e878d5be3cf919a2672 Signed-off-by: Tolik Litovsky <tlitovsk(a)redhat.com> --- M jobs/confs/shell-scripts/build-local-ovirt-node.sh 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Anatoly Litovsky: Verified -- To view, visit https://gerrit.ovirt.org/38396 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2729f0b324676c8d7b8d5e878d5be3cf919a2672 Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Anatoly Litovsky <tlitovsk(a)redhat.com> Gerrit-Reviewer: Anatoly Litovsky <tlitovsk(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: webadmin: modify style of tables on storage domain dialog
by derez＠redhat.com 04 Mar '15

04 Mar '15

Daniel Erez has submitted this change and it was merged. Change subject: webadmin: modify style of tables on storage domain dialog ...................................................................... webadmin: modify style of tables on storage domain dialog Tune style of lun/target tables to prevent overflow. Change-Id: Ic8100dc2231be08e046a03245cb9e7a181f4a4e3 Bug-Url: https://bugzilla.redhat.com/1141755 Signed-off-by: Daniel Erez <derez(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/storage/SanStorageLunToTargetList.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/storage/SanStorageTargetToLunList.java M frontend/webadmin/modules/gwt-common/src/main/resources/org/ovirt/engine/ui/common/css/SanStorageListLunRoot.css 3 files changed, 6 insertions(+), 4 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Daniel Erez: Verified -- To view, visit https://gerrit.ovirt.org/38362 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic8100dc2231be08e046a03245cb9e7a181f4a4e3 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: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: webadmin: disable disk dialog widgets on error
by derez＠redhat.com 04 Mar '15

04 Mar '15

Daniel Erez has submitted this change and it was merged. Change subject: webadmin: disable disk dialog widgets on error ...................................................................... webadmin: disable disk dialog widgets on error Disable all the widgets of new disk dialog when creating a disk is not applicable. E.g. http://i.imgur.com/kBvem9U.jpg Change-Id: Ic3c7ea57d203d96e5c3595a4f22c0ce21ffe8136 Bug-Url: https://bugzilla.redhat.com/1141797 Signed-off-by: Daniel Erez <derez(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/AbstractModelBoundPopupWidget.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/vm/VmDiskPopupWidget.java 2 files changed, 27 insertions(+), 0 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Daniel Erez: Verified -- To view, visit https://gerrit.ovirt.org/38361 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic3c7ea57d203d96e5c3595a4f22c0ce21ffe8136 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: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: api: Add created Disk object on import response
by amureini＠redhat.com 04 Mar '15

04 Mar '15

Allon Mureinik has submitted this change and it was merged. Change subject: api: Add created Disk object on import response ...................................................................... api: Add created Disk object on import response When doing the following 2 actions: * ovirt-engine/api/openstackimageproviders/UUID/images/UUID/import * ovirt-engine/api/storagedomains/UUID/images/UUID/import with async set to false, the response returns a <disk> element containing id of the new newly created disk. Example POST data: <action> <async>false</async> <storage_domain> <name>mp2</name> </storage_domain> </action> Example POST response: <action> <async>false</async> <storage_domain> <name>mp2</name> </storage_domain> <disk href="/ovirt-engine/api/disks/UUID" id="UUID"> <actions> <link href="/ovirt-engine/api/disks/UUID/export" rel="export"/> <link href="/ovirt-engine/api/disks/UUID/move" rel="move"/> <link href="/ovirt-engine/api/disks/UUID/copy" rel="copy"/> </actions> </disk> <status> <state>complete</state> </status> </action> This patch also introduces SimpleIdResolver, an EntityResolver that receives a business logic entity class and an API model calls and resolves IDs of the business logic entities into model resource objects. Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1169228 Change-Id: Ic7a6203a318763f13fca8f0627bb0188b7eb2984 Signed-off-by: Dima Kuznetsov <dkuznets(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ImportRepoImageCreateTaskHandler.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/AbstractBackendActionableResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendStorageDomainImageResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/openstack/BackendOpenStackImageResource.java 4 files changed, 70 insertions(+), 2 deletions(-) Approvals: Juan Hernandez: Looks good to me, but someone else must approve Dima Kuznetsov: Verified Allon Mureinik: Looks good to me, approved Oved Ourfali: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/38255 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic7a6203a318763f13fca8f0627bb0188b7eb2984 Gerrit-PatchSet: 9 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Dima Kuznetsov <dkuznets(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Dima Kuznetsov <dkuznets(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Mooli Tayer <mtayer(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: Cleaup AddVmPoolWithVmsParameters
by ofrenkel＠redhat.com 04 Mar '15

04 Mar '15

Omer Frenkel has submitted this change and it was merged. Change subject: core: Cleaup AddVmPoolWithVmsParameters ...................................................................... core: Cleaup AddVmPoolWithVmsParameters Change-Id: I562ca8faf5fbdfcbadbea1c0f7f5439ca05560f4 Signed-off-by: Moti Asayag <masayag(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/AddVmPoolWithVmsParameters.java 1 file changed, 12 insertions(+), 16 deletions(-) Approvals: Omer Frenkel: Looks good to me, approved Moti Asayag: Verified -- To view, visit https://gerrit.ovirt.org/38345 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I562ca8faf5fbdfcbadbea1c0f7f5439ca05560f4 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: Cleanup AddVmToPoolParameters
by ofrenkel＠redhat.com 04 Mar '15

04 Mar '15

Omer Frenkel has submitted this change and it was merged. Change subject: core: Cleanup AddVmToPoolParameters ...................................................................... core: Cleanup AddVmToPoolParameters Change-Id: I8259d7c2d7883d4addeb8a09d24236d8bff96483 Signed-off-by: Moti Asayag <masayag(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/AddVmToPoolParameters.java 1 file changed, 7 insertions(+), 7 deletions(-) Approvals: Omer Frenkel: Looks good to me, approved Moti Asayag: Verified -- To view, visit https://gerrit.ovirt.org/38344 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8259d7c2d7883d4addeb8a09d24236d8bff96483 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: Removed unused c'tor
by ofrenkel＠redhat.com 04 Mar '15

04 Mar '15

Omer Frenkel has submitted this change and it was merged. Change subject: core: Removed unused c'tor ...................................................................... core: Removed unused c'tor Change-Id: If0ec30a5828289c9f582681d1fc9879decaf8389 Signed-off-by: Moti Asayag <masayag(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/AddVmAndAttachToPoolParameters.java 1 file changed, 0 insertions(+), 8 deletions(-) Approvals: Daniel Erez: Looks good to me, approved Omer Frenkel: Looks good to me, approved Moti Asayag: Verified -- To view, visit https://gerrit.ovirt.org/38343 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If0ec30a5828289c9f582681d1fc9879decaf8389 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in jenkins[master]: create vdsm rpms on f21, too.
by sbonazzo＠redhat.com 04 Mar '15

04 Mar '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: create vdsm rpms on f21, too. ...................................................................... create vdsm rpms on f21, too. Change-Id: I68a1dfdd86d8dbef62ef131f6e60933ff32a56dc Signed-off-by: Dan Kenigsberg <danken(a)redhat.com> --- M jobs/confs/yaml/jobs/vdsm/vdsm_create-rpms.yaml 1 file changed, 5 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/38273 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I68a1dfdd86d8dbef62ef131f6e60933ff32a56dc Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Dan Kenigsberg <danken(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0