Change in ovirt-engine[master]: userportal: disable host pinning in user portal (#849042)
Tomas Jelinek has submitted this change and it was merged.
Change subject: userportal: disable host pinning in user portal (#849042)
......................................................................
userportal: disable host pinning in user portal (#849042)
https://bugzilla.redhat.com/849042
The behavior:
- new/edit VM and edit Template does not have the
"Host" side tab visible
- when the host pinning is set in web admin and the
VM or Template is edited in user portal, this values
will be send as-is to the server
- if the host pinning will be set in the template,
the new/edit VM dialog will ignore this settings
This patch also contains the following fixes:
1: changes the calling of admin query
AsyncDataProvider.GetHostListByCluster by user
query GetHostsByClusterId for edit template.
2: CPU pinning field was ignored in the edit VM dialog
in userportal. It can not be edited directly by the
user but it can not be ignored to be possible to
send the same value to the server as it was set in
web admin
Change-Id: Ie0737807a46845e8ad8ee27c91c6a782df7b3fd3
Signed-off-by: Tomas Jelinek <tjelinek(a)redhat.com>
---
M
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/templates/TemplateListModel.java
M
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/userportal/UserPortalListModel.java
M
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/userportal/UserPortalTemplateListModel.java
M
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/UserPortalExistingVmModelBehavior.java
M
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/UserPortalNewVmModelBehavior.java
A
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/UserPortalTemplateVmModelBehavior.java
M
frontend/webadmin/modules/userportal-gwtp/src/main/java/org/ovirt/engine/ui/userportal/section/main/view/popup/template/TemplateNewPopupView.java
7 files changed, 102 insertions(+), 2 deletions(-)
Approvals:
Tomas Jelinek: Verified; Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/7292
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ie0737807a46845e8ad8ee27c91c6a782df7b3fd3
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Tomas Jelinek <tjelinek(a)redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server