- Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: packaging - starting ovirt-engine only after yum rollback
by juan.hernandez＠redhat.com 06 May '13

06 May '13

Juan Hernandez has submitted this change and it was merged. Change subject: packaging - starting ovirt-engine only after yum rollback ...................................................................... packaging - starting ovirt-engine only after yum rollback With the previous implementation ovirt-engine service was started before exiting the rhyum transaction context after an execption. So the newly installed version of ovirt-engine was running with the restored PKI and DB. Now the ovirt-engine service start is done only after yum rollback transaction. Change-Id: If8160293d45361534dfda6c6aa3870e72368d10b Bug-Url: https://bugzilla.redhat.com/879969 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M packaging/fedora/setup/engine-upgrade.py 1 file changed, 133 insertions(+), 133 deletions(-) Approvals: Alon Bar-Lev: Looks good to me, but someone else must approve Sandro Bonazzola: Verified Juan Hernandez: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14387 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If8160293d45361534dfda6c6aa3870e72368d10b Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(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: Moran Goldboim <mgoldboi(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-engine[master]: engine: Ability to assign multiple VMs from a pool to single...
by tjelinek＠redhat.com 06 May '13

06 May '13

Tomas Jelinek has submitted this change and it was merged. Change subject: engine: Ability to assign multiple VMs from a pool to single user ...................................................................... engine: Ability to assign multiple VMs from a pool to single user Added a new field VmPool.maxAssignedVmsPerUser representing maximumum number of VMs a single user can attach to from the same pool. Default value of 1 is equivalent to current behavior. This patch also includes necessary minimal frontend changes in userportal due to the logic of showing/hiding VmPools in userportal. Change-Id: I3a1cc836db48d7094c2e7a03ee3a60b0b34a52a9 Signed-off-by: Martin Betak <mbetak(a)redhat.com> --- M backend/manager/dbscripts/create_views.sql A backend/manager/dbscripts/upgrade/03_03_0120_add_max_assigned_vms_per_user.sql M backend/manager/dbscripts/vm_pools_sp.sql M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AttachUserToVmFromPoolAndRunCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CommonVmPoolWithVmsCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/DetachUserFromVmFromPoolCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RemovePermissionCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VmPoolHandler.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/VmPoolSimpleUserParameters.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/VmPoolUserParameters.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VmPool.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/VdcBllMessages.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VmPoolDAODbFacadeImpl.java M backend/manager/modules/dal/src/main/resources/bundles/AppErrors.properties M backend/manager/modules/dal/src/test/resources/fixtures.xml M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/userportal/UserPortalBasicListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/userportal/UserPortalListModel.java 17 files changed, 105 insertions(+), 67 deletions(-) Approvals: Tomas Jelinek: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14005 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3a1cc836db48d7094c2e7a03ee3a60b0b34a52a9 Gerrit-PatchSet: 8 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Beták <mbetak(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Martin Beták <mbetak(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com>

1 0

Change in ovirt-engine[master]: webadmin: Fix repeated logins due to UI plugin REST API hear...
by vszocs＠redhat.com 06 May '13

06 May '13

Vojtech Szocs has submitted this change and it was merged. Change subject: webadmin: Fix repeated logins due to UI plugin REST API heartbeat ...................................................................... webadmin: Fix repeated logins due to UI plugin REST API heartbeat This patch addresses a problem (behavior) of web browser sending HTTP 'Authorization' header unconditionally (always) for each request to given origin after the HTTP 'Authorization' header has been set for the first time (e.g. via XmlHttpRequest). This generally means the JavaScript application doesn't have full control over HTTP 'Authorization' header; once set for the initial request, web browser will always send this header until the browser window is closed by the user. To address this problem in UI plugin REST API integration: 1. all plugins will receive single session ID using GUI (WebAdmin) user credentials, session timeout is set to 6 hours 2. WebAdmin will not try to keep-alive the session via periodic heartbeat requests due to the problem with HTTP 'Authorization' header handling in web browser These changes have following implications: * REST API session will be acquired with reasonably long timeout * it's up to plugins (or other systems using the session) to keep the session alive as necessary In future, we should work around the HTTP 'Authorization' header problem and revisit the general contract of UI plugin REST API integration, i.e. whether to keep-alive the session by WebAdmin, or whether to push session acquiry responsibility to individual plugins. Change-Id: I72c2d4952daac4daa17554b7661ed775c72cb97a Bug-Url: https://bugzilla.redhat.com/894687 Bug-Url: https://bugzilla.redhat.com/906046 Signed-off-by: Vojtech Szocs <vszocs(a)redhat.com> --- M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/plugin/restapi/RestApiSessionManager.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/system/ApplicationInit.java 2 files changed, 28 insertions(+), 59 deletions(-) Approvals: Vojtech Szocs: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14411 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I72c2d4952daac4daa17554b7661ed775c72cb97a Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Einav Cohen <ecohen(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com>

1 0

Change in ovirt-engine[master]: webdamin: make the right side of vm->interface sub tab resiz...
by alkaplan＠redhat.com 06 May '13

06 May '13

Alona Kaplan has submitted this change and it was merged. Change subject: webdamin: make the right side of vm->interface sub tab resizable ...................................................................... webdamin: make the right side of vm->interface sub tab resizable Make the columns of the statistics and guest agent tabs (which are based on EntityModelCellTable) resizable. Change-Id: Ic7b36139e522abe7805a8b09d803d06e6a87e685 Signed-off-by: Alona Kaplan <alkaplan(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/editor/EntityModelCellTable.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/vm/VmInterfaceInfoPanel.java 2 files changed, 13 insertions(+), 7 deletions(-) Approvals: Alona Kaplan: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14366 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic7b36139e522abe7805a8b09d803d06e6a87e685 Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Einav Cohen <ecohen(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com>

1 0

Change in ovirt-engine[master]: engine: Fixing NPE in CollectVdsNetworkDataVDSCommand
by mkublin＠redhat.com 06 May '13

06 May '13

Michael Kublin has submitted this change and it was merged. Change subject: engine: Fixing NPE in CollectVdsNetworkDataVDSCommand ...................................................................... engine: Fixing NPE in CollectVdsNetworkDataVDSCommand The NPE is occuring because of some changes in Base class. As a fix, start to pass a VDS as parameter and not vdsId Change-Id: I9d3027b134e11e9fb22cd007d10ce4b9fd8afbb6 Signed-off-by: Michael Kublin <mkublin(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/host/AddBondCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/host/AttachNetworkToVdsInterfaceCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/host/DetachNetworkFromVdsInterfaceCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/host/RemoveBondCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/host/SetupNetworksCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/host/UpdateNetworkToVdsInterfaceCommand.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/CollectVdsNetworkDataVDSCommand.java 7 files changed, 11 insertions(+), 16 deletions(-) Approvals: Michael Kublin: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14501 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9d3027b134e11e9fb22cd007d10ce4b9fd8afbb6 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Michael Kublin <mkublin(a)redhat.com> Gerrit-Reviewer: Michael Kublin <mkublin(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: Introducing findbugs filter to bll module
by yzaslavs＠redhat.com 06 May '13

06 May '13

Yair Zaslavsky has submitted this change and it was merged. Change subject: core: Introducing findbugs filter to bll module ...................................................................... core: Introducing findbugs filter to bll module Introducing findbugs to bll module. In order to generate findbugs report use "mvn site" and look for findbugs.xml at the target/site directory (this may take several minutes) Marking as WIP as obviously this should be implemented to other modules. I would like you to review and share your opinions Thanks Change-Id: I26b9cafc1832022693a7386277972a7e48a53f0d Signed-off-by: Yair Zaslavsky <yzaslavs(a)redhat.com> --- A backend/manager/modules/bll/exclude-filters.xml M backend/manager/modules/bll/pom.xml 2 files changed, 28 insertions(+), 0 deletions(-) Approvals: Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14110 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I26b9cafc1832022693a7386277972a7e48a53f0d Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Laszlo Hornyak <lhornyak(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Mike Kolesnik <mkolesni(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine[master]: frontend: Fixed default color depth for spice-activex
by tjelinek＠redhat.com 06 May '13

06 May '13

Tomas Jelinek has submitted this change and it was merged. Change subject: frontend: Fixed default color depth for spice-activex ...................................................................... frontend: Fixed default color depth for spice-activex This patch fixes setting default value of ColorDepth attribute of spice browser plugin. For spice-xpi the value is "", whereas for spice-activex the value is 0. Change-Id: I58af05c0a2ff88dccb4634ec9a81dff84afe92a5 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=913183 Signed-off-by: Frantisek Kobzik <fkobzik(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/uicommon/SpicePluginImpl.java 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Tomas Jelinek: Looks good to me, approved Frank Kobzik: Verified -- To view, visit http://gerrit.ovirt.org/14502 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I58af05c0a2ff88dccb4634ec9a81dff84afe92a5 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Frank Kobzik <fkobzik(a)redhat.com> Gerrit-Reviewer: Frank Kobzik <fkobzik(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com>

1 0

Change in ovirt-engine[master]: engine: search remove redundant constructor param (1/N)
by lhornyak＠redhat.com 06 May '13

06 May '13

Laszlo Hornyak has submitted this change and it was merged. Change subject: engine: search remove redundant constructor param (1/N) ...................................................................... engine: search remove redundant constructor param (1/N) Removed a parameter from AutoCompletionField constructor, it was practically always equal to another parameter. Change-Id: I28129e1549232ba33db8b91ed9c5bd8061ad24f2 Signed-off-by: Laszlo Hornyak <lhornyak(a)redhat.com> --- M backend/manager/modules/searchbackend/src/main/java/org/ovirt/engine/core/searchbackend/gluster/GlusterVolumeConditionFieldAutoCompleter.java 1 file changed, 3 insertions(+), 5 deletions(-) Approvals: Laszlo Hornyak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14405 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I28129e1549232ba33db8b91ed9c5bd8061ad24f2 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Laszlo Hornyak <lhornyak(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Laszlo Hornyak <lhornyak(a)redhat.com> Gerrit-Reviewer: Shireesh Anjal <sanjal(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine[master]: userportal, webadmin: bring back change CD menu to SPICE cons...
by tjelinek＠redhat.com 06 May '13

06 May '13

Tomas Jelinek has submitted this change and it was merged. Change subject: userportal,webadmin: bring back change CD menu to SPICE console ...................................................................... userportal,webadmin: bring back change CD menu to SPICE console This has been lost while moving from 3.0 to 3.1. This patch contains: - setting the correct menu to the ActiveX plugin (XPI plugin does not have this functionality) - listening to the onmenuitemselected event and firing the correct event in response - fix the menu item creation - new StringBuilder(int) just creates the empty string (int param means the capacity, not the actual content) - fix the format of the menu - removed items which are not handled by us anymore, but handled by the spice client itself Change-Id: Ifdfa982da00ae333271f60795b0c17fe602e14ef Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=957611 Signed-off-by: Tomas Jelinek <tjelinek(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/uicommon/SpicePluginImpl.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/SpiceConsoleModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/SpiceMenu.java 3 files changed, 11 insertions(+), 27 deletions(-) Approvals: Tomas Jelinek: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14407 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifdfa982da00ae333271f60795b0c17fe602e14ef Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com>

1 0

Change in ovirt-engine[master]: findbugs: EndedTaskInfo defines non-serializable instance fi...
by shavivi＠redhat.com 06 May '13

06 May '13

Shahar Havivi has submitted this change and it was merged. Change subject: findbugs: EndedTaskInfo defines non-serializable instance field ...................................................................... findbugs: EndedTaskInfo defines non-serializable instance field Change-Id: I41ca491586d360f064fcee99165ee8486531c816 Signed-off-by: Shahar Havivi <shavivi(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/asynctasks/AsyncTaskParameters.java 1 file changed, 3 insertions(+), 1 deletion(-) Approvals: Shahar Havivi: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14317 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I41ca491586d360f064fcee99165ee8486531c816 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Asaf Shakarchi <asaf(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0