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

Move exceptions into their own python module #1246

Conversation

tremble
Copy link
Contributor

@tremble tremble commented Nov 4, 2022

SUMMARY

Move base/core exceptions into their own python module

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

module_utils

ADDITIONAL INFORMATION

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

✔️ ansible-galaxy-importer SUCCESS in 4m 17s
✔️ build-ansible-collection SUCCESS in 4m 57s
✔️ ansible-test-sanity-aws-ansible-python38 SUCCESS in 11m 54s
✔️ ansible-test-sanity-aws-ansible-2.12-python38 SUCCESS in 9m 43s
✔️ ansible-test-sanity-aws-ansible-2.13-python38 SUCCESS in 9m 17s
✔️ ansible-test-sanity-aws-ansible-2.14 SUCCESS in 10m 33s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 7m 17s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 6m 33s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 6m 08s
✔️ cloud-tox-py3 SUCCESS in 3m 02s
✔️ ansible-test-splitter SUCCESS in 2m 56s
✔️ integration-amazon.aws-1 SUCCESS in 27m 53s
✔️ integration-amazon.aws-2 SUCCESS in 11m 37s
✔️ integration-amazon.aws-3 SUCCESS in 7m 58s
✔️ integration-amazon.aws-4 SUCCESS in 18m 20s
✔️ integration-amazon.aws-5 SUCCESS in 23m 31s
✔️ integration-amazon.aws-6 SUCCESS in 33m 35s
✔️ integration-amazon.aws-7 SUCCESS in 21m 05s
✔️ integration-amazon.aws-8 SUCCESS in 33m 36s
✔️ integration-amazon.aws-9 SUCCESS in 36m 32s
✔️ integration-amazon.aws-10 SUCCESS in 41m 25s
✔️ integration-amazon.aws-11 SUCCESS in 34m 56s
✔️ integration-amazon.aws-12 SUCCESS in 39m 01s
✔️ integration-amazon.aws-13 SUCCESS in 51m 20s
✔️ integration-amazon.aws-14 SUCCESS in 34m 11s
✔️ integration-amazon.aws-15 SUCCESS in 44m 56s
✔️ integration-amazon.aws-16 SUCCESS in 38m 49s
✔️ integration-amazon.aws-17 SUCCESS in 36m 50s
✔️ integration-amazon.aws-18 SUCCESS in 28m 23s
✔️ integration-amazon.aws-19 SUCCESS in 11m 37s
⚠️ integration-amazon.aws-20 SKIPPED
⚠️ integration-amazon.aws-21 SKIPPED
⚠️ integration-amazon.aws-22 SKIPPED
✔️ integration-community.aws-1 SUCCESS in 34m 30s
✔️ integration-community.aws-2 SUCCESS in 38m 15s
✔️ integration-community.aws-3 SUCCESS in 43m 22s
✔️ integration-community.aws-4 SUCCESS in 11m 05s
✔️ integration-community.aws-5 SUCCESS in 33m 59s
✔️ integration-community.aws-6 SUCCESS in 52m 41s
✔️ integration-community.aws-7 SUCCESS in 57m 05s
✔️ integration-community.aws-8 SUCCESS in 34m 07s
✔️ integration-community.aws-9 SUCCESS in 47m 04s
✔️ integration-community.aws-10 SUCCESS in 19m 13s
✔️ integration-community.aws-11 SUCCESS in 21m 48s
⚠️ 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

@tremble tremble force-pushed the refacter/exceptions branch from 4c8cb9a to 8b94886 Compare November 4, 2022 09:43
@tremble tremble marked this pull request as ready for review November 4, 2022 09:43
@tremble tremble requested a review from alinabuzachis November 4, 2022 09:43
@ansibullbot ansibullbot added the tests tests label Nov 4, 2022
@tremble tremble mentioned this pull request Nov 4, 2022
@tremble tremble added the mergeit Merge the PR (SoftwareFactory) label Nov 4, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build failed (gate pipeline). For information on how to proceed, see
http://docs.openstack.org/infra/manual/developers.html#automated-testing

✔️ ansible-galaxy-importer SUCCESS in 3m 56s
✔️ build-ansible-collection SUCCESS in 5m 46s
✔️ ansible-test-sanity-aws-ansible-python38 SUCCESS in 10m 36s
✔️ ansible-test-sanity-aws-ansible-2.12-python38 SUCCESS in 9m 37s
✔️ ansible-test-sanity-aws-ansible-2.13-python38 SUCCESS in 8m 58s
✔️ ansible-test-sanity-aws-ansible-2.14 SUCCESS in 9m 00s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 7m 08s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 5m 49s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 5m 49s
✔️ cloud-tox-py3 SUCCESS in 3m 07s
✔️ ansible-test-splitter SUCCESS in 2m 53s
✔️ integration-amazon.aws-1 SUCCESS in 29m 46s
✔️ integration-amazon.aws-2 SUCCESS in 11m 52s
✔️ integration-amazon.aws-3 SUCCESS in 9m 02s
✔️ integration-amazon.aws-4 SUCCESS in 20m 00s
✔️ integration-amazon.aws-5 SUCCESS in 20m 50s
✔️ integration-amazon.aws-6 SUCCESS in 43m 20s
✔️ integration-amazon.aws-7 SUCCESS in 16m 56s
✔️ integration-amazon.aws-8 SUCCESS in 35m 23s
✔️ integration-amazon.aws-9 SUCCESS in 33m 54s
✔️ integration-amazon.aws-10 SUCCESS in 39m 23s
✔️ integration-amazon.aws-11 SUCCESS in 33m 50s
integration-amazon.aws-12 FAILURE in 48m 02s
✔️ integration-amazon.aws-13 SUCCESS in 44m 46s
✔️ integration-amazon.aws-14 SUCCESS in 28m 48s
✔️ integration-amazon.aws-15 SUCCESS in 40m 37s
✔️ integration-amazon.aws-16 SUCCESS in 46m 36s
✔️ integration-amazon.aws-17 SUCCESS in 42m 27s
✔️ integration-amazon.aws-18 SUCCESS in 23m 55s
✔️ integration-amazon.aws-19 SUCCESS in 11m 35s
⚠️ integration-amazon.aws-20 SKIPPED
⚠️ integration-amazon.aws-21 SKIPPED
⚠️ integration-amazon.aws-22 SKIPPED
✔️ integration-community.aws-1 SUCCESS in 34m 17s
✔️ integration-community.aws-2 SUCCESS in 26m 21s
✔️ integration-community.aws-3 SUCCESS in 50m 16s
✔️ integration-community.aws-4 SUCCESS in 32m 41s
✔️ integration-community.aws-5 SUCCESS in 33m 25s
✔️ integration-community.aws-6 SUCCESS in 29m 45s
✔️ integration-community.aws-7 SUCCESS in 55m 14s
✔️ integration-community.aws-8 SUCCESS in 33m 30s
✔️ integration-community.aws-9 SUCCESS in 30m 36s
✔️ integration-community.aws-10 SUCCESS in 18m 21s
✔️ integration-community.aws-11 SUCCESS in 20m 10s
⚠️ 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 15s

@tremble
Copy link
Contributor Author

tremble commented Nov 4, 2022

regate

@softwarefactory-project-zuul
Copy link
Contributor

Build failed (gate pipeline). For information on how to proceed, see
http://docs.openstack.org/infra/manual/developers.html#automated-testing

✔️ ansible-galaxy-importer SUCCESS in 4m 07s
✔️ build-ansible-collection SUCCESS in 4m 52s
✔️ ansible-test-sanity-aws-ansible-python38 SUCCESS in 8m 34s
✔️ ansible-test-sanity-aws-ansible-2.12-python38 SUCCESS in 8m 52s
✔️ ansible-test-sanity-aws-ansible-2.13-python38 SUCCESS in 9m 28s
✔️ ansible-test-sanity-aws-ansible-2.14 SUCCESS in 12m 01s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 6m 05s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 6m 47s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 5m 59s
✔️ cloud-tox-py3 SUCCESS in 3m 07s
✔️ ansible-test-splitter SUCCESS in 2m 50s
✔️ integration-amazon.aws-1 SUCCESS in 30m 02s
✔️ integration-amazon.aws-2 SUCCESS in 13m 28s
✔️ integration-amazon.aws-3 SUCCESS in 10m 17s
✔️ integration-amazon.aws-4 SUCCESS in 27m 21s
✔️ integration-amazon.aws-5 SUCCESS in 21m 12s
✔️ integration-amazon.aws-6 SUCCESS in 40m 34s
✔️ integration-amazon.aws-7 SUCCESS in 19m 48s
✔️ integration-amazon.aws-8 SUCCESS in 37m 58s
✔️ integration-amazon.aws-9 SUCCESS in 40m 14s
✔️ integration-amazon.aws-10 SUCCESS in 40m 45s
✔️ integration-amazon.aws-11 SUCCESS in 28m 23s
✔️ integration-amazon.aws-12 SUCCESS in 40m 04s
✔️ integration-amazon.aws-13 SUCCESS in 44m 04s
✔️ integration-amazon.aws-14 SUCCESS in 31m 05s
✔️ integration-amazon.aws-15 SUCCESS in 42m 47s
integration-amazon.aws-16 FAILURE in 27m 25s
integration-amazon.aws-17 FAILURE in 47m 09s
✔️ integration-amazon.aws-18 SUCCESS in 30m 05s
✔️ integration-amazon.aws-19 SUCCESS in 15m 57s
⚠️ integration-amazon.aws-20 SKIPPED
⚠️ integration-amazon.aws-21 SKIPPED
⚠️ integration-amazon.aws-22 SKIPPED
✔️ integration-community.aws-1 SUCCESS in 31m 07s
✔️ integration-community.aws-2 SUCCESS in 22m 40s
✔️ integration-community.aws-3 SUCCESS in 58m 35s
integration-community.aws-4 TIMED_OUT in 1h 00m 31s
✔️ integration-community.aws-5 SUCCESS in 32m 13s
✔️ integration-community.aws-6 SUCCESS in 32m 21s
✔️ integration-community.aws-7 SUCCESS in 55m 14s
✔️ integration-community.aws-8 SUCCESS in 36m 08s
✔️ integration-community.aws-9 SUCCESS in 33m 03s
✔️ integration-community.aws-10 SUCCESS in 20m 24s
✔️ integration-community.aws-11 SUCCESS in 22m 33s
⚠️ 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 18s

@tremble
Copy link
Contributor Author

tremble commented Nov 4, 2022

regate

@softwarefactory-project-zuul
Copy link
Contributor

Build failed (gate pipeline). For information on how to proceed, see
http://docs.openstack.org/infra/manual/developers.html#automated-testing

✔️ ansible-galaxy-importer SUCCESS in 6m 01s
✔️ build-ansible-collection SUCCESS in 4m 59s
✔️ ansible-test-sanity-aws-ansible-python38 SUCCESS in 11m 55s
✔️ ansible-test-sanity-aws-ansible-2.12-python38 SUCCESS in 10m 07s
✔️ ansible-test-sanity-aws-ansible-2.13-python38 SUCCESS in 9m 50s
✔️ ansible-test-sanity-aws-ansible-2.14 SUCCESS in 9m 02s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 6m 40s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 7m 33s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 7m 31s
✔️ cloud-tox-py3 SUCCESS in 3m 11s
✔️ ansible-test-splitter SUCCESS in 3m 04s
✔️ integration-amazon.aws-1 SUCCESS in 52m 58s
✔️ integration-amazon.aws-2 SUCCESS in 12m 03s
✔️ integration-amazon.aws-3 SUCCESS in 7m 28s
✔️ integration-amazon.aws-4 SUCCESS in 18m 35s
✔️ integration-amazon.aws-5 SUCCESS in 23m 12s
✔️ integration-amazon.aws-6 SUCCESS in 33m 08s
✔️ integration-amazon.aws-7 SUCCESS in 19m 03s
✔️ integration-amazon.aws-8 SUCCESS in 34m 38s
✔️ integration-amazon.aws-9 SUCCESS in 34m 24s
✔️ integration-amazon.aws-10 SUCCESS in 36m 54s
✔️ integration-amazon.aws-11 SUCCESS in 26m 40s
✔️ integration-amazon.aws-12 SUCCESS in 38m 41s
✔️ integration-amazon.aws-13 SUCCESS in 46m 03s
✔️ integration-amazon.aws-14 SUCCESS in 29m 36s
✔️ integration-amazon.aws-15 SUCCESS in 44m 02s
✔️ integration-amazon.aws-16 SUCCESS in 48m 23s
integration-amazon.aws-17 FAILURE in 43m 36s
integration-amazon.aws-18 FAILURE in 23m 56s
✔️ integration-amazon.aws-19 SUCCESS in 11m 22s
⚠️ integration-amazon.aws-20 SKIPPED
⚠️ integration-amazon.aws-21 SKIPPED
⚠️ integration-amazon.aws-22 SKIPPED
✔️ integration-community.aws-1 SUCCESS in 36m 01s
✔️ integration-community.aws-2 SUCCESS in 21m 38s
✔️ integration-community.aws-3 SUCCESS in 38m 46s
✔️ integration-community.aws-4 SUCCESS in 11m 21s
✔️ integration-community.aws-5 SUCCESS in 27m 22s
✔️ integration-community.aws-6 SUCCESS in 30m 53s
integration-community.aws-7 TIMED_OUT in 1h 00m 44s
✔️ integration-community.aws-8 SUCCESS in 34m 41s
✔️ integration-community.aws-9 SUCCESS in 32m 11s
✔️ integration-community.aws-10 SUCCESS in 18m 59s
✔️ integration-community.aws-11 SUCCESS in 20m 49s
⚠️ 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 24s

@tremble
Copy link
Contributor Author

tremble commented Nov 4, 2022

regate

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (gate pipeline).

✔️ ansible-galaxy-importer SUCCESS in 4m 51s
✔️ build-ansible-collection SUCCESS in 5m 31s
✔️ ansible-test-sanity-aws-ansible-python38 SUCCESS in 8m 59s
✔️ ansible-test-sanity-aws-ansible-2.12-python38 SUCCESS in 12m 39s
✔️ ansible-test-sanity-aws-ansible-2.13-python38 SUCCESS in 9m 32s
✔️ ansible-test-sanity-aws-ansible-2.14 SUCCESS in 9m 05s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 6m 31s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 6m 32s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 6m 08s
✔️ cloud-tox-py3 SUCCESS in 3m 08s
✔️ ansible-test-splitter SUCCESS in 2m 41s
✔️ integration-amazon.aws-1 SUCCESS in 28m 34s
✔️ integration-amazon.aws-2 SUCCESS in 12m 01s
✔️ integration-amazon.aws-3 SUCCESS in 9m 17s
✔️ integration-amazon.aws-4 SUCCESS in 24m 40s
✔️ integration-amazon.aws-5 SUCCESS in 27m 18s
✔️ integration-amazon.aws-6 SUCCESS in 28m 51s
✔️ integration-amazon.aws-7 SUCCESS in 17m 34s
✔️ integration-amazon.aws-8 SUCCESS in 35m 16s
✔️ integration-amazon.aws-9 SUCCESS in 35m 18s
✔️ integration-amazon.aws-10 SUCCESS in 41m 35s
✔️ integration-amazon.aws-11 SUCCESS in 28m 09s
✔️ integration-amazon.aws-12 SUCCESS in 38m 40s
✔️ integration-amazon.aws-13 SUCCESS in 46m 36s
✔️ integration-amazon.aws-14 SUCCESS in 25m 53s
✔️ integration-amazon.aws-15 SUCCESS in 40m 39s
✔️ integration-amazon.aws-16 SUCCESS in 40m 41s
✔️ integration-amazon.aws-17 SUCCESS in 40m 02s
✔️ integration-amazon.aws-18 SUCCESS in 28m 48s
✔️ integration-amazon.aws-19 SUCCESS in 12m 55s
⚠️ integration-amazon.aws-20 SKIPPED
⚠️ integration-amazon.aws-21 SKIPPED
⚠️ integration-amazon.aws-22 SKIPPED
✔️ integration-community.aws-1 SUCCESS in 59m 21s
✔️ integration-community.aws-2 SUCCESS in 23m 15s
✔️ integration-community.aws-3 SUCCESS in 47m 40s
✔️ integration-community.aws-4 SUCCESS in 12m 37s
✔️ integration-community.aws-5 SUCCESS in 35m 09s
✔️ integration-community.aws-6 SUCCESS in 26m 07s
✔️ integration-community.aws-7 SUCCESS in 56m 16s
✔️ integration-community.aws-8 SUCCESS in 36m 32s
✔️ integration-community.aws-9 SUCCESS in 30m 29s
✔️ integration-community.aws-10 SUCCESS in 19m 06s
✔️ integration-community.aws-11 SUCCESS in 19m 52s
⚠️ 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 20s

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit 8ff6340 into ansible-collections:main Nov 4, 2022
softwarefactory-project-zuul bot pushed a commit that referenced this pull request Nov 5, 2022
…e reuse (#1227)

Move botocore version validation into module_utils.botocore for future reuse

Depends-On: #1246
SUMMARY
In preparation for #1225 move the SDK version validation pieces over into module_utils.botocore
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
plugins/module_utils/botocore.py
plugins/module_utils/core.py
plugins/module_utils/exceptions.py
plugins/module_utils/modules.py
ADDITIONAL INFORMATION

Reviewed-by: Alina Buzachis <None>
Reviewed-by: Mark Chappell <None>
softwarefactory-project-zuul bot pushed a commit that referenced this pull request Nov 5, 2022
Refacter get_aws_connection_info / get_aws_region

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

Reviewed-by: Alina Buzachis <None>
@tremble tremble deleted the refacter/exceptions 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 new_plugin New plugin plugins plugin (any type) tests tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants