Bug 985635 - Changing email address for event notification results in error "User is already subscribed to this event with the same Notification method"
Summary: Changing email address for event notification results in error "User is alrea...
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat Enterprise Virtualization Manager
Classification: Red Hat
Component: ovirt-engine-webadmin-portal
Version: 3.2.0
Hardware: Unspecified
OS: Unspecified
high
medium
Target Milestone: ---
: 3.3.0
Assignee: Ravi Nori
QA Contact: Pavel Stehlik
URL:
Whiteboard: infra
Depends On:
Blocks: 996970
TreeView+ depends on / blocked
 
Reported: 2013-07-18 00:15 UTC by Bryan Yount
Modified: 2018-12-03 19:24 UTC (History)
11 users (show)

Fixed In Version: is10
Doc Type: Bug Fix
Doc Text:
--no tech note required
Clone Of:
: 996970 (view as bug list)
Environment:
Last Closed: 2014-01-21 17:34:16 UTC
oVirt Team: Infra
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
engine.log from my test environment (39.78 KB, text/x-log)
2013-07-18 00:15 UTC, Bryan Yount
no flags Details
Check the address as well in the IsAlreadySubscribed method in AddEventSubscriptionCommand.java (2.66 KB, patch)
2013-07-18 02:13 UTC, Mark Huth
no flags Details | Diff


Links
System ID Private Priority Status Summary Last Updated
Red Hat Knowledge Base (Solution) 475413 0 None None None Never
Red Hat Product Errata RHSA-2014:0038 0 normal SHIPPED_LIVE Important: Red Hat Enterprise Virtualization Manager 3.3.0 update 2014-01-21 22:03:06 UTC
oVirt gerrit 17984 0 None None None Never
oVirt gerrit 18401 0 None None None Never

Description Bryan Yount 2013-07-18 00:15:04 UTC
Created attachment 775028 [details]
engine.log from my test environment

Description of problem:
A user (for example the admin@internal user) is subscribed to event notifications and an email address is set to receive these notifications. After saving the changes, the user realizes they set the wrong email address. They re-enter the dialogue box and change the email address which results in a pop-up error box with the message: "User is already subscribed to this event with the same Notification method."

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

How reproducible:
Very

Steps to Reproduce:
1. Log in to Admin Portal
2. Click on Users tab
3. Select a user (in this example, admin@internal)
4. Click on the Event Notifier subtab and select "Manage Events"
5. Configure various event notifications
6. Set an email address
7. Click OK
8. Re-enter the "Manage Events" window
9. Change the email address to something different
10. Click OK

Actual results:
The error message appears and causes all/most of the configured events to disappear

Expected results:
The email address should become the changed address

Comment 2 Mark Huth 2013-07-18 02:13:42 UTC
Created attachment 775032 [details]
Check the address as well in the IsAlreadySubscribed method in AddEventSubscriptionCommand.java

This issue seems to have been in RHEVM since 3.1 GA.  I was able to reproduce the problem on a RHEVM3.1.4 environment.

After applying this patch, I no longer get the "User is already subscribed to this event with the same Notification method" popup if I simply change the email address of the event subscriber.

Its interesting, this comment in AddEventSubscriptionCommand.java mentions checking the address, but the IsAlreadySubscribed method doesn't:

 25     protected boolean canDoAction() {
 26         boolean retValue;
 27         // check if user is not already subscribed to this event with same
 28         // method and address

... unless I'm misinterpreting what 'address' is referring to.

Comment 4 David Botzer 2013-09-04 09:13:38 UTC
The Bug is verified, no error message,
though I cant get any mail, Should I pre-configure anything ?

Comment 5 Ravi Nori 2013-09-04 13:13:36 UTC
To received email we will need to setup engine-notification-service and specify the mail host information in the configuration file for the notification-service on the engine.

Comment 6 David Botzer 2013-09-08 05:24:15 UTC
Hi,
Where and what do I need to setup in - engine-notification-service ?
And where is the configuration file for the notification-service ?

Comment 7 David Botzer 2013-09-08 07:34:23 UTC
3.3/is12 - cannot get mails
I did -
1.
In users tab
I marked for admin user all events, & inserted a mail address

2.
cp /usr/share/ovirt-engine/services/ovirt-engine-notifier/ovirt-engine-notifier.conf to /etc/ovirt-engine/notifier/notifier.conf
Configure with server & mail

# The SMTP mail server address. Required.
MAIL_SERVER=xx.xx.xx.com

MAIL_USER=
MAIL_PASSWORD=

MAIL_FROM=xx

restarted ovirt-engine-notifier

3. 
I had brought a Host to maintanence/non-responsive and did some vm actions but no mails

Comment 8 David Botzer 2013-09-08 07:49:48 UTC
Fixed, 3.3/is12
Got the notification, it took some time, but i got it,

Time:2013-09-08 10:41:00.909
Message:Engine server is not responding.
Severity:1

Fixed, 3.3/is12

Comment 9 Charlie 2013-11-28 00:21:56 UTC
This bug is currently attached to errata RHEA-2013:15231. If this change is not to be documented in the text for this errata please either remove it from the errata, set the requires_doc_text flag to minus (-), or leave a "Doc Text" value of "--no tech note required" if you do not have permission to alter the flag.

Otherwise to aid in the development of relevant and accurate release documentation, please fill out the "Doc Text" field above with these four (4) pieces of information:

* Cause: What actions or circumstances cause this bug to present.
* Consequence: What happens when the bug presents.
* Fix: What was done to fix the bug.
* Result: What now happens when the actions or circumstances above occur. (NB: this is not the same as 'the bug doesn't present anymore')

Once filled out, please set the "Doc Type" field to the appropriate value for the type of change made and submit your edits to the bug.

For further details on the Cause, Consequence, Fix, Result format please refer to:

https://bugzilla.redhat.com/page.cgi?id=fields.html#cf_release_notes 

Thanks in advance.

Comment 11 errata-xmlrpc 2014-01-21 17:34:16 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

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

http://rhn.redhat.com/errata/RHSA-2014-0038.html


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