Bug 1320559 - [Webadmin] uncaught exception notification repeats endlessly
Summary: [Webadmin] uncaught exception notification repeats endlessly
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: ovirt-engine
Classification: oVirt
Component: Frontend.WebAdmin
Version: 4.0.0
Hardware: Unspecified
OS: Unspecified
unspecified
low
Target Milestone: ovirt-4.0.2
: 4.0.2
Assignee: Alexander Wels
QA Contact: Pavel Novotny
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-03-23 13:48 UTC by Greg Sheremeta
Modified: 2016-08-12 14:32 UTC (History)
5 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2016-08-12 14:32:05 UTC
oVirt Team: UX
Embargoed:
rule-engine: ovirt-4.0.z+
rule-engine: planning_ack+
rule-engine: devel_ack+
pnovotny: testing_ack+


Attachments (Terms of Use)
uncaught exception overload (248.97 KB, image/png)
2016-03-23 13:48 UTC, Greg Sheremeta
no flags Details
single exception alert with a counter (24.06 KB, image/png)
2016-07-28 15:21 UTC, Pavel Novotny
no flags Details


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 60990 0 master MERGED userportal,webadmin: show single exception alert. 2016-07-19 18:53:01 UTC
oVirt gerrit 61054 0 ovirt-engine-4.0 MERGED userportal,webadmin: show single exception alert. 2016-07-20 06:51:46 UTC

Description Greg Sheremeta 2016-03-23 13:48:18 UTC
Created attachment 1139594 [details]
uncaught exception overload

Description of problem:
New uncaught exception notification repeats endlessly sometimes.

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

How reproducible:
?

Steps to Reproduce:
I'm not sure how to reproduce. This happened to me while applying an unrelated patch to test it.

Actual results:
see screenshot

Expected results:
The notification should be a single box. If they stack endlessly on top of each other, the app is obviously unusable.

Additional info:

Comment 1 Vojtech Szocs 2016-04-26 16:58:07 UTC
Each red box represents a separate occurence of (uncaught) UI error.

Do we want to:

1, remember content of currently showing boxes and put a number next to existing one to indicate the number of occurences so far?

2, allow only one red box shown at a time? (this hides errors that occur in the meantime, until the current box is closed)

Comment 2 Greg Sheremeta 2016-04-27 00:47:15 UTC
I like option 1. That's what browser dev tools do.

Comment 3 Vojtech Szocs 2016-04-27 12:31:07 UTC
(In reply to Greg Sheremeta from comment #2)
> I like option 1. That's what browser dev tools do.

Me too, the implementation shouldn't be hard anyway.

Comment 4 Sandro Bonazzola 2016-05-02 10:10:48 UTC
Moving from 4.0 alpha to 4.0 beta since 4.0 alpha has been already released and bug is not ON_QA.

Comment 5 Yaniv Lavi 2016-05-23 13:26:42 UTC
oVirt 4.0 beta has been released, moving to RC milestone.

Comment 6 Yaniv Lavi 2016-05-23 13:26:55 UTC
oVirt 4.0 beta has been released, moving to RC milestone.

Comment 7 Oved Ourfali 2016-07-21 16:18:25 UTC
Alexander, was it merged before 4.0.2 branch was created?

Comment 8 Alexander Wels 2016-07-21 16:57:49 UTC
No it was not, I can backport if needed, but this is not something new, so I didn't think it was worth backporting.

Comment 9 Alexander Wels 2016-07-21 17:23:21 UTC
Actually, just checked the 4.0.2 branch, and it was merged before the branch was created.

Comment 10 Pavel Novotny 2016-07-28 15:20:53 UTC
Verified in
ovirt-engine-webadmin-portal-4.0.2.1-0.1.el7ev.noarch
rhevm-4.0.2.1-0.1.el7ev.noarch

If more than one UI exception occurs, only one alert box is shown with a counter on the right side. See attached screenshot for details.

Comment 11 Pavel Novotny 2016-07-28 15:21:53 UTC
Created attachment 1185157 [details]
single exception alert with a counter


Note You need to log in before you can comment on or make changes to this bug.