Add boto3action to aws pack#44
Merged
warrenvw merged 11 commits intoStackStorm-Exchange:boto3from Aug 9, 2017
Merged
Conversation
…d json_serial into lib
warrenvw
reviewed
Jul 31, 2017
| `aws.boto3action` runs boto3 actions in stackstorm dynamically. It has following features. | ||
|
|
||
| - Uses boto3 configurations. Find more information on boto3 configuration in boto3 documentation. http://boto3.readthedocs.io/en/latest/guide/quickstart.html#configuration | ||
| - Ablity to run cross region actions |
warrenvw
reviewed
Jul 31, 2017
|
|
||
| - Uses boto3 configurations. Find more information on boto3 configuration in boto3 documentation. http://boto3.readthedocs.io/en/latest/guide/quickstart.html#configuration | ||
| - Ablity to run cross region actions | ||
| - Ablility to run cross account actions. |
warrenvw
suggested changes
Jul 31, 2017
Contributor
warrenvw
left a comment
There was a problem hiding this comment.
I'm asking for a few minor tweaks. Before we merge boto3 into master, we'll make sure that all outstanding issues are resolved, and users are notified of the upcoming changes.
warrenvw
approved these changes
Aug 4, 2017
Author
|
@warrenvw Thanks for approving changes. Please let me know if you need anything from me to make progress on this PR. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
I create this pull request as a result of discussion happened in, StackStorm-Exchange/exchange-incubator#10. Main features in this pull request are,
For example, I have a st2 deployment in aws account 123456 and us-east-1. I want to deploy VPC in account 456789 and us-west-1.
In addition,
aws.boto3actioncreated with following opinions.Boto3 is the official SDK for AWS. As a user/developer, If I have boto3 configured I expect
awspack to work without doing any additional configuration. For example,In addition, if I want to use use boto3 profiles
yaml generation
Long term, I don’t believe yaml generation scale based on the number of services AWS have and introduce. In addition, IMO Boto3 documentation is detailed, has examples. Having yaml for each action is redundant and add no value to the end user.
pack maintenance
Since there are no yaml to generate, this pack should be easy to maintain. Any new service boto3 introduce, available to pack user right away.