Bug 1378935 - [tracker] oVirt UI / Internet Explorer performance improvements
Summary: [tracker] oVirt UI / Internet Explorer performance improvements
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: ovirt-engine
Classification: oVirt
Component: Frontend.WebAdmin
Version: 4.1.0
Hardware: Unspecified
OS: Unspecified
unspecified
medium
Target Milestone: ovirt-4.1.0-beta
: 4.1.0.2
Assignee: Vojtech Szocs
QA Contact: Pavel Novotny
URL:
Whiteboard:
Depends On:
Blocks: 1404610
TreeView+ depends on / blocked
 
Reported: 2016-09-23 14:34 UTC by Vojtech Szocs
Modified: 2020-02-16 07:12 UTC (History)
7 users (show)

Fixed In Version:
Clone Of:
: 1404610 (view as bug list)
Environment:
Last Closed: 2017-03-16 14:50:32 UTC
oVirt Team: UX
Embargoed:
rule-engine: ovirt-4.1+
rule-engine: planning_ack+
oourfali: devel_ack+
pstehlik: testing_ack+

Attachments (Terms of Use)
IE11 - webadmin memory usage chart (24.73 KB, image/png)
2017-03-02 18:14 UTC, Pavel Novotny 		no flags Details
View All


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 64282 0 master MERGED webadmin: Remove IE9 hack in UI suggest box widget 2020-12-12 14:40:34 UTC
oVirt gerrit 64294 0 master MERGED webadmin: Improve IE image size quirk workaround performance 2020-12-12 14:41:03 UTC
oVirt gerrit 64302 0 master MERGED webadmin,userportal: Remove IE8 related code 2020-12-12 14:40:34 UTC
oVirt gerrit 64359 0 master MERGED webadmin: Remove IE-specific code in ApplicationFocusManager 2020-12-12 14:40:32 UTC
oVirt gerrit 68483 0 ovirt-engine-4.0 MERGED webadmin: Remove IE9 hack in UI suggest box widget 2020-12-12 14:40:34 UTC
oVirt gerrit 68484 0 ovirt-engine-4.0 MERGED webadmin: Improve IE image size quirk workaround performance 2020-12-12 14:40:35 UTC
oVirt gerrit 68485 0 ovirt-engine-4.0 MERGED webadmin,userportal: Remove IE8 related code 2020-12-12 14:40:35 UTC
oVirt gerrit 68486 0 ovirt-engine-4.0 MERGED webadmin: Remove IE-specific code in ApplicationFocusManager 2020-12-12 14:40:39 UTC

Description Vojtech Szocs 2016-09-23 14:34:27 UTC 
This is a tracker bug for all oVirt UI performance improvements.

Fixes should be targeted to 4.0, if possible.
Comment 1 Red Hat Bugzilla Rules Engine 2016-10-12 12:17:53 UTC 
Target release should be placed once a package build is known to fix a issue. Since this bug is not modified, the target version has been reset. Please use target milestone to plan a fix for a oVirt release.
Comment 2 Vojtech Szocs 2016-12-05 18:18:27 UTC 
> Status: POST → MODIFIED

This bug was originally meant to collect all kinds of post-memory-leak-fix improvements.

Currently, it has 4 IE-specific patches associated. None of them is merged into master, so moving back to POST.

If we wish to proceed with those 4 IE-specific patches (only), maybe we should rename this bug.
Comment 3 Vojtech Szocs 2016-12-06 15:29:31 UTC 
All IE-specific patches verified on local Engine (master) build using IE11 / Windows 8.1 (thanks to Pavel Novotny).
Comment 4 Vojtech Szocs 2016-12-13 14:23:52 UTC 
(In reply to vszocs from comment #2)
> Currently, it has 4 IE-specific patches associated. None of them is merged
> into master, so moving back to POST.
> 
> If we wish to proceed with those 4 IE-specific patches (only), maybe we
> should rename this bug.

Relevant patches are merged in master, updated BZ title.
Comment 6 Pavel Novotny 2017-03-02 18:13:43 UTC 
Verified in 
rhevm-4.1.1.2-0.1.el7.noarch

Browser: Internet Explorer 11 @ Windows 8.1 Pro (4 vCPU, 8 GB RAM)
RHEVM environment: 1 DC, 2 clusters, 1 host, 2 NFS storages (1 data, 1 export), 41 VMs (blank, w/o OS installed, 4 of them Up).

I ran a test in a loop, where it went through the main tabs and then opened & closed the New VM dialog.
After each pass I wrote down the memory usage from Developer Tools / Memory / Profiling session - Process memory usage (private bytes).
I compared the data against rhevm-4.0.7.4-0.1.el7ev, which contains the same IE-related fixes,
as well as against the "unfixed" released rhevm-4.0.6.3-0.1.el7ev (for differences between 4.0.6 and 4.0.7 see bug 1404610 comment 9). 
See the chart attached.

With 4.0.7, the maximum I managed to run was 9 passes before reaching a RAM usage ~1.6 GB.
This seems to be the browser limit, where it becomes unresponsive beyond usability and the app. eventually freezes or crashes.
The 9 passes took 19 minutes.

With 4.1.1, I managed to run 14 passes before reaching the 1.6 GB RAM usage.
To compare with 4.0.7, 9 passes here took 12 minutes. All 14 passes took 25 minutes.
Comment 7 Pavel Novotny 2017-03-02 18:14:29 UTC 
Created attachment 1259245 [details]
IE11 - webadmin memory usage chart
Note You need to log in before you can comment on or make changes to this bug.