Change in ovirt-engine[ovirt-engine-3.6]: webadmin, userportal: Cache of VmConsoles removed

tnisan at redhat.com tnisan at redhat.com
Tue Aug 11 12:17:22 UTC 2015


Tal Nisan has submitted this change and it was merged.

Change subject: webadmin, userportal: Cache of VmConsoles removed
......................................................................


webadmin, userportal: Cache of VmConsoles removed

VmConsoles objects are not cached, rather created on demand thus they
can be well configured all the time.

The bug was caused by resolving wrong console type during first list
show (when the vms were down) and keeping that settings even after vms
were capable of non-rdp connection.

Signed-off-by: Jakub Niedermertl <jniederm at redhat.com>
Change-Id: I92fd4807a9d2daea67b8ba547c75717a1764268d
Bug-Url: https://bugzilla.redhat.com/1236674
---
D frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/ConsoleModelsCache.java
A frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/ConsolesFactory.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/userportal/AbstractUserPortalListModel.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/userportal/UserPortalBasicListModel.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/userportal/UserPortalItemModel.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/vms/VmListModel.java
7 files changed, 102 insertions(+), 165 deletions(-)

Approvals:
  Tomas Jelinek: Looks good to me, approved
  Tal Nisan: Passed CI tests
  Jakub Niedermertl: Verified



-- 
To view, visit https://gerrit.ovirt.org/44440
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I92fd4807a9d2daea67b8ba547c75717a1764268d
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.6
Gerrit-Owner: Jakub Niedermertl <jniederm at redhat.com>
Gerrit-Reviewer: Jakub Niedermertl <jniederm at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjelinek at redhat.com>
Gerrit-Reviewer: automation at ovirt.org



More information about the Engine-commits mailing list