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, 2 months
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, 2 months
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, 2 months
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, 2 months
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, 2 months
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, 2 months
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, 2 months
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, 2 months
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, 2 months
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, 2 months