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

fix: improve actions waiting timeout based on data #488

Merged
merged 4 commits into from
Apr 15, 2024

Conversation

jooola
Copy link
Collaborator

@jooola jooola commented Apr 4, 2024

SUMMARY

Some action waiting time have been set to an arbitrary number, which could force the users to wait for too long, while we could have raised a timeout.

This changes the arbitrary numbers with rough estimate based on the average actions time and some leeway.

@jooola jooola force-pushed the unchain-actions-wait branch 2 times, most recently from e519687 to 20885cd Compare April 8, 2024 13:57
@jooola jooola force-pushed the unchain-actions-wait branch from 20885cd to 7028e24 Compare April 8, 2024 13:58
@jooola jooola marked this pull request as ready for review April 8, 2024 13:58
@jooola jooola requested a review from apricote April 8, 2024 13:58
plugins/modules/certificate.py Outdated Show resolved Hide resolved
plugins/modules/server.py Outdated Show resolved Hide resolved
@jooola jooola merged commit 0709552 into ansible-collections:main Apr 15, 2024
22 of 25 checks passed
@jooola jooola deleted the unchain-actions-wait branch April 15, 2024 09:08
jooola pushed a commit that referenced this pull request Apr 15, 2024
🤖 I have created a release *beep* *boop*
---


##
[3.1.0](3.0.0...3.1.0)
(2024-04-15)


### Features

* assign primary ip to a server on create
([#465](#465))
([51afb23](51afb23))
* rename server `image_allow_deprecated` option
([#487](#487))
([d88ecdb](d88ecdb))
* use resources name or ID in server module arguments
([#484](#484))
([7fdefcf](7fdefcf))


### Bug Fixes

* improve actions waiting timeout based on data
([#488](#488))
([0709552](0709552))
* return sorted `alias_ips` in `server_network` module
([#458](#458))
([1ae6769](1ae6769))
* use empty string to remove server from its placement group
([#489](#489))
([00a4fdd](00a4fdd))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants