Bug 1412172

Summary: UI lag due to more than 3650 messages in notification
Product: Red Hat CloudForms Management Engine Reporter: vaclav.miller
Component: UI - OPSAssignee: Martin Hradil <mhradil>
Status: CLOSED CURRENTRELEASE QA Contact: Shveta <sshveta>
Severity: high Docs Contact:
Priority: high    
Version: 5.7.0CC: dclarizi, hkataria, jhardy, mhradil, mpovolny, obarenbo, simaishi
Target Milestone: GAKeywords: TestOnly
Target Release: 5.9.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: notification
Fixed In Version: 5.9.0.1 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
: 1448045 1448046 (view as bug list) Environment:
Last Closed: 2018-03-06 15:20:51 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: Bug
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: CFME Core Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1448045, 1448046    

Description vaclav.miller 2017-01-11 12:51:51 UTC
Description of problem:

It was not possible to work with UI due to lags (complete web browser including other, not CFME related tabs were frozen)

There was identified more 3650 unread messages in notification.
In production.log were messages:
[----] I, [2017-01-11T13:30:37.001241 #17205:ea0a88]  INFO -- : Started GET "/api/notifications?expand=resources&attributes=details"
 for 193.182.20.95 at 2017-01-11 13:30:37 +0100
[----] I, [2017-01-11T13:30:37.031701 #17205:ea0a88]  INFO -- : Processing by Api::NotificationsController#show as JSON
[----] I, [2017-01-11T13:30:37.031938 #17205:ea0a88]  INFO -- :   Parameters: {"expand"=>"resources", "attributes"=>"details"}
[----] I, [2017-01-11T13:30:58.960555 #17205:ea0a88]  INFO -- : Completed 200 OK in 21928ms (Views: 0.4ms | ActiveRecord: 0.0ms)

It was not sufficient just to "Mark us read".
To resolve issue, all messages from notification had to be deleted.

Martin Hradil helped to troubleshoot this issue.

Version-Release number of selected component (if applicable):


How reproducible:
Generate meny messages for notification.
Browse through UI

Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:


Additional info:

Comment 7 CFME Bot 2017-04-22 05:02:48 UTC
New commit detected on ManageIQ/manageiq-ui-classic/master:
https://github.com/ManageIQ/manageiq-ui-classic/commit/e96b024a9e3d6b1fb75107d5fe364f982a311b9c

commit e96b024a9e3d6b1fb75107d5fe364f982a311b9c
Author:     Martin Hradil <mhradil>
AuthorDate: Wed Mar 8 17:52:48 2017 +0000
Commit:     Martin Hradil <mhradil>
CommitDate: Wed Apr 19 13:12:01 2017 +0000

    notification-drawer - only render group items when the group is open
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1412172

 app/views/static/notification_drawer/notification-drawer.html.haml | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

Comment 10 vaclav.miller 2017-07-13 14:21:12 UTC
Hello, there is update in 5.7.3 where only first 100 messages is generated that really improved performance.

But originally there were available "buttons" to Mark all messages as read / Delete all of them. 
These "buttons" are not available. you have to "delete" messages one by one clicking on "x" icon at each message. Then at some moment (or never) appears buttons with options to Mark all as read / Delete all.

It would be really useful if these "buttons" will be displayed al the time. Or at least it will be possible to scroll through list of notification to get to "buttons".

Thanks,
Vaclav

Comment 11 Martin Hradil 2017-07-13 14:30:38 UTC
Vaclav.. aah, thanks for noticing, the buttons are actually always there, but notification area doesn't resize properly so they can move off screen.

If you want to track that, it's https://bugzilla.redhat.com/show_bug.cgi?id=1445689 .

Comment 12 Shveta 2017-10-31 19:19:15 UTC
Fixed in 5.9.0.4.20171024163837_ef71ea6