September 2013 - Engine-commits - Ovirt List Archives

Change in ovirt-scheduler-proxy[master]: External scheduler - move to class based plugins

by dfediuck＠redhat.com

Doron Fediuck has submitted this change and it was merged. Change subject: External scheduler - move to class based plugins ...................................................................... External scheduler - move to class based plugins - modules now contain classes and do not direcly hold the functions - Move to the final function names - Added examples and a README file on plugins Signed-off-by: Noam Slomianko <nslomian(a)redhat.com> Change-Id: I94c7de6d036bd279f34c0ba246a129d88caa130a --- A plugins/README D plugins/dummy.py A plugins/examples/even_vm_distribution.py A plugins/examples/max_vm_filter.py A plugins/examples/vm_balance.py A plugins/test_plugin.py M src/ovirtscheduler/loader.py M src/ovirtscheduler/request_handler.py M src/ovirtscheduler/request_handler_test.py M src/ovirtscheduler/runner.py M src/ovirtscheduler/runner_test.py M src/ovirtscheduler/utils.py M tests/java/src/test/java/org/ovirt/schedulerproxy/SchedulerProxyTest.java 13 files changed, 313 insertions(+), 190 deletions(-) Approvals: Doron Fediuck: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/18739 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I94c7de6d036bd279f34c0ba246a129d88caa130a Gerrit-PatchSet: 7 Gerrit-Project: ovirt-scheduler-proxy Gerrit-Branch: master Gerrit-Owner: Noam Slomianko <nslomian(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Laszlo Hornyak <lhornyak(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Noam Slomianko <nslomian(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

11 years, 3 months

1
0
0 / 0

Change in ovirt-engine[ovirt-engine-3.3]: webadmin: Hide Network QoS sub-tab

by gchaplik＠redhat.com

Gilad Chaplik has submitted this change and it was merged. Change subject: webadmin: Hide Network QoS sub-tab ...................................................................... webadmin: Hide Network QoS sub-tab *** This patch is 3.3 branch only *** Hide the Network QoS sub-tab from the UI. Since the Vnic Profiles were not introduced in 3.3, Network QoS has no use. Change-Id: Ifb2f22b9f5e6e240d42c23a433d2e75ec571fafd Signed-off-by: Ofri Masad <omasad(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/DataCenterListModel.java 1 file changed, 3 insertions(+), 1 deletion(-) Approvals: Gilad Chaplik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/18734 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifb2f22b9f5e6e240d42c23a433d2e75ec571fafd Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3 Gerrit-Owner: ofri masad <omasad(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: ofri masad <omasad(a)redhat.com>

11 years, 3 months

1
0
0 / 0

Change in ovirt-engine[ovirt-engine-3.3]: webadmin: disable quota selection in edit vm pool

by gchaplik＠redhat.com

Gilad Chaplik has submitted this change and it was merged. Change subject: webadmin: disable quota selection in edit vm pool ...................................................................... webadmin: disable quota selection in edit vm pool quota selection should not be editable when editing a vm pool. fixed the selection box editability. Change-Id: Id5778ee37b1086618608edc5b470a1dbe6b5b3d8 Bug-Url: https://bugzilla.redhat.com/966003 Signed-off-by: Ofri Masad <omasad(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/pool/PoolEditPopupWidget.java 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Gilad Chaplik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/18685 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id5778ee37b1086618608edc5b470a1dbe6b5b3d8 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3 Gerrit-Owner: ofri masad <omasad(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: ofri masad <omasad(a)redhat.com>

11 years, 3 months

1
0
0 / 0

Change in ovirt-engine[master]: core: scheduling: fix ClassCastException

by dfediuck＠redhat.com

Doron Fediuck has submitted this change and it was merged. Change subject: core: scheduling: fix ClassCastException ...................................................................... core: scheduling: fix ClassCastException Since the external scheduler balance command returns Host id as LinkedList and the migrate command expects ArrayList the exception occurs. Change-Id: I011d1d01b6d2de1718a73f687ed6d181ccadb135 Bug-Url: https://bugzilla.redhat.com/1002093 Signed-off-by: Gilad Chaplik <gchaplik(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/InitBackendServicesOnStartupBean.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/MigrationHandler.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/SchedulingManager.java 3 files changed, 6 insertions(+), 5 deletions(-) Approvals: Doron Fediuck: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/18802 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I011d1d01b6d2de1718a73f687ed6d181ccadb135 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Laszlo Hornyak <lhornyak(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

11 years, 3 months

1
0
0 / 0

Change in ovirt-engine[master]: core: lock VM for whole live disk migration process

by ofrenkel＠redhat.com

Omer Frenkel has submitted this change and it was merged. Change subject: core: lock VM for whole live disk migration process ...................................................................... core: lock VM for whole live disk migration process In order to prevent a race between VM migration and storage commands, the storage commands need to lock the VM for their entire execution. This patch takes shared lock for the VM for the entire live storage migration process. Change-Id: I432e9230da697a1a67c5d40e926e16215c2583ef Bug-Url: https://bugzilla.redhat.com/952147 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CommandBase.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/lsm/LiveMigrateDiskCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/lsm/LiveMigrateVmDisksCommand.java 3 files changed, 33 insertions(+), 5 deletions(-) Approvals: Omer Frenkel: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/18545 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I432e9230da697a1a67c5d40e926e16215c2583ef Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

11 years, 3 months

1
0
0 / 0

Change in ovirt-engine[master]: db: fix upgrade for app mode to roles

by emesika＠redhat.com

Eli Mesika has submitted this change and it was merged. Change subject: db: fix upgrade for app mode to roles ...................................................................... db: fix upgrade for app mode to roles Role UserTemplateBasedVm was missing from the update list always triggering the following error: ERROR: column "app_mode" contains null values Change-Id: Ic5ebbba79deb6de091d27621196f361fd4fd3caf Signed-off-by: Federico Simoncelli <fsimonce(a)redhat.com> --- M packaging/dbscripts/upgrade/03_03_0810_add_app_mode_to_roles.sql 1 file changed, 2 insertions(+), 1 deletion(-) Approvals: Eli Mesika: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/18777 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic5ebbba79deb6de091d27621196f361fd4fd3caf 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: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

11 years, 3 months

1
0
0 / 0

Change in ovirt-engine[master]: webadmin: Stopped progress bar after initializing network list

by gchaplik＠redhat.com

Gilad Chaplik has submitted this change and it was merged. Change subject: webadmin: Stopped progress bar after initializing network list ...................................................................... webadmin: Stopped progress bar after initializing network list The code didn't stop the progress bar, so when a DC was selected in the system tree (rather than a network) the progress bar would continue indefinitely when the dialog was opened. Change-Id: I469e4d95e479c99f5315fbc89f369e254b16cfd9 Bug-Url: https://bugzilla.redhat.com/1001000 Signed-off-by: Lior Vernia <lvernia(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/profiles/VnicProfileListModel.java 1 file changed, 3 insertions(+), 2 deletions(-) Approvals: Gilad Chaplik: Looks good to me, approved Lior Vernia: Verified -- To view, visit http://gerrit.ovirt.org/18782 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I469e4d95e479c99f5315fbc89f369e254b16cfd9 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Lior Vernia <lvernia(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Lior Vernia <lvernia(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

11 years, 3 months

1
0
0 / 0

Change in ovirt-live[master]: ovirt-live: migrate ovirt live plugin to otopi

by obasan＠redhat.com

Ohad Basan has submitted this change and it was merged. Change subject: ovirt-live: migrate ovirt live plugin to otopi ...................................................................... ovirt-live: migrate ovirt live plugin to otopi Change-Id: I703f64dc1183a6fe176d9d0352f93de381d906bb Signed-off-by: Ohad Basan <obasan(a)redhat.com> --- M fedora/kickstart/ovirt.ks D fedora/oVirtLiveFiles/ovirt_live_101.py R fedora/oVirtLiveFiles/root/etc/gdm/custom.conf R fedora/oVirtLiveFiles/root/etc/ovirt-host-deploy.conf.d/50-vdsm-conf-fake-nic.conf R fedora/oVirtLiveFiles/root/etc/sysconfig/network-scripts/ifcfg-dummy0 R fedora/oVirtLiveFiles/root/etc/sysconfig/network-scripts/ifcfg-eth0 R fedora/oVirtLiveFiles/root/etc/sysconfig/network-scripts/ifcfg-ovirtmgmt R fedora/oVirtLiveFiles/root/etc/sysconfig/network-scripts/ifcfg-wlan0 R fedora/oVirtLiveFiles/root/etc/yum.repos.d/ovirt.repo R fedora/oVirtLiveFiles/root/usr/share/applications/engine-setup.desktop R fedora/oVirtLiveFiles/root/usr/share/applications/ovirt.desktop R fedora/oVirtLiveFiles/root/usr/share/glib-2.0/schemas/org.gnome.desktop.lockdown.gschema.override R fedora/oVirtLiveFiles/root/usr/share/glib-2.0/schemas/org.gnome.desktop.screensaver.gschema.override R fedora/oVirtLiveFiles/root/usr/share/glib-2.0/schemas/org.gnome.settings-daemon.plugins.updates.gschema.override R fedora/oVirtLiveFiles/root/usr/share/glib-2.0/schemas/org.gnome.shell.gschema.override A fedora/oVirtLiveFiles/root/usr/share/ovirt-engine/setup/plugins/ovirt-engine-setup/olive/__init__.py A fedora/oVirtLiveFiles/root/usr/share/ovirt-engine/setup/plugins/ovirt-engine-setup/olive/core.py A fedora/oVirtLiveFiles/root/usr/share/ovirt-engine/setup/plugins/ovirt-engine-setup/olive/oliveconst.py 18 files changed, 353 insertions(+), 330 deletions(-) Approvals: Ohad Basan: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17518 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I703f64dc1183a6fe176d9d0352f93de381d906bb Gerrit-PatchSet: 11 Gerrit-Project: ovirt-live Gerrit-Branch: master Gerrit-Owner: Ohad Basan <obasan(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(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: Ohad Basan <obasan(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>

11 years, 3 months

1
0
0 / 0

Change in ovirt-engine[master]: packaging: setup: Fix issue while installing Gluster only mode

by oschreib＠redhat.com

Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: setup: Fix issue while installing Gluster only mode ...................................................................... packaging: setup: Fix issue while installing Gluster only mode When application mode is selected as Gluster, engine-setup was failing with error "Failed to execute stage 'Misc configuration': 'NFS'" Change-Id: I25656cff4f25a47199905dd4f4b2e683f1066c00 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1002854 Signed-off-by: Aravinda VK <avishwan(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-setup/config/domain_type.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/18712 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I25656cff4f25a47199905dd4f4b2e683f1066c00 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master 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: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

11 years, 3 months

1
0
0 / 0

Change in ovirt-scheduler-proxy[master]: throw exception if plugin not found

by lhornyak＠redhat.com

Laszlo Hornyak has submitted this change and it was merged. Change subject: throw exception if plugin not found ...................................................................... throw exception if plugin not found if a plugin is not found, throw an exception rather than ignoring it Change-Id: Iffa60c3a1e900465f41dc003263cd1e4d235ee9e Bug-Url: https://bugzilla.redhat.com/1002443 Signed-off-by: Laszlo Hornyak <lhornyak(a)redhat.com> --- M src/ovirtscheduler/request_handler.py M tests/java/src/test/java/org/ovirt/schedulerproxy/SchedulerProxyTest.java 2 files changed, 7 insertions(+), 1 deletion(-) Approvals: Laszlo Hornyak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/18728 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iffa60c3a1e900465f41dc003263cd1e4d235ee9e Gerrit-PatchSet: 2 Gerrit-Project: ovirt-scheduler-proxy Gerrit-Branch: master Gerrit-Owner: Laszlo Hornyak <lhornyak(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Laszlo Hornyak <lhornyak(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

11 years, 3 months

1
0
0 / 0

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

Engine-commits September 2013