Skip to content

Add policies and Terminator classes for Cloudfront Functions#323

Open
boutetnico wants to merge 2 commits intomattclay:mainfrom
boutetnico:boutetnico/cf-function
Open

Add policies and Terminator classes for Cloudfront Functions#323
boutetnico wants to merge 2 commits intomattclay:mainfrom
boutetnico:boutetnico/cf-function

Conversation

@boutetnico
Copy link

@boutetnico boutetnico commented Oct 16, 2025

@alinabuzachis
Copy link
Collaborator

Hi @boutetnico, I tried to test these permissions with your community.aws PR, but I got this error:

TASK [cloudfront_function : Ensure function was created] ***********************
task path: /root/ansible_collections/community/aws/tests/output/.tmp/integration/cloudfront_function-npqepufo-ÅÑŚÌβŁÈ/tests/integration/targets/cloudfront_function/tasks/main.yml:28
[WARNING]: Jinja constant strings should not contain embedded templates. This feature will be disabled by default in ansible-core 2.23.
Origin: /root/ansible_collections/community/aws/tests/output/.tmp/integration/cloudfront_function-npqepufo-ÅÑŚÌβŁÈ/tests/integration/targets/cloudfront_function/tasks/main.yml:32:13

30         that:
31           - create_function is changed
32           - create_function.function.name == "{{ function_name }}-test1"
               ^ column 13

Use inline expressions, for example: `when: "{{ a_var }}" == 42` becomes `when: a_var == 42`

[ERROR]: Task failed: Action failed: Assertion failed
Origin: /root/ansible_collections/community/aws/tests/output/.tmp/integration/cloudfront_function-npqepufo-ÅÑŚÌβŁÈ/tests/integration/targets/cloudfront_function/tasks/main.yml:28:7

26       register: create_function
27
28     - name: Ensure function was created
         ^ column 7

fatal: [testhost]: FAILED! => {
    "assertion": "create_function.function.status == \"UNASSOCIATED\"",
    "changed": false,
    "evaluated_to": false,
    "msg": "Assertion failed"
}

Can you please check?

@boutetnico
Copy link
Author

@alinabuzachis I have updated the PR, could you check again? Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments