-
Notifications
You must be signed in to change notification settings - Fork 167
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
Initial pass at interactive aws provider #178
Conversation
I updated this a little bit:
I'm going to implement some alternative outputs, but I'll do them in a separate PR since the core of this one should just be the implementation of the interactive provider. Now you can specify Right now interactive builds display each change set. I'm going to change this in a separate PR so by default we only display when the change set requires a replacement. I'll then add a |
https://github.com/mhahn/stacker/pull/1/files I'll submit that against this one when this is merged. Adds the "verbose" option that you had mentioned. By default we'll only list out the replacements, but you can provide "v" to list out all the changes:
|
@@ -242,6 +246,9 @@ def execute(self): | |||
if attempts == 1: | |||
self._check_point() | |||
self._wait_func(self.sleep_time) | |||
except CancelExecution: | |||
logger.info("Cancelling execution") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Worth including the plan step here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i'll keep this in mind, i might make some changes to the output, i'll take a look at this then
Looks good other than my comments, thanks. |
|
||
from ... import exceptions | ||
from .default import ( | ||
Provider as BaseProvider, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
AWSProvider maybe instead.
Not ready to merge, but wanted to get it out there to start some discussion around the output:
Right now it looks like:
I'm going to play with some cli libraries to see if I can come up with something cleaner.