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@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@redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik@redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali@redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek@redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server