Skip to content

Add wait_until_finished Method to Action#10

Merged
LKaemmerling merged 11 commits intomasterfrom
add-wait-until-finished-to-action
Feb 14, 2019
Merged

Add wait_until_finished Method to Action#10
LKaemmerling merged 11 commits intomasterfrom
add-wait-until-finished-to-action

Conversation

@LKaemmerling
Copy link
Copy Markdown
Member

Closes #4

Comment thread hcloud/actions/client.py Outdated
Comment thread hcloud/actions/client.py Outdated
Comment thread hcloud/actions/client.py Outdated
Comment thread tests/unit/actions/test_client.py Outdated
Comment thread hcloud/actions/client.py Outdated
Use correct poll_interval variable from client
@LKaemmerling
Copy link
Copy Markdown
Member Author

It shouldn't be part of the BoundAction because we shouldn't want to let the user configure the polling interval on per action base.

The user can configure the interval once, (on the client configuration):

client.with_poll_interval(0.5) # So every poll we do has a interval (or timeout) of 0.5 sec

We implemented the same way in the hcloud-go. https://github.com/hetznercloud/hcloud-go/blob/master/hcloud/client.go#L91

Comment thread hcloud/actions/domain.py Outdated
Comment thread tests/unit/test_hcloud.py Outdated
@LKaemmerling LKaemmerling merged commit cf9cfea into master Feb 14, 2019
@LKaemmerling LKaemmerling deleted the add-wait-until-finished-to-action branch February 14, 2019 09:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants