Bug 1446585

Summary:	CFME servers not deleting from Web UI
Product:	Red Hat CloudForms Management Engine	Reporter:	Ryan Spagnola <rspagnol>
Component:	Appliance	Assignee:	Gregg Tanzillo <gtanzill>
Status:	CLOSED ERRATA	QA Contact:	Alex Newman <anewman>
Severity:	medium	Docs Contact:
Priority:	medium
Version:	5.7.0	CC:	abellott, dluong, fdewaley, gekis, greartes, gtanzill, hkataria, jhardy, mfeifer, mpovolny, ncarboni, obarenbo, rspagnol, simaishi, yrudman
Target Milestone:	GA	Keywords:	Reopened
Target Release:	5.9.0
Hardware:	All
OS:	All
Whiteboard:	appliance:distributed
Fixed In Version:	5.9.0.1	Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2018-03-01 13:12:04 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	CFME Core	Target Upstream Version:
Embargoed:

Comment 16 Nick Carboni 2017-09-15 20:01:01 UTC

I was not able to reproduce this issue, but I didn't try with any significant number of events for the server I was deleting.

While trying to reproduce this I noticed that the configuration dropdown to delete the server is missing in version 5.8.1.5 so I opened bug 1492216 to track that issue separately.

I'm tempted to close this, but would like to propose changing the dependent destroy to an after destroy callback which queues the deletion of the events.

This should prevent the call to destroy the server from hanging, but I'm not sure what kind of problems the hanging reference to the removed server would cause. Thoughts?

Comment 17 Yuri Rudman 2017-09-19 12:08:07 UTC

I think if server stopped and there were no activity for some time than it is OK to delete server first and replace dependent=>destroy with after_destroy for EventStream.
Gregg, what do you think?

Comment 18 CFME Bot 2017-09-19 18:06:54 UTC

https://github.com/ManageIQ/manageiq/pull/15995

Comment 19 CFME Bot 2017-09-21 18:07:00 UTC

New commit detected on ManageIQ/manageiq/master:
https://github.com/ManageIQ/manageiq/commit/79ca041e68148f98c3506614416f3ebd4ee569f8

commit 79ca041e68148f98c3506614416f3ebd4ee569f8
Author:     Yuri Rudman <yrudman>
AuthorDate: Tue Sep 19 13:35:12 2017 -0400
Commit:     Yuri Rudman <yrudman>
CommitDate: Wed Sep 20 15:07:05 2017 -0400

    queue deletion of lihked events instead of dependent=>destroy. It should prevent UI to hang when deleting instance of MiqServer
    https://bugzilla.redhat.com/show_bug.cgi?id=1446585

 app/models/miq_server.rb | 16 +++++++++++++++-
 1 file changed, 15 insertions(+), 1 deletion(-)

Comment 32 errata-xmlrpc 2018-03-01 13:12:04 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.

https://access.redhat.com/errata/RHSA-2018:0380