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

improve ec2_vpc_nat_gateway stability #427

Merged

Conversation

tremble
Copy link
Contributor

@tremble tremble commented Feb 20, 2021

SUMMARY

We're seeing a lot of failures with ec2_vpc_nat_gateway

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

ec2_vpc_nat_gateway

ADDITIONAL INFORMATION

@ansibullbot
Copy link

@ansibullbot ansibullbot added WIP Work in progress bug This issue/PR relates to a bug module module needs_triage owner_pr PR created by owner/maintainer plugins plugin (any type) small_patch Hopefully easy to review labels Feb 20, 2021
@tremble
Copy link
Contributor Author

tremble commented Feb 20, 2021

Looking at the calls invoked I suspect that the 'search' for gateways is flakey:

TASK [ec2_vpc_nat_gateway : Update with CamelCase tags] ************************
fatal: [testhost]: FAILED! => {"changed": true, "msg": "Wait time out reached, while waiting for results", "resource_actions": ["ec2:CreateTags", "ec2:DescribeNatGateways", "ec2:DescribeTags", "ec2:CreateNatGateway"], "success": false}

Note: ec2:CreateNatGateway was invoked!

@ansibullbot ansibullbot removed the small_patch Hopefully easy to review label Feb 20, 2021
@tremble tremble force-pushed the stability/ec2_vpc_nat_gateway branch from b98e4d9 to d34d850 Compare February 20, 2021 19:52
@tremble tremble linked an issue Feb 21, 2021 that may be closed by this pull request
@tremble tremble force-pushed the stability/ec2_vpc_nat_gateway branch from 2593277 to 42c70dc Compare February 21, 2021 10:45
@ansibullbot ansibullbot added integration tests/integration tests tests and removed owner_pr PR created by owner/maintainer labels Feb 21, 2021
@tremble tremble force-pushed the stability/ec2_vpc_nat_gateway branch from 42c70dc to e19eca6 Compare February 21, 2021 14:36
@tremble tremble changed the title [WIP] improve ec2_vpc_nat_gateway stability improve ec2_vpc_nat_gateway stability Feb 22, 2021
@ansibullbot ansibullbot added community_review and removed WIP Work in progress labels Feb 22, 2021
@tremble
Copy link
Contributor Author

tremble commented Feb 23, 2021

Reran the tests again for good measure, still coming back all green!

I have a +1 from @alinabuzachis so I'll get this merged and so the other PRs can be rebased.

@tremble tremble merged commit 38be7f9 into ansible-collections:main Feb 23, 2021
@tremble tremble deleted the stability/ec2_vpc_nat_gateway branch June 29, 2021 12:55
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request Jul 16, 2021
)

* attempt to add more information when failures occur
* Use pagination and add more retry wrappers
* Mark ec2_vpc_nat_gateway stable again.  Using paginator and retries seems to have fixed things
* Add changelog

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections@38be7f9
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request Jul 19, 2021
* attempt to add more information when failures occur
* Use pagination and add more retry wrappers
* Mark ec2_vpc_nat_gateway stable again.  Using paginator and retries seems to have fixed things
* Add changelog
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request Jul 19, 2021
* attempt to add more information when failures occur
* Use pagination and add more retry wrappers
* Mark ec2_vpc_nat_gateway stable again.  Using paginator and retries seems to have fixed things
* Add changelog
danielcotton pushed a commit to danielcotton/community.aws that referenced this pull request Nov 23, 2021
* attempt to add more information when failures occur
* Use pagination and add more retry wrappers
* Mark ec2_vpc_nat_gateway stable again.  Using paginator and retries seems to have fixed things
* Add changelog
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request May 25, 2022
ec2_key : Fix tests

SUMMARY
The Zuul nodes don't have OpenSSL installed on them, this breaks the generation of the fingerprints
ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME
ec2_key
ADDITIONAL INFORMATION
https://dashboard.zuul.ansible.com/t/ansible/build/d79dcec2e3024558800bba5fd6917505/log/job-output.txt
"stderr": "/bin/sh: line 1: openssl: command not found"

Depends-on: ansible-collections#460

Reviewed-by: Alina Buzachis <None>
Reviewed-by: None <None>
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request May 25, 2022
* attempt to add more information when failures occur
* Use pagination and add more retry wrappers
* Mark ec2_vpc_nat_gateway stable again.  Using paginator and retries seems to have fixed things
* Add changelog

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections@38be7f9
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue/PR relates to a bug community_review integration tests/integration module module needs_triage plugins plugin (any type) tests tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ec2_vpc_nat_gateway - integration tests unstable
3 participants