Bug 1552439 - [UI] - Alerts - wrong message when clearing all Alerts
Summary: [UI] - Alerts - wrong message when clearing all Alerts
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: ovirt-engine
Classification: oVirt
Component: Frontend.WebAdmin
Version: 4.2.2
Hardware: x86_64
OS: Linux
unspecified
low
Target Milestone: ovirt-4.2.3
: ---
Assignee: Ori Liel
QA Contact: samuel macko
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2018-03-07 07:09 UTC by Michael Burman
Modified: 2018-05-10 06:30 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2018-05-10 06:30:31 UTC
oVirt Team: Infra
Embargoed:
rule-engine: ovirt-4.2+
lsvaty: testing_ack+


Attachments (Terms of Use)
engine log (190.52 KB, application/x-gzip)
2018-03-07 07:14 UTC, Michael Burman
no flags Details


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 89276 0 master MERGED Clear Alert- Change Message 2018-03-22 10:56:41 UTC
oVirt gerrit 89342 0 ovirt-engine-4.2 MERGED Clear Alert- Change Message 2018-04-02 16:12:59 UTC

Description Michael Burman 2018-03-07 07:09:54 UTC
Description of problem:
Now when clear all Alerts was fixed, the message in the audit log is wrong. 
When clearing all Alerts(only alerts), the next message appears - 

All events were removed. (User: admin@internal-authz)
3/7/189:03:15 AM

But it is confusing and wrong, it should be 'All alerts were removed' 

- When clearing only events, the message is ok(All events were removed. (User: admin@internal-authz)

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

How reproducible:
100%

Steps to Reproduce:
1. Clear all Alerts only

Actual results:
All events were removed. (User: admin@internal-authz)
3/7/189:03:15 AM

Expected results:
All "Alerts" were removed. (User: admin@internal-authz)
3/7/189:03:15 AM

Comment 1 Michael Burman 2018-03-07 07:14:39 UTC
Created attachment 1405158 [details]
engine log

Comment 2 Alexander Wels 2018-03-08 20:35:59 UTC
@Tomas

The UI simply displays the stuff from the table. so whomever fixed the backend forgot to fix the associated message. Can you assign it to them, as the UI simply displays the string it gets from the backend.

Comment 3 samuel macko 2018-04-16 12:37:14 UTC
Verified in ovirt version 4.2.3-0.1.el7.

Verified by following the reproducer.

Comment 4 Sandro Bonazzola 2018-05-10 06:30:31 UTC
This bugzilla is included in oVirt 4.2.3 release, published on May 4th 2018.

Since the problem described in this bug report should be
resolved in oVirt 4.2.3 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.