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

Implement Resource Waiter Actions #43

Merged
merged 3 commits into from Dec 17, 2014
Merged

Conversation

kyleknap
Copy link
Member

So for this PR, I expose waiter actions for resources. For each waiter action, the name was prepended with wait_until_. So for the EC2 Instance resource definition, it has waiter resource definitions: "Running", "Stopped", and "Terminated": https://github.com/boto/boto3/blob/develop/boto3/data/resources/ec2-2014-10-01.resources.json#L694

This gets translated to the methods: wait_until_running(), wait_until_stopped(), and wait_until_terminated() for the Instance resource. These methods will then make a wait() call to the client's waiter method to actually perform the wait.

cc @danielgtaylor @jamesls

For each resource that has a resource waiter definition a ``wait_until_...``
method was added to the resource. Documentation was updated as well to
document both the resource waiter and client waiters.
@coveralls
Copy link

Coverage Status

Coverage increased (+0.06%) when pulling 8124dde on kyleknap:waiters into 8b10f29 on boto:develop.

danielgtaylor referenced this pull request Dec 17, 2014
Remove the dependency on six module.
@danielgtaylor
Copy link
Member

LGTM 🚢-it!

@danielgtaylor danielgtaylor added the enhancement This issue requests an improvement to a current feature. label Dec 17, 2014
kyleknap added a commit that referenced this pull request Dec 17, 2014
Implement Resource Waiter Actions
@kyleknap kyleknap merged commit 279d058 into boto:develop Dec 17, 2014
@kyleknap kyleknap deleted the waiters branch December 17, 2014 19:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement This issue requests an improvement to a current feature.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants