Bug 1535864 - Webadmin- Events - notification drawer- clear all alarms clear also all events
Summary: Webadmin- Events - notification drawer- clear all alarms clear also all events
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: ovirt-engine
Classification: oVirt
Component: Frontend.WebAdmin
Version: 4.2.1.1
Hardware: Unspecified
OS: Unspecified
unspecified
medium
Target Milestone: ovirt-4.2.2
: ---
Assignee: Eli Mesika
QA Contact: Petr Matyáš
URL:
Whiteboard:
: 1539721 (view as bug list)
Depends On:
Blocks: 1552473
TreeView+ depends on / blocked
 
Reported: 2018-01-18 07:57 UTC by Avihai
Modified: 2018-03-29 10:54 UTC (History)
7 users (show)

Fixed In Version:
Clone Of:
: 1552473 (view as bug list)
Environment:
Last Closed: 2018-03-29 10:54:02 UTC
oVirt Team: Infra
Embargoed:
rule-engine: ovirt-4.2+
rule-engine: blocker+


Attachments (Terms of Use)
engine log + print screen (554.57 KB, application/x-gzip)
2018-01-18 07:57 UTC, Avihai
no flags Details
4.1 record (1.02 MB, application/x-gzip)
2018-02-14 11:51 UTC, Michael Burman
no flags Details


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 88238 0 master MERGED core: fixing removal of alerts/events 2018-02-27 16:25:54 UTC
oVirt gerrit 88239 0 ovirt-engine-4.2 MERGED core: fixing removal of alerts/events 2018-02-27 16:44:17 UTC

Description Avihai 2018-01-18 07:57:01 UTC
Created attachment 1382761 [details]
engine log + print screen

Description of problem:
Webadmin -> Events -->  notification drawer-> clear all alarms

In the same window you have both the notification drawer & the events (see print screen) - See that also all events are cleared even only alarms should be cleared.

Version-Release number of selected component (if applicable):
4.2.1.1-0.1.el7

How reproducible:
100% so far

Steps to Reproduce:
1.Webadmin -> Events -->  notification drawer-> clear all alarms

Actual results:
In the same window you have both the notification drawer & the events (see print screen) - See that also all events are cleared even only alarms should be cleared.


Expected results:


Additional info:

Comment 1 Greg Sheremeta 2018-01-19 15:51:24 UTC
I reproduced. There are 2 "clear / display" rows, one each for Events and Alerts. They should probably function independently.

@Alexander, bug?

Comment 2 Alexander Wels 2018-01-19 15:58:02 UTC
Could be a backend bug, the UI calls the clearAllCommand on the EventListModel for events, and clearAllCommand on the AlertListModel for the alerts. Once it goes to the backend I don't know what it does. I do know that Alerts and Events both share the same table, so its possible it executes the same command on the backend.

Comment 3 Greg Sheremeta 2018-01-19 16:09:04 UTC
Moving to infra per Comment 2

Comment 4 Greg Sheremeta 2018-01-30 17:32:18 UTC
*** Bug 1539721 has been marked as a duplicate of this bug. ***

Comment 5 Michael Burman 2018-01-30 19:23:51 UTC
4.1.9.2-0.1.el7 affected as well(not sure if you are aware) and this a regression bug.

Comment 6 Red Hat Bugzilla Rules Engine 2018-01-30 19:23:56 UTC
This bug report has Keywords: Regression or TestBlocker.
Since no regressions or test blockers are allowed between releases, it is also being identified as a blocker for this release. Please resolve ASAP.

Comment 7 Martin Perina 2018-02-07 09:18:49 UTC
Eli, how severe is this one?

Comment 8 Martin Perina 2018-02-14 10:13:30 UTC
(In reply to Michael Burman from comment #5)
> 4.1.9.2-0.1.el7 affected as well(not sure if you are aware) and this a
> regression bug.

We haven't found any options in 4.1.9 how to remove all alerts, so could you please provide reproduction steps in 4.1?

Comment 9 Michael Burman 2018-02-14 11:50:08 UTC
(In reply to Martin Perina from comment #8)
> (In reply to Michael Burman from comment #5)
> > 4.1.9.2-0.1.el7 affected as well(not sure if you are aware) and this a
> > regression bug.
> 
> We haven't found any options in 4.1.9 how to remove all alerts, so could you
> please provide reproduction steps in 4.1?

Sure,
Go to Alerts sub tab, click the right click mouse menu and choose Clear All )
Attaching record

Comment 10 Michael Burman 2018-02-14 11:51:16 UTC
Created attachment 1395869 [details]
4.1 record

Comment 11 Petr Matyáš 2018-03-06 09:45:26 UTC
Verified on ovirt-engine-4.2.2.2-0.1.el7.noarch

Comment 12 Michael Burman 2018-03-07 07:22:30 UTC
Hi Martin,
And what about a fix for 4.1?? i don't see any patches for 4.1. I honestly think this should be fixed on 4.1 as well.

Comment 13 Martin Perina 2018-03-07 08:24:54 UTC
(In reply to Michael Burman from comment #12)
> Hi Martin,
> And what about a fix for 4.1?? i don't see any patches for 4.1. I honestly
> think this should be fixed on 4.1 as well.

Right, sorry, I completely missed your comment and I've looked to Version field only where is no longer possible to enter 4.1 :-(

Comment 15 Sandro Bonazzola 2018-03-29 10:54:02 UTC
This bugzilla is included in oVirt 4.2.2 release, published on March 28th 2018.

Since the problem described in this bug report should be
resolved in oVirt 4.2.2 release, it has been closed with a resolution of CURRENT RELEASE.

If the solution does not work for you, please open a new bug report.


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