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

Refacter get_aws_connection_info / get_aws_region #1231

Conversation

tremble
Copy link
Contributor

@tremble tremble commented Nov 1, 2022

Depends-On: #1246

SUMMARY

Splits up the logic for get_aws_region and get_aws_connection_info so that it can be reused by non-module plugins.

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

plugins/module_utils/botocore.py

ADDITIONAL INFORMATION

@softwarefactory-project-zuul

This comment was marked as resolved.

@tremble tremble force-pushed the refacter/boto3_connections branch from a228804 to 28eae36 Compare November 1, 2022 12:35
@softwarefactory-project-zuul

This comment was marked as resolved.

@ansibullbot ansibullbot added community_review feature This issue/PR relates to a feature request module_utils module_utils needs_triage new_plugin New plugin plugins plugin (any type) labels Nov 1, 2022
@tremble tremble mentioned this pull request Nov 1, 2022
@goneri
Copy link
Member

goneri commented Nov 1, 2022

recheck

@softwarefactory-project-zuul

This comment was marked as outdated.

@tremble tremble changed the title Refacter connection_info et al Refacter get_aws_connection_info / get_aws_region Nov 2, 2022
@tremble tremble force-pushed the refacter/boto3_connections branch from 28eae36 to bc2ee31 Compare November 2, 2022 20:14
@ansibullbot ansibullbot added the tests tests label Nov 2, 2022
@softwarefactory-project-zuul

This comment was marked as outdated.

@tremble tremble force-pushed the refacter/boto3_connections branch 2 times, most recently from 79555eb to 027e5be Compare November 3, 2022 10:22
@tremble tremble marked this pull request as ready for review November 3, 2022 10:38
@tremble tremble requested a review from goneri November 3, 2022 10:38
@tremble tremble force-pushed the refacter/boto3_connections branch from 027e5be to e456b4e Compare November 3, 2022 10:41
@softwarefactory-project-zuul

This comment was marked as outdated.

@tremble tremble requested a review from alinabuzachis November 3, 2022 16:50
@tremble tremble force-pushed the refacter/boto3_connections branch from e456b4e to 2108a68 Compare November 3, 2022 17:26
@softwarefactory-project-zuul

This comment was marked as outdated.

@tremble tremble force-pushed the refacter/boto3_connections branch from 2108a68 to cfdd170 Compare November 3, 2022 21:07
@softwarefactory-project-zuul

This comment was marked as outdated.

@softwarefactory-project-zuul

This comment was marked as outdated.

@tremble tremble force-pushed the refacter/boto3_connections branch from cfdd170 to 28f7772 Compare November 4, 2022 10:44
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

✔️ ansible-galaxy-importer SUCCESS in 4m 09s
✔️ build-ansible-collection SUCCESS in 5m 33s
✔️ ansible-test-sanity-aws-ansible-python38 SUCCESS in 12m 15s
✔️ ansible-test-sanity-aws-ansible-2.12-python38 SUCCESS in 9m 30s
✔️ ansible-test-sanity-aws-ansible-2.13-python38 SUCCESS in 9m 16s
✔️ ansible-test-sanity-aws-ansible-2.14 SUCCESS in 8m 40s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 6m 38s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 6m 37s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 6m 19s
✔️ cloud-tox-py3 SUCCESS in 3m 03s
✔️ ansible-test-splitter SUCCESS in 2m 54s
✔️ integration-amazon.aws-1 SUCCESS in 9m 52s
⚠️ integration-amazon.aws-2 SKIPPED
⚠️ integration-amazon.aws-3 SKIPPED
⚠️ 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-amazon.aws-19 SKIPPED
⚠️ integration-amazon.aws-20 SKIPPED
⚠️ integration-amazon.aws-21 SKIPPED
⚠️ integration-amazon.aws-22 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
⚠️ 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 13s

@tremble tremble added the mergeit Merge the PR (SoftwareFactory) label Nov 5, 2022
@ansibullbot ansibullbot removed the new_plugin New plugin label Nov 5, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (gate pipeline).

✔️ ansible-galaxy-importer SUCCESS in 4m 07s
✔️ build-ansible-collection SUCCESS in 5m 25s
✔️ ansible-test-sanity-aws-ansible-python38 SUCCESS in 9m 29s
✔️ ansible-test-sanity-aws-ansible-2.12-python38 SUCCESS in 10m 11s
✔️ ansible-test-sanity-aws-ansible-2.13-python38 SUCCESS in 10m 57s
✔️ ansible-test-sanity-aws-ansible-2.14 SUCCESS in 8m 48s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 5m 59s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 6m 17s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 5m 49s
✔️ cloud-tox-py3 SUCCESS in 3m 01s
✔️ ansible-test-splitter SUCCESS in 3m 02s
✔️ integration-amazon.aws-1 SUCCESS in 10m 00s
⚠️ integration-amazon.aws-2 SKIPPED
⚠️ integration-amazon.aws-3 SKIPPED
⚠️ 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-amazon.aws-19 SKIPPED
⚠️ integration-amazon.aws-20 SKIPPED
⚠️ integration-amazon.aws-21 SKIPPED
⚠️ integration-amazon.aws-22 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
⚠️ 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 19s
✔️ noop SUCCESS in 0s

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit 25e8e35 into ansible-collections:main Nov 5, 2022
@tremble tremble deleted the refacter/boto3_connections branch February 15, 2023 09:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community_review feature This issue/PR relates to a feature request mergeit Merge the PR (SoftwareFactory) module_utils module_utils needs_triage plugins plugin (any type) tests tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants