Bug 1534460

Summary:	EventCatcher is not restarted when Nuage provider is updated
Product:	Red Hat CloudForms Management Engine	Reporter:	mplesko
Component:	Providers	Assignee:	Gregor Berginc <gregor.berginc>
Status:	ON_QA ---	QA Contact:	Gregor Berginc <gregor.berginc>
Severity:	medium	Docs Contact:
Priority:	medium
Version:	unspecified	CC:	cpelland, jhardy, simaishi
Target Milestone:	GA	Keywords:	TestOnly, ZStream
Target Release:	5.10.0
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:	5.10.0.0	Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:
Clones:	1552335 (view as bug list)		Environment:
Last Closed:		Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:
Bug Blocks:	1552335

Description mplesko 2018-01-15 10:03:26 UTC

Description of problem:
When I update the endpoint of Nuage AMQP and press "Save", the changes are not applied until I `kill -9` the EventCatcher thread manually or restart the whole server.

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


How reproducible:
Always

Steps to Reproduce:
1. Edit Nuage provider, entert new AMQP endpoint
2. Hit "Save"

Actual results:
EventCatcher is still connected to the old endpoint.

Expected results:
EventCatcher should be restarted. On restart it should grab new endpoint and connect there.


Additional info:

Comment 2 Dave Johnson 2018-01-15 10:03:57 UTC

Please assess the impact of this issue and update the severity accordingly.  Please refer to https://bugzilla.redhat.com/page.cgi?id=fields.html#bug_severity for a reminder on each severity's definition.

If it's something like a tracker bug where it doesn't matter, please set the severity to Low.

Comment 4 CFME Bot 2018-02-06 14:00:03 UTC

https://github.com/ManageIQ/manageiq-providers-nuage/pull/65

Comment 5 CFME Bot 2018-02-06 14:30:32 UTC

New commit detected on ManageIQ/manageiq-providers-nuage/master:
https://github.com/ManageIQ/manageiq-providers-nuage/commit/db46d35e1daf0771bf8802efc466922a8be30ed3

commit db46d35e1daf0771bf8802efc466922a8be30ed3
Author:     sasoc <saso.cvitkovic>
AuthorDate: Tue Feb 6 14:38:58 2018 +0100
Commit:     sasoc <saso.cvitkovic>
CommitDate: Tue Feb 6 14:38:58 2018 +0100

    Add `stop_event_monitor_queue_on_change` method
    
    `stop_event_monitor_queue_on_change` is now overriden for Nuage network provider so EventCatcher restarts when AMQP endpoints or fallback endpoints or port are changed.
    
    This commit replaces https://github.com/ManageIQ/manageiq-providers-nuage/pull/60#issuecomment-359870507.
    
    Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1534460

 app/models/manageiq/providers/nuage/manager_mixin.rb | 11 +++++++++++
 1 file changed, 11 insertions(+)