Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fails to create change sets when a stack is in the REVIEW_IN_PROGRESS state #48

Open
amcinnes opened this issue Nov 6, 2017 · 5 comments
Labels

Comments

@amcinnes
Copy link
Contributor

amcinnes commented Nov 6, 2017

Using stackup 1.2.0

If I do change-set create against a stack that doesn't exist, a stack is created in the REVIEW_IN_PROGRESS state, as expected.

If I then try to create another change set, I get ERROR: no such stack.

I believe this is because stackup uses a change set type of UPDATE when the stack already exists -- but for a stack in the REVIEW_IN_PROGRESS state, it's appropriate to use a change set type of CREATE instead.

@amcinnes
Copy link
Contributor Author

amcinnes commented Nov 6, 2017

(I concede that this isn't a particularly important issue, as it's always safe to delete a REVIEW_IN_PROGRESS stack and try again...)

@ghost ghost added the bug label Nov 6, 2017
@amcinnes
Copy link
Contributor Author

amcinnes commented Nov 6, 2017

A related observation:

Normally if I update a stack that's in the ROLLBACK_COMPLETE state, stackup will automatically delete the stack and recreate it.

But if I try to create a change set for a stack that's in the ROLLBACK_COMPLETE state, stackup will say ERROR: Stack:... is in ROLLBACK_COMPLETE state and can not be updated.

I wonder if it would be appropriate to automatically delete a stack in ROLLBACK_COMPLETE (or CREATE_FAILED) before trying to create a change set for it.

@mbelekar
Copy link
Contributor

mbelekar commented Nov 9, 2017

@amcinnes The stackup up/down command behaves like an upsert, where as the change sets are a CRUD operation. We can update the documentation to reflect this difference in behaviour.
Right now we don't know when we will be supporting an upsert operation on change sets.

@stevemac007
Copy link

Just ran into this one myself - requires manual intervention by someone to delete these dangling stacks - It would be awesome if stackup could cleanup these REVIEW_IN_PROGRESS stacks as part of normal operation.

@mdub
Copy link
Contributor

mdub commented May 30, 2018

It would make sense to be consistent with the auto-cleanup on stack creation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants