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

Fixup ec2_eip tests - work around ec2_instance flakes #333

Merged
merged 3 commits into from
Dec 14, 2020

Conversation

tremble
Copy link
Contributor

@tremble tremble commented Dec 14, 2020

SUMMARY

We're seeing some ec2_instance flakes which appear to be caused by filter based searches not consistently returning. Work around these for now to unblock #328

ISSUE TYPE
  • Tests Pull Request
COMPONENT NAME

ec2_eip

ADDITIONAL INFORMATION

Copy link
Contributor

@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.

thanks @tremble

@jillr jillr merged commit 6c0273b into ansible-collections:main Dec 14, 2020
@jillr jillr mentioned this pull request Dec 14, 2020
@tremble tremble deleted the ec2_eip/ec2_instance-ids branch February 11, 2021 09:25
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request Jul 19, 2021
…tions#333)

* Explicitly pass the subnet the instance should live on - try to avoid https://github.com/ansible-collections/community.aws/issues/329

* Make sure we delete the Instance (and free the EIP) before we try to drop the IGW

* Use IDs when cleaning up EC2 instances
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request Jul 19, 2021
…tions#333)

* Explicitly pass the subnet the instance should live on - try to avoid https://github.com/ansible-collections/community.aws/issues/329

* Make sure we delete the Instance (and free the EIP) before we try to drop the IGW

* Use IDs when cleaning up EC2 instances
danielcotton pushed a commit to danielcotton/community.aws that referenced this pull request Nov 23, 2021
…tions#333)

* Explicitly pass the subnet the instance should live on - try to avoid https://github.com/ansible-collections/community.aws/issues/329

* Make sure we delete the Instance (and free the EIP) before we try to drop the IGW

* Use IDs when cleaning up EC2 instances
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
…tions#333)

* Explicitly pass the subnet the instance should live on - try to avoid https://github.com/ansible-collections/community.aws/issues/329

* Make sure we delete the Instance (and free the EIP) before we try to drop the IGW

* Use IDs when cleaning up EC2 instances

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections@6c0273b
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