Skip to content

Add policies and Terminator classes for Cloudfront Functions#323

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

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

Conversation

@boutetnico
Copy link
Copy Markdown

@boutetnico boutetnico commented Oct 16, 2025

@alinabuzachis
Copy link
Copy Markdown
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
Copy Markdown
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

2 participants