Skip to content

Add policies and Terminator classes for SES Configuration Sets - #325

Open
boutetnico wants to merge 3 commits into
mattclay:mainfrom
boutetnico:boutetnico/ses-conf-set
Open

Add policies and Terminator classes for SES Configuration Sets#325
boutetnico wants to merge 3 commits into
mattclay:mainfrom
boutetnico:boutetnico/ses-conf-set

Conversation

@boutetnico

Copy link
Copy Markdown

@boutetnico

Copy link
Copy Markdown
Author

Hello @mattclay, is there any chance to get a review?

@mattclay

mattclay commented Dec 4, 2025

Copy link
Copy Markdown
Owner

@alinabuzachis Can you or someone on your team take a look at this?

@alinabuzachis

Copy link
Copy Markdown
Collaborator

@boutetnico I apologise for the delay. I'm setting aside some time to review this early next week and will let you know.

@alinabuzachis

Copy link
Copy Markdown
Collaborator

@boutetnico I'm trying to test your PR, but I'm getting:

TASK [ses_configuration_set : Test create configuration set (check mode)] ******
task path: /root/ansible_collections/community/aws/tests/output/.tmp/integration/ses_configuration_set-wl177ewb-ÅÑŚÌβŁÈ/tests/integration/targets/ses_configuration_set/tasks/main.yml:10
Using module file /root/ansible_collections/community/aws/plugins/modules/ses_configuration_set.py
Pipelining is enabled.
<testhost> ESTABLISH LOCAL CONNECTION FOR USER: root
<testhost> EXEC /bin/sh -c 'ANSIBLE_DEBUG_BOTOCORE_LOGS=True /usr/bin/python3.14 && sleep 0'
[ERROR]: Task failed: Module failed: 'NoneType' object is not iterable
Origin: /root/ansible_collections/community/aws/tests/output/.tmp/integration/ses_configuration_set-wl177ewb-ÅÑŚÌβŁÈ/tests/integration/targets/ses_configuration_set/tasks/main.yml:10:7

 8   block:
 9     # ============================================================
10     - name: Test create configuration set (check mode)
         ^ column 7

fatal: [testhost]: FAILED! => {
    "changed": false,
    "msg": "Task failed: Module failed: 'NoneType' object is not iterable"
}

Can you please check?

@boutetnico

Copy link
Copy Markdown
Author

@alinabuzachis thanks for reviewing, I have updated my PR but I can't test it, could you check again or provide me a way to test it myself?

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.

3 participants