Skip to content

Update CloudStack NodeStates for Stopped/Stopping#246

Closed
cderamus wants to merge 13 commits into
apache:trunkfrom
DivvyCloud:Update_CloudStack_Node_States
Closed

Update CloudStack NodeStates for Stopped/Stopping#246
cderamus wants to merge 13 commits into
apache:trunkfrom
DivvyCloud:Update_CloudStack_Node_States

Conversation

@cderamus
Copy link
Copy Markdown
Contributor

@cderamus cderamus commented Feb 6, 2014

No description provided.

@Kami
Copy link
Copy Markdown
Member

Kami commented Feb 7, 2014

We've discussed this on the IRC with @cderamus yesterday.

I think we should also map stopping and expunging to pending instead of terminated. The reason for that is that not all the operations are available until the node is fully stopped (stopped state).

This means that currently your code might fail if you wait until node.state == stopped because node might actually still be in stopping state.

@Kami
Copy link
Copy Markdown
Member

Kami commented Feb 7, 2014

I've merged the changes mentioned above and merged this into trunk.

Thanks.

@cderamus cderamus closed this Feb 7, 2014
@cderamus cderamus deleted the Update_CloudStack_Node_States branch February 8, 2014 12:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants