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.

Sign up for GitHub

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

Jump to bottom

ec2_instance retry on InsuffienctInstanceCapacity #1039

Merged
merged 2 commits into from
Sep 20, 2022
Merged

ec2_instance retry on InsuffienctInstanceCapacity #1039

merged 2 commits into from
Sep 20, 2022

Conversation

briantist
Copy link
Contributor

SUMMARY

Closes #1038

Expands the use of the AWSRetry decorator to include InsuffienctInstanceCapacity errors.

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

ec2_instance

@tremble tremble added the backport-4 PR should be backported to the stable-4 branch label Sep 20, 2022
@briantist briantist changed the title Ec2 instance/insufficient capacity ec2_instance retry on InsuffienctInstanceCapacity Sep 20, 2022
@ansibullbot
Copy link

cc @jillr @ryansb @s-hertel @tremble
click here for bot help

@ansibullbot ansibullbot added community_review feature This issue/PR relates to a feature request module module plugins plugin (any type) small_patch Hopefully easy to review labels Sep 20, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

✔️ ansible-galaxy-importer SUCCESS in 4m 46s
✔️ build-ansible-collection SUCCESS in 5m 20s
✔️ ansible-test-sanity-aws-ansible-python38 SUCCESS in 9m 46s
✔️ ansible-test-sanity-aws-ansible-2.12-python38 SUCCESS in 10m 49s
✔️ ansible-test-sanity-aws-ansible-2.13-python38 SUCCESS in 10m 04s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 8m 12s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 7m 17s
✔️ ansible-test-splitter SUCCESS in 2m 52s
✔️ integration-amazon.aws-1 SUCCESS in 31m 35s
✔️ integration-amazon.aws-2 SUCCESS in 36m 41s
✔️ integration-amazon.aws-3 SUCCESS in 40m 54s
⚠️ integration-amazon.aws-4 SKIPPED
⚠️ integration-amazon.aws-5 SKIPPED
⚠️ integration-amazon.aws-6 SKIPPED
⚠️ integration-amazon.aws-7 SKIPPED
⚠️ integration-amazon.aws-8 SKIPPED
⚠️ integration-amazon.aws-9 SKIPPED
⚠️ integration-amazon.aws-10 SKIPPED
⚠️ integration-amazon.aws-11 SKIPPED
⚠️ integration-amazon.aws-12 SKIPPED
⚠️ integration-amazon.aws-13 SKIPPED
⚠️ integration-amazon.aws-14 SKIPPED
⚠️ integration-amazon.aws-15 SKIPPED
⚠️ integration-amazon.aws-16 SKIPPED
⚠️ integration-amazon.aws-17 SKIPPED
⚠️ integration-amazon.aws-18 SKIPPED
⚠️ integration-community.aws-1 SKIPPED
⚠️ integration-community.aws-2 SKIPPED
⚠️ integration-community.aws-3 SKIPPED
⚠️ integration-community.aws-4 SKIPPED
⚠️ integration-community.aws-5 SKIPPED
⚠️ integration-community.aws-6 SKIPPED
⚠️ integration-community.aws-7 SKIPPED
⚠️ integration-community.aws-8 SKIPPED
⚠️ integration-community.aws-9 SKIPPED
⚠️ integration-community.aws-10 SKIPPED
⚠️ integration-community.aws-11 SKIPPED
⚠️ integration-community.aws-12 SKIPPED
⚠️ integration-community.aws-13 SKIPPED
⚠️ integration-community.aws-14 SKIPPED
⚠️ integration-community.aws-15 SKIPPED
⚠️ integration-community.aws-16 SKIPPED
⚠️ integration-community.aws-17 SKIPPED
⚠️ integration-community.aws-18 SKIPPED
✔️ ansible-test-changelog SUCCESS in 2m 26s

@tremble tremble added the mergeit Merge the PR (SoftwareFactory) label Sep 20, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (gate pipeline).

✔️ ansible-galaxy-importer SUCCESS in 4m 51s
✔️ build-ansible-collection SUCCESS in 5m 23s
✔️ ansible-test-sanity-aws-ansible-python38 SUCCESS in 9m 11s
✔️ ansible-test-sanity-aws-ansible-2.12-python38 SUCCESS in 9m 08s
✔️ ansible-test-sanity-aws-ansible-2.13-python38 SUCCESS in 9m 01s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 6m 40s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 7m 42s
✔️ ansible-test-splitter SUCCESS in 2m 55s
✔️ integration-amazon.aws-1 SUCCESS in 22m 39s
✔️ integration-amazon.aws-2 SUCCESS in 47m 32s
✔️ integration-amazon.aws-3 SUCCESS in 36m 13s
⚠️ integration-amazon.aws-4 SKIPPED
⚠️ integration-amazon.aws-5 SKIPPED
⚠️ integration-amazon.aws-6 SKIPPED
⚠️ integration-amazon.aws-7 SKIPPED
⚠️ integration-amazon.aws-8 SKIPPED
⚠️ integration-amazon.aws-9 SKIPPED
⚠️ integration-amazon.aws-10 SKIPPED
⚠️ integration-amazon.aws-11 SKIPPED
⚠️ integration-amazon.aws-12 SKIPPED
⚠️ integration-amazon.aws-13 SKIPPED
⚠️ integration-amazon.aws-14 SKIPPED
⚠️ integration-amazon.aws-15 SKIPPED
⚠️ integration-amazon.aws-16 SKIPPED
⚠️ integration-amazon.aws-17 SKIPPED
⚠️ integration-amazon.aws-18 SKIPPED
⚠️ integration-community.aws-1 SKIPPED
⚠️ integration-community.aws-2 SKIPPED
⚠️ integration-community.aws-3 SKIPPED
⚠️ integration-community.aws-4 SKIPPED
⚠️ integration-community.aws-5 SKIPPED
⚠️ integration-community.aws-6 SKIPPED
⚠️ integration-community.aws-7 SKIPPED
⚠️ integration-community.aws-8 SKIPPED
⚠️ integration-community.aws-9 SKIPPED
⚠️ integration-community.aws-10 SKIPPED
⚠️ integration-community.aws-11 SKIPPED
⚠️ integration-community.aws-12 SKIPPED
⚠️ integration-community.aws-13 SKIPPED
⚠️ integration-community.aws-14 SKIPPED
⚠️ integration-community.aws-15 SKIPPED
⚠️ integration-community.aws-16 SKIPPED
⚠️ integration-community.aws-17 SKIPPED
⚠️ integration-community.aws-18 SKIPPED
✔️ ansible-test-changelog SUCCESS in 2m 14s

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit 0374961 into ansible-collections:main Sep 20, 2022
@patchback
Copy link

patchback bot commented Sep 20, 2022 •
edited
Loading

Backport to stable-4: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-4/0374961a71c89c62c36d650304a099d706d0bc9f/pr-1039

Backported as #1040

🤖 @patchback
I'm built with octomachinery and
my source is open — https://github.com/sanitizers/patchback-github-app.

patchback bot pushed a commit that referenced this pull request Sep 20, 2022
@briantist @patchback
ec2_instance retry on InsuffienctInstanceCapacity (#1039)
f05a159 
`ec2_instance` retry on `InsuffienctInstanceCapacity `

SUMMARY

Closes #1038
Expands the use of the AWSRetry decorator to include InsuffienctInstanceCapacity errors.

ISSUE TYPE

Feature Pull Request

COMPONENT NAME

ec2_instance

Reviewed-by: Mark Chappell <None>
(cherry picked from commit 0374961)
@patchback patchback bot mentioned this pull request Sep 20, 2022
Merged
@github-actions
Copy link

Docs Build 📝

Thank you for contribution!✨

This PR has been merged and your docs changes will be incorporated when they are next published.

softwarefactory-project-zuul bot pushed a commit that referenced this pull request Sep 22, 2022
@patchback
ec2_instance retry on InsuffienctInstanceCapacity (#1039) (#1040)
c848983 
[PR #1039/0374961a backport][stable-4] `ec2_instance` retry on `InsuffienctInstanceCapacity `

This is a backport of PR #1039 as merged into main (0374961).
SUMMARY

Closes #1038
Expands the use of the AWSRetry decorator to include InsuffienctInstanceCapacity errors.

ISSUE TYPE


Feature Pull Request

COMPONENT NAME

ec2_instance

Reviewed-by: Jill R <None>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tremble tremble tremble approved these changes

Assignees
No one assigned
Labels
backport-4 PR should be backported to the stable-4 branch community_review feature This issue/PR relates to a feature request mergeit Merge the PR (SoftwareFactory) module module plugins plugin (any type) small_patch Hopefully easy to review
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Use AWSRetry on ec2 launch failures due to lack of capacity (InsuffienctInstanceCapacity)
3 participants
@briantist @ansibullbot @tremble