Change in ovirt-engine[master]: webadmin: When live migrating a disk filter domains only if ...
by tnisan＠redhat.com
Tal Nisan has submitted this change and it was merged.
Change subject: webadmin: When live migrating a disk filter domains only if needed
......................................................................
webadmin: When live migrating a disk filter domains only if needed
Up until now when moving a disk connected to a running VM (LSM) the domains
were filtered so that live move would have been possible only within the
same storage types (block/file).
This patch cause the domains to be filtered only for data centers with
versions up to 3.6 since we from 3.6 onwards we support LSM between
different storage types
Change-Id: I5eaaf8209a8e7187987697baae1086a9d4311ce3
Related-To: https://bugzilla.redhat.com/1058757
Signed-off-by: Tal Nisan <tnisan(a)redhat.com>
---
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/ConfigurationValues.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/dataprovider/AsyncDataProvider.java
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/MoveDiskModel.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmDiskListModel.java
M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/UIConstants.java
6 files changed, 32 insertions(+), 5 deletions(-)
Approvals:
Nir Soffer: Verified
Daniel Erez: Looks good to me, approved
Jenkins CI: Looks good to me, but someone else must approve
--
To view, visit https://gerrit.ovirt.org/40550
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I5eaaf8209a8e7187987697baae1086a9d4311ce3
Gerrit-PatchSet: 9
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: Jenkins CI
Gerrit-Reviewer: Nir Soffer <nsoffer(a)redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
8 years, 11 months
Change in ovirt-engine[master]: core: Remove LSM in different storage sub types limitation
by tnisan＠redhat.com
Tal Nisan has submitted this change and it was merged.
Change subject: core: Remove LSM in different storage sub types limitation
......................................................................
core: Remove LSM in different storage sub types limitation
Change-Id: Ic5cbf6b7e3fd14af3dc22fca2469ba43ec6a4353
Related-To: https://bugzilla.redhat.com/1058757
Signed-off-by: Tal Nisan <tnisan(a)redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/lsm/LiveMigrateVmDisksCommand.java
M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/lsm/LiveMigrateVmDisksCommandTest.java
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/FeatureSupported.java
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.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
M packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql
9 files changed, 46 insertions(+), 8 deletions(-)
Approvals:
Nir Soffer: Verified
Allon Mureinik: Looks good to me, approved
--
To view, visit https://gerrit.ovirt.org/37019
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ic5cbf6b7e3fd14af3dc22fca2469ba43ec6a4353
Gerrit-PatchSet: 12
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <nsoffer(a)redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
8 years, 11 months
Change in ovirt-engine[master]: engine: Add more search filters for VM
by ofrenkel＠redhat.com
Omer Frenkel has submitted this change and it was merged.
Change subject: engine: Add more search filters for VM
......................................................................
engine: Add more search filters for VM
The patch enables to search VMs based on:
1. Cluster compatibility level
2. Custom VM emulated-machine \ CPU-model
Change-Id: I607cc9cc3dd6ca80453aa44e20a213504a4df855
Bug-Url: https://bugzilla.redhat.com/1201511
Signed-off-by: Eldan Shachar <eshachar(a)redhat.com>
---
M backend/manager/modules/searchbackend/src/main/java/org/ovirt/engine/core/searchbackend/VmConditionFieldAutoCompleter.java
M backend/manager/modules/searchbackend/src/test/java/org/ovirt/engine/core/searchbackend/SyntaxCheckerTest.java
2 files changed, 13 insertions(+), 1 deletion(-)
Approvals:
Eldan Shachar: Verified
Jenkins CI: Looks good to me, but someone else must approve
Omer Frenkel: Looks good to me, approved
--
To view, visit https://gerrit.ovirt.org/39434
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I607cc9cc3dd6ca80453aa44e20a213504a4df855
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Eldan Shachar <eshachar(a)redhat.com>
Gerrit-Reviewer: Eldan Shachar <eshachar(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
8 years, 11 months
Change in ovirt-engine[ovirt-engine-3.5.3]: packaging: setup: pki: Enroll certs on upgrade if not exist
by piotr.kliczewski＠gmail.com
Piotr Kliczewski has submitted this change and it was merged.
Change subject: packaging: setup: pki: Enroll certs on upgrade if not exist
......................................................................
packaging: setup: pki: Enroll certs on upgrade if not exist
reports key/cert were generated by 3.5 clean setup, but not by upgrade
(and did not exist in <=3.4). This didn't cause a failure because they
are not used by default. Generate on upgrade.
Bug-Url: https://bugzilla.redhat.com/1224656
Change-Id: I9ed93d47a55f2971337bac277ce69c672901bb07
Signed-off-by: Yedidyah Bar David <didi(a)redhat.com>
(cherry picked from commit 0e87ed2877f56d12da4dea4387ad67f56bbc45e4)
---
M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/pki/ca.py
1 file changed, 8 insertions(+), 1 deletion(-)
Approvals:
Sandro Bonazzola: Looks good to me, approved
Yedidyah Bar David: Verified
--
To view, visit https://gerrit.ovirt.org/41375
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I9ed93d47a55f2971337bac277ce69c672901bb07
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.5.3
Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
8 years, 11 months
Change in ovirt-engine[ovirt-engine-3.5]: packaging: setup: pki: Enroll certs on upgrade if not exist
by piotr.kliczewski＠gmail.com
Piotr Kliczewski has submitted this change and it was merged.
Change subject: packaging: setup: pki: Enroll certs on upgrade if not exist
......................................................................
packaging: setup: pki: Enroll certs on upgrade if not exist
reports key/cert were generated by 3.5 clean setup, but not by upgrade
(and did not exist in <=3.4). This didn't cause a failure because they
are not used by default. Generate on upgrade.
Bug-Url: https://bugzilla.redhat.com/1224656
Change-Id: I9ed93d47a55f2971337bac277ce69c672901bb07
Signed-off-by: Yedidyah Bar David <didi(a)redhat.com>
(cherry picked from commit de16430f6609586a5530f3db579c36be1b12815e)
---
M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/pki/ca.py
1 file changed, 8 insertions(+), 1 deletion(-)
Approvals:
Sandro Bonazzola: Looks good to me, approved
Yedidyah Bar David: Verified
--
To view, visit https://gerrit.ovirt.org/41282
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I9ed93d47a55f2971337bac277ce69c672901bb07
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.5
Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
8 years, 11 months
Change in ovirt-guest-agent[master]: app-list: Tracking also cloud-init now
by vfeenstr＠redhat.com
Vinzenz Feenstra has submitted this change and it was merged.
Change subject: app-list: Tracking also cloud-init now
......................................................................
app-list: Tracking also cloud-init now
Change-Id: I15a1d4fedaee29c3da2f2e4602e081b53ed2270a
Bug-Url: https://bugzilla.redhat.com/1206102
Signed-off-by: Vinzenz Feenstra <vfeenstr(a)redhat.com>
---
M configurations/default.conf
M configurations/ovirt-guest-agent.conf
2 files changed, 2 insertions(+), 2 deletions(-)
Approvals:
Jenkins CI: Looks good to me, but someone else must approve
Vinzenz Feenstra: Verified; Looks good to me, approved
Michal Skrivanek: Looks good to me, but someone else must approve
--
To view, visit https://gerrit.ovirt.org/41246
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I15a1d4fedaee29c3da2f2e4602e081b53ed2270a
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-guest-agent
Gerrit-Branch: master
Gerrit-Owner: Vinzenz Feenstra <vfeenstr(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <mskrivan(a)redhat.com>
Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
8 years, 11 months
Change in ovirt-engine[master]: packaging: setup: pki: Enroll certs on upgrade if not exist
by sbonazzo＠redhat.com
Sandro Bonazzola has submitted this change and it was merged.
Change subject: packaging: setup: pki: Enroll certs on upgrade if not exist
......................................................................
packaging: setup: pki: Enroll certs on upgrade if not exist
reports key/cert were generated by 3.5 clean setup, but not by upgrade
(and did not exist in <=3.4). This didn't cause a failure because they
are not used by default. Generate on upgrade.
Bug-Url: https://bugzilla.redhat.com/1223826
Change-Id: I9ed93d47a55f2971337bac277ce69c672901bb07
Signed-off-by: Yedidyah Bar David <didi(a)redhat.com>
---
M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/pki/ca.py
1 file changed, 8 insertions(+), 1 deletion(-)
Approvals:
Alon Bar-Lev: Looks good to me, but someone else must approve
Sandro Bonazzola: Looks good to me, approved
Simone Tiraboschi: Looks good to me, but someone else must approve
Jenkins CI: Looks good to me, but someone else must approve
Yedidyah Bar David: Verified
--
To view, visit https://gerrit.ovirt.org/41264
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I9ed93d47a55f2971337bac277ce69c672901bb07
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
8 years, 11 months
Change in ovirt-guest-agent[master]: linux: Logout active user support
by vfeenstr＠redhat.com
Vinzenz Feenstra has submitted this change and it was merged.
Change subject: linux: Logout active user support
......................................................................
linux: Logout active user support
Change-Id: I1173dc8cb6c2bd68b5e2a2ba4ad69284b2085171
Bug-Url: https://bugzilla.redhat.com/1171638
Signed-off-by: Vinzenz Feenstra <vfeenstr(a)redhat.com>
---
M ovirt-guest-agent.rhel6.spec
M ovirt-guest-agent.spec
M ovirt-guest-agent/GuestAgentLinux2.py
A ovirt-guest-agent/LogoutActiveUser.py
M ovirt-guest-agent/Makefile.am
M ovirt-guest-agent/consoleapps/Makefile.am
A ovirt-guest-agent/consoleapps/ovirt-logout
M ovirt-guest-agent/pam/Makefile.am
A ovirt-guest-agent/pam/ovirt-logout
M scripts/wrappers/Makefile.am
A scripts/wrappers/ovirt-logout-wrapper.sh
11 files changed, 94 insertions(+), 1 deletion(-)
Approvals:
Vinzenz Feenstra: Verified; Looks good to me, approved
Michal Skrivanek: Looks good to me, but someone else must approve
--
To view, visit https://gerrit.ovirt.org/35416
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I1173dc8cb6c2bd68b5e2a2ba4ad69284b2085171
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-guest-agent
Gerrit-Branch: master
Gerrit-Owner: Vinzenz Feenstra <vfeenstr(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <mskrivan(a)redhat.com>
Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
8 years, 11 months
Change in ovirt-engine[ovirt-engine-3.5]: webadmin: fix refresh status.
by tnisan＠redhat.com
Tal Nisan has submitted this change and it was merged.
Change subject: webadmin: fix refresh status.
......................................................................
webadmin: fix refresh status.
- The refresh flag was improperly set for some operation on the snapshot
VM subtab.
Change-Id: I73e0238229e458491e6e8e9995b54d39419357f5
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1206332
Signed-off-by: Alexander Wels <awels(a)redhat.com>
---
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/dataprovider/AsyncDataProvider.java
1 file changed, 2 insertions(+), 2 deletions(-)
Approvals:
Alexander Wels: Verified
Greg Sheremeta: Looks good to me, approved
--
To view, visit https://gerrit.ovirt.org/41360
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I73e0238229e458491e6e8e9995b54d39419357f5
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.5
Gerrit-Owner: Alexander Wels <awels(a)redhat.com>
Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com>
Gerrit-Reviewer: Einav Cohen <ecohen(a)redhat.com>
Gerrit-Reviewer: Greg Sheremeta <gshereme(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
8 years, 11 months
Change in ovirt-engine[ovirt-engine-3.5]: userportal, webadmin: fix broken helptag implementation
by tnisan＠redhat.com
Tal Nisan has submitted this change and it was merged.
Change subject: userportal, webadmin: fix broken helptag implementation
......................................................................
userportal, webadmin: fix broken helptag implementation
Helptags were reusing some of the hashname infrastructure. This
was 1. messy, 2. caused a bug where the help icon would not render
unless the helptag == hashname. Cleaned this up and fixed the bug.
Change-Id: Id244c5d068c6146da92287536bf919da1d5aee3c
Bug-Url: https://bugzilla.redhat.com/1176581
Bug-Url: https://bugzilla.redhat.com/1176583
Bug-Url: https://bugzilla.redhat.com/1223529
Signed-off-by: Greg Sheremeta <gshereme(a)redhat.com>
---
M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/presenter/AbstractModelBoundPopupPresenterWidget.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/Model.java
2 files changed, 15 insertions(+), 12 deletions(-)
Approvals:
Alexander Wels: Looks good to me, approved
Jenkins CI: Looks good to me, but someone else must approve
Greg Sheremeta: Verified
--
To view, visit https://gerrit.ovirt.org/41348
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Id244c5d068c6146da92287536bf919da1d5aee3c
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.5
Gerrit-Owner: Greg Sheremeta <gshereme(a)redhat.com>
Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com>
Gerrit-Reviewer: Greg Sheremeta <gshereme(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
8 years, 11 months