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

Refactor module_utils/cloudfront_facts and add unit tests #1265

Conversation

abikouo
Copy link
Contributor

@abikouo abikouo commented Nov 14, 2022

Depends-On: ansible/ansible-zuul-jobs#1695

SUMMARY

refactor module_utils/cloudfront_facts.py and unit tests

ISSUE TYPE
  • Feature Pull Request

@softwarefactory-project-zuul
Copy link
Contributor

Merge Failed.

This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset.
Warning:
Error merging github.com/ansible-collections/amazon.aws for 1265,d6ef31a1218895d013f9956192359107b6ea2647

@abikouo abikouo force-pushed the cloudfront_facts_unit_tests branch from d6ef31a to d894aaa Compare November 14, 2022 10:28
@ansibullbot ansibullbot added community_review feature This issue/PR relates to a feature request module_utils module_utils needs_triage plugins plugin (any type) tests tests labels Nov 14, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

✔️ ansible-galaxy-importer SUCCESS in 4m 00s
✔️ build-ansible-collection SUCCESS in 5m 38s
ansible-test-sanity-aws-ansible-python38 FAILURE in 8m 31s (non-voting)
ansible-test-sanity-aws-ansible-2.12-python38 FAILURE in 9m 25s
ansible-test-sanity-aws-ansible-2.13-python38 FAILURE in 8m 52s
ansible-test-sanity-aws-ansible-2.14 FAILURE in 8m 56s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 6m 30s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 7m 33s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 6m 08s
✔️ cloud-tox-py3 SUCCESS in 3m 09s
✔️ ansible-test-changelog SUCCESS in 2m 06s
✔️ ansible-test-splitter SUCCESS in 2m 56s
integration-amazon.aws-1 FAILURE in 5m 05s
integration-amazon.aws-2 FAILURE in 6m 06s
integration-amazon.aws-3 FAILURE in 5m 20s
integration-amazon.aws-4 FAILURE in 5m 23s
integration-amazon.aws-5 FAILURE in 5m 04s
integration-amazon.aws-6 FAILURE in 5m 33s
integration-amazon.aws-7 FAILURE in 5m 24s
integration-amazon.aws-8 FAILURE in 5m 08s
integration-amazon.aws-9 FAILURE in 4m 59s
integration-amazon.aws-10 FAILURE in 4m 44s
integration-amazon.aws-11 FAILURE in 4m 59s
integration-amazon.aws-12 FAILURE in 5m 03s
integration-amazon.aws-13 FAILURE in 5m 51s
integration-amazon.aws-14 FAILURE in 6m 35s
integration-amazon.aws-15 FAILURE in 4m 42s
integration-amazon.aws-16 FAILURE in 5m 10s
integration-amazon.aws-17 FAILURE in 7m 45s
integration-amazon.aws-18 FAILURE in 5m 01s
integration-amazon.aws-19 FAILURE in 7m 00s
integration-amazon.aws-20 FAILURE in 5m 16s
integration-amazon.aws-21 FAILURE in 5m 02s
integration-amazon.aws-22 FAILURE in 5m 06s
integration-community.aws-1 FAILURE in 5m 18s
integration-community.aws-2 FAILURE in 4m 49s
integration-community.aws-3 FAILURE in 5m 02s
integration-community.aws-4 FAILURE in 8m 04s
integration-community.aws-5 FAILURE in 7m 17s
integration-community.aws-6 FAILURE in 5m 35s
integration-community.aws-7 FAILURE in 6m 17s
integration-community.aws-8 FAILURE in 4m 58s
integration-community.aws-9 FAILURE in 6m 16s
integration-community.aws-10 FAILURE in 5m 12s
integration-community.aws-11 FAILURE in 7m 17s
integration-community.aws-12 FAILURE in 5m 39s
integration-community.aws-13 FAILURE in 7m 01s
integration-community.aws-14 FAILURE in 4m 46s
integration-community.aws-15 FAILURE in 5m 05s
integration-community.aws-16 FAILURE in 5m 36s
integration-community.aws-17 FAILURE in 6m 34s
integration-community.aws-18 FAILURE in 5m 22s
integration-community.aws-19 FAILURE in 4m 52s
integration-community.aws-20 FAILURE in 5m 14s
integration-community.aws-21 FAILURE in 5m 11s
integration-community.aws-22 FAILURE in 6m 21s

@goneri
Copy link
Member

goneri commented Nov 14, 2022

recheck

@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

✔️ ansible-galaxy-importer SUCCESS in 4m 20s
✔️ build-ansible-collection SUCCESS in 5m 23s
ansible-test-sanity-aws-ansible-python38 FAILURE in 8m 51s (non-voting)
ansible-test-sanity-aws-ansible-2.12-python38 FAILURE in 9m 40s
ansible-test-sanity-aws-ansible-2.13-python38 FAILURE in 11m 19s
ansible-test-sanity-aws-ansible-2.14 FAILURE in 9m 25s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 7m 01s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 8m 03s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 7m 31s
✔️ cloud-tox-py3 SUCCESS in 3m 32s
✔️ ansible-test-changelog SUCCESS in 2m 42s
✔️ ansible-test-splitter SUCCESS in 3m 13s
integration-amazon.aws-1 FAILURE in 5m 29s
integration-amazon.aws-2 FAILURE in 5m 00s
integration-amazon.aws-3 FAILURE in 5m 12s
integration-amazon.aws-4 FAILURE in 5m 07s
integration-amazon.aws-5 FAILURE in 5m 24s
integration-amazon.aws-6 FAILURE in 4m 55s
integration-amazon.aws-7 FAILURE in 5m 02s
integration-amazon.aws-8 FAILURE in 4m 51s
integration-amazon.aws-9 FAILURE in 5m 04s
integration-amazon.aws-10 FAILURE in 4m 53s
integration-amazon.aws-11 FAILURE in 4m 56s
integration-amazon.aws-12 FAILURE in 5m 16s
integration-amazon.aws-13 FAILURE in 5m 19s
integration-amazon.aws-14 FAILURE in 5m 50s
integration-amazon.aws-15 FAILURE in 4m 47s
integration-amazon.aws-16 FAILURE in 4m 59s
integration-amazon.aws-17 FAILURE in 6m 58s
integration-amazon.aws-18 FAILURE in 5m 07s
integration-amazon.aws-19 FAILURE in 5m 23s
integration-amazon.aws-20 FAILURE in 4m 57s
integration-amazon.aws-21 FAILURE in 6m 33s
integration-amazon.aws-22 FAILURE in 4m 51s
integration-community.aws-1 FAILURE in 6m 35s
integration-community.aws-2 FAILURE in 4m 49s
integration-community.aws-3 FAILURE in 6m 22s
integration-community.aws-4 FAILURE in 6m 33s
integration-community.aws-5 FAILURE in 5m 21s
integration-community.aws-6 FAILURE in 7m 55s
integration-community.aws-7 FAILURE in 4m 48s
integration-community.aws-8 FAILURE in 4m 56s
integration-community.aws-9 FAILURE in 7m 03s
integration-community.aws-10 FAILURE in 4m 42s
integration-community.aws-11 FAILURE in 4m 44s
integration-community.aws-12 FAILURE in 5m 17s
integration-community.aws-13 FAILURE in 5m 21s
integration-community.aws-14 FAILURE in 6m 12s
integration-community.aws-15 FAILURE in 4m 59s
integration-community.aws-16 FAILURE in 5m 01s
integration-community.aws-17 FAILURE in 7m 07s
integration-community.aws-18 FAILURE in 5m 36s
integration-community.aws-19 FAILURE in 4m 56s
integration-community.aws-20 FAILURE in 7m 19s
integration-community.aws-21 FAILURE in 4m 57s
integration-community.aws-22 FAILURE in 7m 15s

if 'Items' in item['Aliases']:
aliases = item['Aliases']['Items']
for alias in aliases:
keyed_list.update({alias: item})
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've a bit confused by the code of this function. I understand it's the original code and you've just moved it. We're ignoring all the aliases but the last one. It sounds wrong.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't why you are saying that it sounds wrong, basically the code extend the item so that we can find element using either distribution id or one its aliases

plugins/module_utils/cloudfront_facts.py Outdated Show resolved Hide resolved
@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

✔️ ansible-galaxy-importer SUCCESS in 4m 08s
✔️ build-ansible-collection SUCCESS in 5m 11s
ansible-test-sanity-aws-ansible-python38 FAILURE in 7m 53s (non-voting)
✔️ ansible-test-sanity-aws-ansible-2.12-python38 SUCCESS in 10m 13s
✔️ ansible-test-sanity-aws-ansible-2.13-python38 SUCCESS in 8m 44s
✔️ ansible-test-sanity-aws-ansible-2.14 SUCCESS in 8m 49s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 6m 56s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 7m 01s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 7m 12s
✔️ cloud-tox-py3 SUCCESS in 3m 18s
✔️ ansible-test-changelog SUCCESS in 2m 08s
✔️ ansible-test-splitter SUCCESS in 2m 27s
integration-amazon.aws-1 FAILURE in 4m 53s
integration-amazon.aws-2 FAILURE in 4m 55s
integration-amazon.aws-3 FAILURE in 4m 45s
integration-amazon.aws-4 FAILURE in 4m 59s
integration-amazon.aws-5 FAILURE in 5m 17s
integration-amazon.aws-6 FAILURE in 4m 54s
integration-amazon.aws-7 FAILURE in 7m 06s
integration-amazon.aws-8 FAILURE in 5m 07s
integration-amazon.aws-9 FAILURE in 4m 55s
integration-amazon.aws-10 FAILURE in 5m 11s
integration-amazon.aws-11 FAILURE in 6m 27s
integration-amazon.aws-12 FAILURE in 5m 29s
integration-amazon.aws-13 FAILURE in 5m 19s
integration-amazon.aws-14 FAILURE in 4m 45s
integration-amazon.aws-15 FAILURE in 5m 45s
integration-amazon.aws-16 FAILURE in 7m 26s
integration-amazon.aws-17 FAILURE in 6m 29s
integration-amazon.aws-18 FAILURE in 5m 36s
integration-amazon.aws-19 FAILURE in 5m 22s
integration-amazon.aws-20 FAILURE in 5m 13s
integration-amazon.aws-21 FAILURE in 5m 22s
integration-amazon.aws-22 FAILURE in 4m 31s
integration-community.aws-1 FAILURE in 7m 03s
integration-community.aws-2 FAILURE in 7m 02s
integration-community.aws-3 FAILURE in 6m 44s
integration-community.aws-4 FAILURE in 6m 22s
integration-community.aws-5 FAILURE in 5m 10s
integration-community.aws-6 FAILURE in 4m 49s
integration-community.aws-7 FAILURE in 5m 22s
integration-community.aws-8 FAILURE in 4m 57s
integration-community.aws-9 FAILURE in 4m 50s
integration-community.aws-10 FAILURE in 6m 53s
integration-community.aws-11 FAILURE in 5m 05s
integration-community.aws-12 FAILURE in 6m 36s
integration-community.aws-13 FAILURE in 5m 08s
integration-community.aws-14 FAILURE in 5m 31s
integration-community.aws-15 FAILURE in 4m 48s
integration-community.aws-16 FAILURE in 5m 10s
integration-community.aws-17 FAILURE in 5m 08s
integration-community.aws-18 FAILURE in 5m 32s
integration-community.aws-19 FAILURE in 4m 55s
integration-community.aws-20 FAILURE in 5m 00s
integration-community.aws-21 FAILURE in 5m 09s
integration-community.aws-22 FAILURE in 4m 55s

@goneri
Copy link
Member

goneri commented Nov 15, 2022

recheck

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

✔️ ansible-galaxy-importer SUCCESS in 4m 29s
✔️ build-ansible-collection SUCCESS in 6m 33s
ansible-test-sanity-aws-ansible-python38 FAILURE in 9m 14s (non-voting)
✔️ ansible-test-sanity-aws-ansible-2.12-python38 SUCCESS in 9m 29s
✔️ ansible-test-sanity-aws-ansible-2.13-python38 SUCCESS in 10m 03s
✔️ ansible-test-sanity-aws-ansible-2.14 SUCCESS in 9m 47s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 7m 03s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 6m 37s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 6m 31s
✔️ cloud-tox-py3 SUCCESS in 3m 48s
✔️ ansible-test-changelog SUCCESS in 2m 16s
✔️ ansible-test-splitter SUCCESS in 3m 01s
⚠️ integration-amazon.aws-1 SKIPPED
⚠️ 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

@goneri
Copy link
Member

goneri commented Nov 15, 2022

check ondemand

@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

✔️ build-ansible-collection SUCCESS in 5m 01s
✔️ ansible-test-splitter SUCCESS in 2m 29s
✔️ integration-amazon.aws-1 SUCCESS in 14m 20s
✔️ integration-amazon.aws-2 SUCCESS in 42m 11s
✔️ integration-amazon.aws-3 SUCCESS in 30m 48s
✔️ integration-amazon.aws-4 SUCCESS in 24m 57s
✔️ integration-amazon.aws-5 SUCCESS in 19m 02s
✔️ integration-amazon.aws-6 SUCCESS in 45m 03s
✔️ integration-amazon.aws-7 SUCCESS in 34m 20s
✔️ integration-amazon.aws-8 SUCCESS in 37m 26s
✔️ integration-amazon.aws-9 SUCCESS in 47m 07s
✔️ integration-amazon.aws-10 SUCCESS in 35m 51s
✔️ integration-amazon.aws-11 SUCCESS in 31m 23s
✔️ integration-amazon.aws-12 SUCCESS in 32m 42s
✔️ integration-amazon.aws-13 SUCCESS in 36m 06s
✔️ integration-amazon.aws-14 SUCCESS in 42m 39s
✔️ integration-amazon.aws-15 SUCCESS in 32m 07s
✔️ integration-amazon.aws-16 SUCCESS in 40m 46s
✔️ integration-amazon.aws-17 SUCCESS in 15m 56s
✔️ integration-amazon.aws-18 SUCCESS in 10m 34s
⚠️ integration-amazon.aws-19 SKIPPED
⚠️ integration-amazon.aws-20 SKIPPED
⚠️ integration-amazon.aws-21 SKIPPED
⚠️ integration-amazon.aws-22 SKIPPED
integration-community.aws-1 FAILURE in 40m 37s
✔️ integration-community.aws-2 SUCCESS in 23m 31s
✔️ integration-community.aws-3 SUCCESS in 43m 33s
✔️ integration-community.aws-4 SUCCESS in 11m 40s
✔️ integration-community.aws-5 SUCCESS in 34m 43s
✔️ integration-community.aws-6 SUCCESS in 27m 41s
✔️ integration-community.aws-7 SUCCESS in 58m 39s
✔️ integration-community.aws-8 SUCCESS in 47m 01s
✔️ integration-community.aws-9 SUCCESS in 28m 55s
✔️ integration-community.aws-10 SUCCESS in 42m 47s
✔️ integration-community.aws-11 SUCCESS in 21m 53s
✔️ integration-community.aws-12 SUCCESS in 11m 08s
⚠️ 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

Warning:
Failed to create check run ansible/ondemand: 403 Client Error: Forbidden for url: https://api.github.com/repos/ansible-collections/amazon.aws/check-runs

@goneri goneri added the mergeit Merge the PR (SoftwareFactory) label Nov 15, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (gate pipeline).

✔️ ansible-galaxy-importer SUCCESS in 5m 01s
✔️ build-ansible-collection SUCCESS in 5m 02s
✔️ ansible-test-splitter SUCCESS in 2m 37s
⚠️ integration-amazon.aws-1 SKIPPED
⚠️ 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

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit 50e26da into ansible-collections:main Nov 15, 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.

softwarefactory-project-zuul bot pushed a commit to ansible-collections/community.aws that referenced this pull request Feb 1, 2023
update module using module_utils/cloudfront_facts.py

Depends-On: ansible-collections/amazon.aws#1265
SUMMARY

update cloudfront_* modules, fix some bugs and add integration tests

ISSUE TYPE


Feature Pull Request

Reviewed-by: Alina Buzachis <None>
Reviewed-by: Bikouo Aubin <None>
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Sep 18, 2023
… support (ansible-collections#1265)

[5.0.0] iam_server_cerificate - complete deprecation cycles

SUMMARY
fixes: ansible-collections#1257
In ansible-collections#735 we deprecated passing filenames and prepared to switch the default value of dup_ok to True.
This PR completes the deprecation cycle (1 release late)
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
iam_server_certificate
ADDITIONAL INFORMATION

Reviewed-by: Alina Buzachis <None>
abikouo added a commit to abikouo/amazon.aws that referenced this pull request Sep 18, 2023
…ions#1596)

update module using module_utils/cloudfront_facts.py

Depends-On: ansible-collections#1265
SUMMARY

update cloudfront_* modules, fix some bugs and add integration tests

ISSUE TYPE


Feature Pull Request

Reviewed-by: Alina Buzachis <None>
Reviewed-by: Bikouo Aubin <None>
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Sep 18, 2023
… support (ansible-collections#1265)

[5.0.0] iam_server_cerificate - complete deprecation cycles

SUMMARY
fixes: ansible-collections#1257
In ansible-collections#735 we deprecated passing filenames and prepared to switch the default value of dup_ok to True.
This PR completes the deprecation cycle (1 release late)
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
iam_server_certificate
ADDITIONAL INFORMATION

Reviewed-by: Alina Buzachis <None>
abikouo added a commit to abikouo/amazon.aws that referenced this pull request Sep 18, 2023
…ions#1596)

update module using module_utils/cloudfront_facts.py

Depends-On: ansible-collections#1265
SUMMARY

update cloudfront_* modules, fix some bugs and add integration tests

ISSUE TYPE


Feature Pull Request

Reviewed-by: Alina Buzachis <None>
Reviewed-by: Bikouo Aubin <None>
alinabuzachis pushed a commit to alinabuzachis/amazon.aws that referenced this pull request Oct 13, 2023
…ansible-collections#1265)

[5.0.0] iam_server_cerificate - complete deprecation cycles

SUMMARY
fixes: ansible-collections#1257
In ansible-collections#735 we deprecated passing filenames and prepared to switch the default value of dup_ok to True.
This PR completes the deprecation cycle (1 release late)
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
iam_server_certificate
ADDITIONAL INFORMATION

Reviewed-by: Alina Buzachis <None>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@058ecff
alinabuzachis pushed a commit to alinabuzachis/amazon.aws that referenced this pull request Oct 13, 2023
…ansible-collections#1265)

[5.0.0] iam_server_cerificate - complete deprecation cycles

SUMMARY
fixes: ansible-collections#1257
In ansible-collections#735 we deprecated passing filenames and prepared to switch the default value of dup_ok to True.
This PR completes the deprecation cycle (1 release late)
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
iam_server_certificate
ADDITIONAL INFORMATION

Reviewed-by: Alina Buzachis <None>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@058ecff
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Oct 24, 2023
… support (ansible-collections#1265)

[5.0.0] iam_server_cerificate - complete deprecation cycles

SUMMARY
fixes: ansible-collections#1257
In ansible-collections#735 we deprecated passing filenames and prepared to switch the default value of dup_ok to True.
This PR completes the deprecation cycle (1 release late)
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
iam_server_certificate
ADDITIONAL INFORMATION

Reviewed-by: Alina Buzachis <None>
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.

3 participants