Change name of server certs in integration tests (ansible-collections…

…#1503) Change name of server certs in integration tests SUMMARY There is a mismatch in the terminator repo for the permissions on iam server certs and the terminator class for server certs. This has led to the CI account reaching the quota for server certs, as some of the older certs are not getting cleaned up. I don't know the reasoning behind limiting the terminator class to only certs prefixed with ansible-test. The safest option here seems to be to change the permissions to only allow creating certs with this prefix. Once this update is merged I'll change the permissions on the terminator repo and manually clear out the older certs. ISSUE TYPE COMPONENT NAME ADDITIONAL INFORMATION Reviewed-by: Mark Chappell <None>
abikouo · Sep 20, 2022 · 3847690 · 3847690
1 parent 9748d1d
commit 3847690
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/tests/integration/targets/elb_network_lb/tasks/generate-certs.yml b/tests/integration/targets/elb_network_lb/tasks/generate-certs.yml
@@ -45,7 +45,7 @@
 
 - name: create certificate
   iam_server_certificate:
-    name: 'nlb_{{ tiny_prefix }}'
+    name: 'ansible-test-nlb-{{ tiny_prefix }}'
     state: present
     cert: "{{ lookup('file', path_cert_pem) }}"
     key: "{{ lookup('file', path_cert_key) }}"

diff --git a/tests/integration/targets/elb_network_lb/tasks/main.yml b/tests/integration/targets/elb_network_lb/tasks/main.yml
@@ -237,7 +237,7 @@
 
     - name: destroy certificate
       iam_server_certificate:
-        name: 'nlb_{{ tiny_prefix }}'
+        name: 'ansible-test-nlb-{{ tiny_prefix }}'
         state: absent
       register: remove_cert
       ignore_errors: yes