Rename KMS modules (ansible-collections#1284)

Rename KMS modules SUMMARY In line with the naming guidelines, rename aws_kms and aws_kms_info ISSUE TYPE Feature Pull Request COMPONENT NAME plugins/modules/aws_kms.py plugins/modules/aws_kms_info.py plugins/modules/kms_key.py plugins/modules/kms_key_info.py ADDITIONAL INFORMATION Reviewed-by: Joseph Torcasso <None> Reviewed-by: Alina Buzachis <None> This commit was initially merged in https://github.com/ansible-collections/community.aws See: ansible-collections@644225c
abikouo · Sep 22, 2022 · 41fd866 · 41fd866
1 parent eb1ee34
commit 41fd866
Show file tree

Hide file tree

Showing 15 changed files with 3,400 additions and 0 deletions.
diff --git a/plugins/modules/kms_key.py b/plugins/modules/kms_key.py
diff --git a/plugins/modules/kms_key_info.py b/plugins/modules/kms_key_info.py
diff --git a/tests/integration/targets/kms_key/aliases b/tests/integration/targets/kms_key/aliases
@@ -0,0 +1,8 @@
+# Various race conditions - likely needs waiters
+# https://github.com/ansible-collections/community.aws/issues/433
+# No KMS supported waiters, and manual waiting for updates didn't fix the issue either.
+# Issue likely from AWS side - added waits on updates in integration tests to workaround this.
+
+cloud/aws
+
+kms_key_info
diff --git a/tests/integration/targets/kms_key/inventory b/tests/integration/targets/kms_key/inventory
@@ -0,0 +1,10 @@
+# inventory names shortened down to fit resource name length limits
+[tests]
+states
+grants
+modify
+tagging
+
+[all:vars]
+ansible_connection=local
+ansible_python_interpreter="{{ ansible_playbook_python }}"
diff --git a/tests/integration/targets/kms_key/main.yml b/tests/integration/targets/kms_key/main.yml
@@ -0,0 +1,10 @@
+---
+# Beware: most of our tests here are run in parallel.
+# To add new tests you'll need to add a new host to the inventory and a matching
+# '{{ inventory_hostname }}'.yml file in roles/aws_kms/tasks/
+
+- hosts: all
+  gather_facts: no
+  strategy: free
+  roles:
+    - aws_kms
diff --git a/tests/integration/targets/kms_key/meta/main.yml b/tests/integration/targets/kms_key/meta/main.yml
@@ -0,0 +1 @@
+dependencies: []
diff --git a/tests/integration/targets/kms_key/roles/aws_kms/defaults/main.yml b/tests/integration/targets/kms_key/roles/aws_kms/defaults/main.yml
@@ -0,0 +1,2 @@
+---
+kms_key_alias: "ansible-test-{{ inventory_hostname | replace('_','-') }}{{ tiny_prefix }}"
diff --git a/tests/integration/targets/kms_key/roles/aws_kms/tasks/main.yml b/tests/integration/targets/kms_key/roles/aws_kms/tasks/main.yml
@@ -0,0 +1,13 @@
+---
+- name: 'aws_kms integration tests'
+  collections:
+    - amazon.aws
+    - community.aws
+  module_defaults:
+    group/aws:
+      aws_access_key: '{{ aws_access_key }}'
+      aws_secret_key: '{{ aws_secret_key }}'
+      security_token: '{{ security_token | default(omit) }}'
+      region: '{{ aws_region }}'
+  block:
+    - include: './test_{{ inventory_hostname }}.yml'
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1,2 @@
		---
		kms_key_alias: "ansible-test-{{ inventory_hostname \| replace('_','-') }}{{ tiny_prefix }}"