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

Add retries on the same failures as AWSRetry for the waiters #185

Merged
merged 3 commits into from
Nov 11, 2020

Conversation

tremble
Copy link
Contributor

@tremble tremble commented Oct 29, 2020

SUMMARY

Try getting our waiters to retry on the usual ratelimiting errors

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

plugins/module_utils/waiters.py

ADDITIONAL INFORMATION

Copy link
Collaborator

@jillr jillr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tremble the general idea seems sensible, could you please ci_complete this to be safe? I'd also like to do some testing of community.aws against this change (locally).

@jillr
Copy link
Collaborator

jillr commented Nov 6, 2020

I've made it through half of the community.aws tests for modules with waiters with no issues so far

@tremble tremble changed the title [WIP] Add retries on the same failures as AWSRetry for the waiters Add retries on the same failures as AWSRetry for the waiters Nov 7, 2020
@tremble tremble closed this Nov 11, 2020
@tremble tremble reopened this Nov 11, 2020
Copy link
Collaborator

@jillr jillr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No other relevant test failures running this with community.aws modules (cloudfront_distribution is being difficult for unrelated reasons). LGTM - thanks @tremble!

@jillr jillr merged commit 3ab74ec into ansible-collections:main Nov 11, 2020
@tremble tremble deleted the waiters/retry_ratelimit branch May 28, 2021 07:08
alinabuzachis pushed a commit to alinabuzachis/amazon.aws that referenced this pull request Sep 9, 2022
* Prepare 1.1.0 release

Run add_docs.py
Run antsibull-changelog
Bump release in galaxy.yml
Update meta/runtime.yml with supported ansible revs
Add requirements file to support execution environments
Update amazon.aws rev in test requirements

* Cleanup changelog formatting for new, sticter antsibull-changelog formatting
Rerun add_docs.py with --branch-name flag
Correct required amazon.aws revision to match the current in-progress release

* add_docs generated bad markup, fix that

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@e95fb90
alinabuzachis pushed a commit to alinabuzachis/amazon.aws that referenced this pull request Sep 9, 2022
* Prepare 1.1.0 release

Run add_docs.py
Run antsibull-changelog
Bump release in galaxy.yml
Update meta/runtime.yml with supported ansible revs
Add requirements file to support execution environments
Update amazon.aws rev in test requirements

* Cleanup changelog formatting for new, sticter antsibull-changelog formatting
Rerun add_docs.py with --branch-name flag
Correct required amazon.aws revision to match the current in-progress release

* add_docs generated bad markup, fix that

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@e95fb90
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Oct 24, 2023
* Prepare 1.1.0 release

Run add_docs.py
Run antsibull-changelog
Bump release in galaxy.yml
Update meta/runtime.yml with supported ansible revs
Add requirements file to support execution environments
Update amazon.aws rev in test requirements

* Cleanup changelog formatting for new, sticter antsibull-changelog formatting
Rerun add_docs.py with --branch-name flag
Correct required amazon.aws revision to match the current in-progress release

* add_docs generated bad markup, fix that
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