March 2014 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: webadmin: Filter domains on disk move dialog from main disk tab
by tnisan＠redhat.com 23 Mar '14

23 Mar '14

Tal Nisan has submitted this change and it was merged. Change subject: webadmin: Filter domains on disk move dialog from main disk tab ...................................................................... webadmin: Filter domains on disk move dialog from main disk tab Since we currently do not support live storage migration from block domains to file domains and vice versa, the destination domains list will be filtered when moving a disk that is plugged to a running VM (from the main disks tab) Change-Id: Ia28d3acd7ceeaff07745f16af920f8d225e485c7 Signed-off-by: Tal Nisan <tnisan(a)redhat.com> Bug-Url: https://bugzilla.redhat.com/1073343 --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/MoveOrCopyDiskModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/DiskModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/MoveDiskModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmDiskListModel.java 4 files changed, 59 insertions(+), 21 deletions(-) Approvals: Tal Nisan: Verified Daniel Erez: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/25777 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia28d3acd7ceeaff07745f16af920f8d225e485c7 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tal Nisan <tnisan(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[ovirt-engine-3.4]: core: prevent guaranteed memory bigger than memory size
by oourfali＠redhat.com 23 Mar '14

23 Mar '14

Oved Ourfali has submitted this change and it was merged. Change subject: core: prevent guaranteed memory bigger than memory size ...................................................................... core: prevent guaranteed memory bigger than memory size Change-Id: I75a4b997fc4d3fe2484fa4189ec6ca1e6cf01079 Bug-Url: https://bugzilla.redhat.com/1045131 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVmCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVmCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVmTemplateCommand.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/errors/VdcBllMessages.java M backend/manager/modules/dal/src/main/resources/bundles/AppErrors.properties M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/AppErrors.java M frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties 8 files changed, 19 insertions(+), 0 deletions(-) Approvals: Gilad Chaplik: Looks good to me, approved Arik Hadas: Verified -- To view, visit http://gerrit.ovirt.org/25802 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I75a4b997fc4d3fe2484fa4189ec6ca1e6cf01079 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.4 Gerrit-Owner: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(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 ovirt-engine[master]: aaa: Fix apperance of empty profile
by yzaslavs＠redhat.com 23 Mar '14

23 Mar '14

Yair Zaslavsky has submitted this change and it was merged. Change subject: aaa: Fix apperance of empty profile ...................................................................... aaa: Fix apperance of empty profile Change-Id: Icbf9384f40ad061ec35c03c4fbcbd1dd70085526 Topic: AAA Signed-off-by: Yair Zaslavsky <yzaslavs(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/InitBackendServicesOnStartupBean.java 1 file changed, 22 insertions(+), 19 deletions(-) Approvals: Alon Bar-Lev: Looks good to me, but someone else must approve Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/25993 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icbf9384f40ad061ec35c03c4fbcbd1dd70085526 Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: engine: Changing HA Reservation alert interval to 10 min
by gchaplik＠redhat.com 23 Mar '14

23 Mar '14

Gilad Chaplik has submitted this change and it was merged. Change subject: engine: Changing HA Reservation alert interval to 10 min ...................................................................... engine: Changing HA Reservation alert interval to 10 min Changing the interval which an alert of HA reservation can be invoked from 1 hour to 10 min Change-Id: Icb7665c335a00ac6cee67e1293d8a93409d79ca2 Bug-Url: https://bugzilla.redhat.com/1076211 Signed-off-by: Kobi Ianko <kobi(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/AuditLogType.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Jiří Moskovčák: Looks good to me, but someone else must approve Gilad Chaplik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/25935 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icb7665c335a00ac6cee67e1293d8a93409d79ca2 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Kobi Ianko <kobi(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Jiří Moskovčák <jmoskovc(a)redhat.com> Gerrit-Reviewer: Kobi Ianko <kobi(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: Engine: Enable Cloud-Init SSH login via public key
by shavivi＠redhat.com 23 Mar '14

23 Mar '14

Shahar Havivi has submitted this change and it was merged. Change subject: Engine: Enable Cloud-Init SSH login via public key ...................................................................... Engine: Enable Cloud-Init SSH login via public key Fixes: 1. Root is able to login via public key and password 2. Root password is not expired, i.e. no need to change password first login (only if password was set) Change-Id: Idb48fe4306f803aca801c03ddf29f010eaeb635f Signed-off-by: Shahar Havivi <shaharh(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/CloudInitHandler.java 1 file changed, 7 insertions(+), 1 deletion(-) Approvals: Shahar Havivi: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/25625 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Idb48fe4306f803aca801c03ddf29f010eaeb635f Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: [RFE] Change the Slot field to...
by emesika＠redhat.com 23 Mar '14

23 Mar '14

Eli Mesika has submitted this change and it was merged. Change subject: core: [RFE] Change the Slot field to... ...................................................................... core: [RFE] Change the Slot field to... [RFE] Change the Slot field to "Service Profile" when cisco_ucs is selected as the fencing type Change-Id: Ic572e678decad8d032a6400796cf29baf9417093 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=804530 Signed-off-by: Eli Mesika <emesika(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostModel.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/presenter/popup/host/HostPopupPresenterWidget.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/HostPopupView.java 4 files changed, 74 insertions(+), 14 deletions(-) Approvals: Eli Mesika: Verified Vojtech Szocs: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/25969 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic572e678decad8d032a6400796cf29baf9417093 Gerrit-PatchSet: 2 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: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-dwh[master]: packaging: added version dependency on ovirt-engine
by ydary＠redhat.com 23 Mar '14

23 Mar '14

Yaniv Dary has submitted this change and it was merged. Change subject: packaging: added version dependency on ovirt-engine ...................................................................... packaging: added version dependency on ovirt-engine Change-Id: I66595659639604c11991c9aba4d42ecd524545ca Related-To: https://bugzilla.redhat.com/1059283 Signed-off-by: Yaniv Dary <ydary(a)redhat.com> --- M ovirt-engine-dwh.spec.in 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Alon Bar-Lev: Looks good to me, but someone else must approve Yaniv Dary: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/25991 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I66595659639604c11991c9aba4d42ecd524545ca Gerrit-PatchSet: 1 Gerrit-Project: ovirt-dwh Gerrit-Branch: master Gerrit-Owner: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.4]: packaging: upgrade-check branding support
by oourfali＠redhat.com 23 Mar '14

23 Mar '14

Oved Ourfali has submitted this change and it was merged. Change subject: packaging: upgrade-check branding support ...................................................................... packaging: upgrade-check branding support Added branding support to ovirt-engine-upgrade-check Change-Id: I9e44cad12a27106c94610d9084b89e6bf1b8dae5 Bug-Url: https://bugzilla.redhat.com/1078955 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> (cherry picked from commit c1b516d7296e2dcabb4f93ca27882e9f3429ae8b) --- M packaging/setup/bin/ovirt-engine-upgrade-check 1 file changed, 78 insertions(+), 3 deletions(-) Approvals: Alon Bar-Lev: Looks good to me, but someone else must approve Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/25981 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9e44cad12a27106c94610d9084b89e6bf1b8dae5 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.4 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(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]: webadmin: change few queries to be not refreshable
by oourfali＠redhat.com 23 Mar '14

23 Mar '14

Oved Ourfali has submitted this change and it was merged. Change subject: webadmin: change few queries to be not refreshable ...................................................................... webadmin: change few queries to be not refreshable The automatic logout did not work when the snapshot sub-tab was shown. It did not work because queries which were executed in this state were invoked as refreshable, thus preventing the session from being terminated automatically. So the solution is to change those queries to be not refreshable. Change-Id: I6c367c274bfc518f920344a57d7efd3fefd75d30 Bug-Url: https://bugzilla.redhat.com/1058629 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/dataprovider/AsyncDataProvider.java 1 file changed, 12 insertions(+), 5 deletions(-) Approvals: Daniel Erez: Looks good to me, approved Arik Hadas: Verified -- To view, visit http://gerrit.ovirt.org/25956 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6c367c274bfc518f920344a57d7efd3fefd75d30 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.4 Gerrit-Owner: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(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[ovirt-engine-3.4]: packaging: update otopi, ovirt-host-deploy dependencies
by oourfali＠redhat.com 23 Mar '14

23 Mar '14

Oved Ourfali has submitted this change and it was merged. Change subject: packaging: update otopi, ovirt-host-deploy dependencies ...................................................................... packaging: update otopi, ovirt-host-deploy dependencies Change-Id: I178a9874a329165efd958adabd94c3982d18a573 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M ovirt-engine.spec.in M pom.xml 2 files changed, 6 insertions(+), 6 deletions(-) Approvals: Alon Bar-Lev: Verified Sandro Bonazzola: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/25973 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I178a9874a329165efd958adabd94c3982d18a573 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.4 Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0