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

[PR #1579/cea2ab5c backport][stable-4] Fix broken ec2_vpc_vgw tests #1580

Conversation

patchback[bot]
Copy link

@patchback patchback bot commented Nov 2, 2022

This is a backport of PR #1579 as merged into main (cea2ab5).

SUMMARY

The APIs are no longer consistently returning deleted attachments. Accept both the attachment being listed but marked 'deleted', and not being listed.

Amazon's APIs used to be very slow to update this, making waiting for it a bad idea.

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

ec2_vpc_vgw

ADDITIONAL INFORMATION

Fix broken ec2_vpc_vgw tests

SUMMARY
The APIs are no longer consistently returning deleted attachments.  Accept both the attachment being listed but marked 'deleted', and not being listed.
Amazon's APIs used to be very slow to update this, making waiting for it a bad idea.
ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME
ec2_vpc_vgw
ADDITIONAL INFORMATION

Reviewed-by: Markus Bergholz <[email protected]>
(cherry picked from commit cea2ab5)
@patchback patchback bot mentioned this pull request Nov 2, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

✔️ ansible-galaxy-importer SUCCESS in 4m 07s
✔️ build-ansible-collection SUCCESS in 5m 49s
ansible-test-sanity-docker-devel FAILURE in 9m 23s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 9m 33s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 9m 29s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 9m 10s
✔️ ansible-test-sanity-docker-stable-2.14 SUCCESS in 9m 34s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 6m 10s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 6m 19s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 7m 18s
✔️ ansible-test-splitter SUCCESS in 2m 53s
✔️ integration-community.aws-1 SUCCESS in 8m 52s
⚠️ 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
⚠️ integration-community.aws-19 SKIPPED
⚠️ integration-community.aws-20 SKIPPED
⚠️ integration-community.aws-21 SKIPPED
⚠️ integration-community.aws-22 SKIPPED
✔️ ansible-test-changelog SUCCESS in 2m 34s

@tremble tremble added the mergeit Merge the PR (SoftwareFactory) label Nov 2, 2022
@ansibullbot
Copy link

@ansibullbot ansibullbot added bug This issue/PR relates to a bug community_review integration tests/integration new_contributor Help guide this first time contributor plugins plugin (any type) tests tests labels Nov 2, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (gate pipeline).

✔️ ansible-galaxy-importer SUCCESS in 4m 00s
✔️ build-ansible-collection SUCCESS in 5m 06s
ansible-test-sanity-docker-devel FAILURE in 13m 24s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 11m 06s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 9m 41s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 10m 12s
✔️ ansible-test-sanity-docker-stable-2.14 SUCCESS in 9m 24s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 6m 55s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 6m 04s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 6m 11s
✔️ ansible-test-splitter SUCCESS in 2m 51s
✔️ integration-community.aws-1 SUCCESS in 8m 43s
⚠️ 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
⚠️ integration-community.aws-19 SKIPPED
⚠️ integration-community.aws-20 SKIPPED
⚠️ integration-community.aws-21 SKIPPED
⚠️ integration-community.aws-22 SKIPPED
✔️ ansible-test-changelog SUCCESS in 2m 37s

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit 2915436 into stable-4 Nov 2, 2022
@softwarefactory-project-zuul softwarefactory-project-zuul bot deleted the patchback/backports/stable-4/cea2ab5cf7a8f778fe2fdf2bfa55fd3e0fba68de/pr-1579 branch November 2, 2022 10:50
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 mergeit Merge the PR (SoftwareFactory) new_contributor Help guide this first time contributor plugins plugin (any type) tests tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants