Bug 795891

Summary:	"Vanished" images which are deleted should also change deployment status
Product:	[Retired] CloudForms Cloud Engine	Reporter:	Matt Wagner <matt.wagner>
Component:	aeolus-conductor	Assignee:	Matt Wagner <matt.wagner>
Status:	CLOSED CURRENTRELEASE	QA Contact:	wes hayutin <whayutin>
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	1.0.0	CC:	akarol, athomas, dajohnso, deltacloud-maint, ssachdev
Target Milestone:	rc
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:	v0.8.0-35	Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2012-08-30 17:17:02 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:

Description Matt Wagner 2012-02-21 18:23:48 UTC

Description of problem:
If you have an instance which disappears on the backend cloud provider, dbomatic will move its state to "vanished," and then destroy the instance on subsequent queries. However, we never actually do anything with the deployment.

This leaves you in a bizarre state, where you have a Deployment in running state, but with zero instances.

There are a few things we should do here (either stop deleting instances in this case, or also delete the deployment), but in any case, our changes to the instance need to propagate up to the deployment.

Comment 1 Matt Wagner 2012-02-21 20:15:28 UTC

Oh, there isn't an actual :status attribute on deployments, it's just derived from the instances. So this is just a bug -- a deployment with no instances shouldn't be in "running" state.

Comment 2 Matt Wagner 2012-02-21 21:10:32 UTC

I sent out a series of patches to the list: http://lists.fedorahosted.org/pipermail/aeolus-devel/2012-February/009068.html

I ended up combining this with 795794 as well in the patchset. They're distinct but related issues.

Comment 3 Matt Wagner 2012-02-22 18:00:33 UTC

Pushed 3 patches to master to address these issues. The commit hashes are in https://bugzilla.redhat.com/show_bug.cgi?id=795794

Comment 4 wes hayutin 2012-02-24 02:41:38 UTC

To test this I think either Dave or myself will have to shut down a vsphere or rhevm hypervisor that hosts a running instance.

Comment 5 Dave Johnson 2012-02-24 03:15:58 UTC

good 2 go with aeolus-conductor-0.8.0-35.el6.noarch