9:24 a.m.
Roy Golan has submitted this change and it was merged.
Change subject: webadmin: Fix migration policy update for running VM
......................................................................
webadmin: Fix migration policy update for running VM
Problem description:
--------------------
When a VM is running and its migration policy is pinned.
Change the migration policy to migrateable allows the running VM
to migrate to another host.
This behavior is a security threat, passing information from the pinned
host to the migration target. And migration could fail due to
incompatible target host.
Desired behavior:
-----------------
When a VM is running and changing migration policy from pinned to
Migrateable (manual or automatic).
- Provide warning and UX indicator.
- If user confirms proceed with normal update.
Implementation:
--------------
1. Replace @EditableOnVmStatusField with @EditableField to
fields {dedicatedVmForVdsList, migrationSupport} (VmBase.java). Disable
(vm restart requirement), when updating dedicated hosts field or
updating migrationSupport field.
2. Redesign VmNextRunConfiguration UI to display warnning if running VM
is unpinned. Modify VmListModel.java to append warning into
VmNextRunConfigurationModel.java . Also Modify VmNextRunConfiguration
layout and widget to display selective information (coresponding to
updated fields).
Change-Id: Idcc9c120606ebe5bc55803213065124f44972e0b
Bug-Url: https://bugzilla.redhat.com/1058384
Signed-off-by: Dudi Maroshi <dudi(a)redhat.com>
---
M
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VmBase.java
M
frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/CommonApplicationConstants.java
M
frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/CommonApplicationMessages.java
M
frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/vm/VmNextRunConfigurationWidget.java
M
frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/vm/VmNextRunConfigurationWidget.ui.xml
M
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmListModel.java
M
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmNextRunConfigurationModel.java
M
frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/UIConstants.java
M
frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/vm/VmNextRunConfigurationPopupView.java
9 files changed, 165 insertions(+), 45 deletions(-)
Approvals:
Dudi Maroshi: Verified
Jenkins CI: Passed CI tests
Roy Golan: Looks good to me, approved
--
To view, visit https://gerrit.ovirt.org/47673
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Idcc9c120606ebe5bc55803213065124f44972e0b
Gerrit-PatchSet: 9
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Dudi Maroshi <dudi(a)redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com>
Gerrit-Reviewer: Dudi Maroshi <dudi(a)redhat.com>
Gerrit-Reviewer: Einav Cohen <ecohen(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org