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 to elb / target group info modules #1113

Merged
merged 3 commits into from
May 17, 2022

Conversation

marknet15
Copy link
Contributor

SUMMARY

Currently there is backoff retries applied with 10 attempts overal, but due to the pagination its defaulting back to 4

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

elb_application_lb_info
elb_target_group_info

ADDITIONAL INFORMATION

@ansibullbot
Copy link

@ansibullbot ansibullbot added bug This issue/PR relates to a bug community_review module module needs_triage plugins plugin (any type) small_patch Hopefully easy to review labels May 3, 2022
@markuman markuman added the backport-3 PR should be backported to the stable-3 branch label May 5, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

ansible-galaxy-importer FAILURE in 4m 21s (non-voting)
✔️ build-ansible-collection SUCCESS in 5m 03s
ansible-test-sanity-docker-devel FAILURE in 11m 41s (non-voting)
ansible-test-sanity-docker-milestone FAILURE in 10m 13s
✔️ ansible-test-sanity-docker-stable-2.9 SUCCESS in 12m 32s
✔️ ansible-test-sanity-docker-stable-2.11 SUCCESS in 13m 38s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 11m 37s
✔️ ansible-test-units-community-aws-python38 SUCCESS in 7m 48s
✔️ ansible-test-splitter SUCCESS in 3m 04s
✔️ integration-community.aws-1 SUCCESS in 10m 08s
✔️ integration-community.aws-2 SUCCESS in 23m 14s
⚠️ 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

@jatorcasso
Copy link
Contributor

recheck

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

ansible-galaxy-importer FAILURE in 4m 34s (non-voting)
✔️ build-ansible-collection SUCCESS in 5m 24s
✔️ ansible-test-sanity-docker-devel SUCCESS in 10m 01s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 10m 22s
✔️ ansible-test-sanity-docker-stable-2.9 SUCCESS in 14m 29s
✔️ ansible-test-sanity-docker-stable-2.11 SUCCESS in 10m 36s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 11m 01s
✔️ ansible-test-units-community-aws-python38 SUCCESS in 5m 59s
✔️ ansible-test-splitter SUCCESS in 3m 21s
✔️ integration-community.aws-1 SUCCESS in 7m 36s
✔️ integration-community.aws-2 SUCCESS in 25m 04s
⚠️ 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

@markuman markuman added the mergeit Merge the PR (SoftwareFactory) label May 10, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (gate pipeline).

ansible-galaxy-importer FAILURE in 4m 24s (non-voting)
✔️ build-ansible-collection SUCCESS in 4m 35s
✔️ ansible-test-sanity-docker-devel SUCCESS in 12m 22s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 9m 33s
✔️ ansible-test-sanity-docker-stable-2.9 SUCCESS in 11m 17s
✔️ ansible-test-sanity-docker-stable-2.11 SUCCESS in 10m 05s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 9m 33s
✔️ ansible-test-units-community-aws-python38 SUCCESS in 5m 54s
✔️ ansible-test-splitter SUCCESS in 2m 56s
✔️ integration-community.aws-1 SUCCESS in 9m 29s
✔️ integration-community.aws-2 SUCCESS in 23m 03s
⚠️ 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

@softwarefactory-project-zuul
Copy link
Contributor

Pull request merge failed: Resource not accessible by integration, You may need to manually rebase your PR and retry.

@markuman
Copy link
Member

@marknet15 can you rebase the PR?

@tremble tremble merged commit 8794784 into ansible-collections:main May 17, 2022
@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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-3 PR should be backported to the stable-3 branch bug This issue/PR relates to a bug community_review mergeit Merge the PR (SoftwareFactory) module module plugins plugin (any type) small_patch Hopefully easy to review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants