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

update module using module_utils/cloudfront_facts.py #1596

Conversation

abikouo
Copy link
Contributor

@abikouo abikouo commented Nov 15, 2022

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

SUMMARY

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

ISSUE TYPE
  • Feature Pull Request

@ansibullbot
Copy link

@ansibullbot ansibullbot added community_review feature This issue/PR relates to a feature request integration tests/integration module module needs_triage plugins plugin (any type) tests tests labels Nov 15, 2022
@softwarefactory-project-zuul
Copy link
Contributor

This change depends on a change that failed to merge.

@abikouo
Copy link
Contributor Author

abikouo commented Nov 15, 2022

recheck

@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

✔️ ansible-galaxy-importer SUCCESS in 4m 42s
✔️ build-ansible-collection SUCCESS in 5m 42s
ansible-test-sanity-docker-devel FAILURE in 12m 00s (non-voting)
ansible-test-sanity-docker-milestone FAILURE in 8m 15s (non-voting)
ansible-test-sanity-docker-stable-2.12 FAILURE in 9m 15s
ansible-test-sanity-docker-stable-2.13 FAILURE in 10m 16s
ansible-test-sanity-docker-stable-2.14 FAILURE in 8m 49s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 7m 20s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 5m 56s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 5m 50s
ansible-test-changelog FAILURE in 2m 20s
✔️ ansible-test-splitter SUCCESS in 2m 42s
integration-community.aws-1 FAILURE in 7m 02s
⚠️ 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
Copy link
Contributor

Build failed.

✔️ ansible-galaxy-importer SUCCESS in 4m 00s
✔️ build-ansible-collection SUCCESS in 5m 11s
ansible-test-sanity-docker-devel FAILURE in 10m 03s (non-voting)
ansible-test-sanity-docker-milestone FAILURE in 9m 13s (non-voting)
ansible-test-sanity-docker-stable-2.12 FAILURE in 10m 24s
ansible-test-sanity-docker-stable-2.13 FAILURE in 9m 58s
ansible-test-sanity-docker-stable-2.14 FAILURE in 10m 31s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 5m 51s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 7m 08s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 5m 49s
ansible-test-changelog FAILURE in 2m 11s
✔️ ansible-test-splitter SUCCESS in 2m 33s
integration-community.aws-1 FAILURE in 5m 51s
⚠️ 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
Copy link
Contributor

Build failed.

✔️ ansible-galaxy-importer SUCCESS in 3m 57s
✔️ build-ansible-collection SUCCESS in 4m 59s
ansible-test-sanity-docker-devel FAILURE in 9m 07s (non-voting)
ansible-test-sanity-docker-milestone FAILURE in 8m 53s (non-voting)
ansible-test-sanity-docker-stable-2.12 FAILURE in 13m 00s
ansible-test-sanity-docker-stable-2.13 FAILURE in 9m 27s
ansible-test-sanity-docker-stable-2.14 FAILURE in 9m 12s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 6m 15s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 6m 00s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 5m 41s
ansible-test-changelog FAILURE in 2m 06s
✔️ ansible-test-splitter SUCCESS in 2m 50s
integration-community.aws-1 FAILURE in 5m 47s
⚠️ 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
Copy link
Contributor

Build failed.

✔️ ansible-galaxy-importer SUCCESS in 4m 11s
✔️ build-ansible-collection SUCCESS in 4m 56s
✔️ ansible-test-sanity-docker-devel SUCCESS in 8m 44s (non-voting)
ansible-test-sanity-docker-milestone FAILURE in 8m 23s (non-voting)
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 10m 23s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 8m 42s
✔️ ansible-test-sanity-docker-stable-2.14 SUCCESS in 8m 48s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 5m 54s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 7m 35s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 6m 50s
✔️ ansible-test-changelog SUCCESS in 2m 13s
✔️ ansible-test-splitter SUCCESS in 2m 29s
integration-community.aws-1 FAILURE in 6m 51s
⚠️ 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

@jillr
Copy link
Collaborator

jillr commented Nov 21, 2022

When running the cloudfront_distribution tests to test the terminator PR, they pass but there are 2 distributions remaining in the CI AWS account that are not automatically deleted by the test suite. Please ensure that all resources created by the tests are removed by the tests.

@abikouo
Copy link
Contributor Author

abikouo commented Nov 22, 2022

When running the cloudfront_distribution tests to test the terminator PR, they pass but there are 2 distributions remaining in the CI AWS account that are not automatically deleted by the test suite. Please ensure that all resources created by the tests are removed by the tests.

only the second cloudfront distribution was not deleted, I have updated the test. Could you please validate once again in a clean environment? Thanks

@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

✔️ ansible-galaxy-importer SUCCESS in 4m 13s
✔️ build-ansible-collection SUCCESS in 5m 04s
✔️ ansible-test-sanity-docker-devel SUCCESS in 9m 43s (non-voting)
ansible-test-sanity-docker-milestone FAILURE in 8m 54s (non-voting)
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 8m 56s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 9m 06s
✔️ ansible-test-sanity-docker-stable-2.14 SUCCESS in 9m 06s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 7m 19s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 5m 39s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 5m 50s
✔️ ansible-test-changelog SUCCESS in 2m 09s
✔️ ansible-test-splitter SUCCESS in 2m 54s
integration-community.aws-1 FAILURE in 6m 47s
⚠️ 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

@gravesm
Copy link
Member

gravesm commented Dec 1, 2022

recheck

@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

✔️ ansible-galaxy-importer SUCCESS in 4m 17s
✔️ build-ansible-collection SUCCESS in 5m 27s
ansible-test-sanity-docker-devel FAILURE in 9m 07s (non-voting)
ansible-test-sanity-docker-milestone FAILURE in 8m 59s (non-voting)
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 11m 22s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 9m 51s
✔️ ansible-test-sanity-docker-stable-2.14 SUCCESS in 10m 17s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 6m 00s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 8m 00s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 7m 42s
✔️ ansible-test-changelog SUCCESS in 2m 39s
✔️ ansible-test-splitter SUCCESS in 4m 07s
integration-community.aws-1 FAILURE in 44m 19s
⚠️ 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

@gravesm
Copy link
Member

gravesm commented Dec 1, 2022

2022-12-01 16:41:30.567401 | controller | TASK [cloudfront_distribution : check that custom origin uses the provided origin_access_identity] ***
2022-12-01 16:41:30.567408 | controller | task path: /home/zuul/.ansible/collections/ansible_collections/community/aws/tests/integration/targets/cloudfront_distribution/tasks/main.yml:505
2022-12-01 16:41:30.643578 | controller | fatal: [testhost]: FAILED! => {
2022-12-01 16:41:30.643613 | controller |     "assertion": "update_distribution_with_specific_access_identity.origins['items'].0.s3_origin_config.origin_access_identity == origin_access_identity",

I saw this failure locally, too, but when I re-ran all the tests passed. Something here seems a little flaky.

@abikouo abikouo force-pushed the cloudfront_facts_1 branch from 839036b to 920e331 Compare January 10, 2023 14:34
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

✔️ ansible-galaxy-importer SUCCESS in 4m 09s
✔️ build-ansible-collection SUCCESS in 5m 07s
✔️ ansible-test-sanity-docker-devel SUCCESS in 9m 56s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 9m 35s (non-voting)
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 10m 21s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 8m 54s
✔️ ansible-test-sanity-docker-stable-2.14 SUCCESS in 9m 38s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 6m 43s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 5m 52s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 5m 46s
✔️ ansible-test-units-amazon-aws-python310 SUCCESS in 6m 06s
✔️ ansible-test-changelog SUCCESS in 2m 12s
✔️ ansible-test-splitter SUCCESS in 2m 42s
✔️ integration-community.aws-1 SUCCESS in 39m 06s
⚠️ 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

@abikouo abikouo force-pushed the cloudfront_facts_1 branch from 920e331 to a38e0f8 Compare January 27, 2023 09:24
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

✔️ ansible-galaxy-importer SUCCESS in 3m 46s
✔️ build-ansible-collection SUCCESS in 5m 40s
✔️ ansible-test-sanity-docker-devel SUCCESS in 10m 18s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 9m 16s (non-voting)
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 10m 05s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 9m 13s
✔️ ansible-test-sanity-docker-stable-2.14 SUCCESS in 9m 52s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 5m 56s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 5m 44s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 7m 09s
✔️ ansible-test-units-amazon-aws-python310 SUCCESS in 6m 56s
✔️ ansible-test-changelog SUCCESS in 2m 23s
✔️ ansible-test-splitter SUCCESS in 2m 36s
✔️ integration-community.aws-1 SUCCESS in 20m 51s
✔️ integration-community.aws-2 SUCCESS in 8m 07s
✔️ integration-community.aws-3 SUCCESS in 9m 33s
⚠️ 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

@abikouo abikouo added the mergeit Merge the PR (SoftwareFactory) label Feb 1, 2023
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (gate pipeline).

✔️ ansible-galaxy-importer SUCCESS in 3m 44s
✔️ build-ansible-collection SUCCESS in 5m 33s
✔️ ansible-test-sanity-docker-devel SUCCESS in 8m 43s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 9m 12s (non-voting)
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 9m 50s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 9m 21s
✔️ ansible-test-sanity-docker-stable-2.14 SUCCESS in 9m 31s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 5m 52s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 5m 36s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 5m 47s
✔️ ansible-test-units-amazon-aws-python310 SUCCESS in 14m 52s
✔️ ansible-test-changelog SUCCESS in 2m 11s
✔️ ansible-test-splitter SUCCESS in 2m 42s
✔️ integration-community.aws-1 SUCCESS in 21m 12s
✔️ integration-community.aws-2 SUCCESS in 10m 02s
✔️ integration-community.aws-3 SUCCESS in 9m 39s
⚠️ 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 dbb482c into ansible-collections:main Feb 1, 2023
@github-actions
Copy link

github-actions bot commented Feb 1, 2023

Docs Build 📝

Thank you for contribution!✨

This PR has been merged and your docs changes will be incorporated when they are next published.

@abikouo abikouo deleted the cloudfront_facts_1 branch October 20, 2023 16:47
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 integration tests/integration mergeit Merge the PR (SoftwareFactory) module module plugins plugin (any type) tests tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants