Bug 1378935

Summary: [tracker] oVirt UI / Internet Explorer performance improvements
Product: [oVirt] ovirt-engine Reporter: Vojtech Szocs <vszocs>
Component: Frontend.WebAdminAssignee: Vojtech Szocs <vszocs>
Status: CLOSED CURRENTRELEASE QA Contact: Pavel Novotny <pnovotny>
Severity: medium Docs Contact:
Priority: unspecified    
Version: 4.1.0CC: awels, bugs, eberman, gklein, oourfali, pnovotny, s.kieske
Target Milestone: ovirt-4.1.0-betaKeywords: Performance, Tracking, ZStream
Target Release: 4.1.0.2Flags: rule-engine: ovirt-4.1+
rule-engine: planning_ack+
oourfali: devel_ack+
pstehlik: testing_ack+
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
: 1404610 (view as bug list) Environment:
Last Closed: 2017-03-16 14:50:32 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: UX RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1404610    
Attachments:
Description Flags
IE11 - webadmin memory usage chart none

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