Bug 795891 - "Vanished" images which are deleted should also change deployment status
Summary: "Vanished" images which are deleted should also change deployment status
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: CloudForms Cloud Engine
Classification: Retired
Component: aeolus-conductor
Version: 1.0.0
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: rc
Assignee: Matt Wagner
QA Contact: wes hayutin
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-02-21 18:23 UTC by Matt Wagner
Modified: 2012-08-30 17:17 UTC (History)
5 users (show)

Fixed In Version: v0.8.0-35
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-08-30 17:17:02 UTC
Embargoed:


Attachments (Terms of Use)

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


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