Add support for tags to molecule tests (#62)

changelogs/fragments/62-molecule-tags.yaml

@@ -0,0 +1,3 @@
+---
+minor_changes:
+  - add support for using tags when running molecule test suite (https://github.com/ansible-collections/kubernetes.core/pull/62).

molecule/default/converge.yml

@@ -20,20 +20,123 @@
       assert:
         that: (pod_list.resources | count) > 5
 
-    - include_tasks: tasks/delete.yml
+    - name: Include access_review.yml
-    - include_tasks: tasks/scale.yml
+      include_tasks:
-    - include_tasks: tasks/apply.yml
+        file: tasks/access_review.yml
-    - include_tasks: tasks/waiter.yml
+        apply:
-    - include_tasks: tasks/full.yml
+          tags: [ access_review, k8s ]
-    - include_tasks: tasks/exec.yml
+      tags:
-    - include_tasks: tasks/log.yml
+        - always
-    - include_tasks: tasks/cluster_info.yml
+    - name: Include append_hash.yml
-    - include_tasks: tasks/access_review.yml
+      include_tasks:
-    - include_tasks: tasks/rollback.yml
+        file: tasks/append_hash.yml
-    - include_tasks: tasks/gc.yml
+        apply:
+          tags: [ append_hash, k8s ]
+      tags:
+        - always
+    - name: Include apply.yml
+      include_tasks:
+        file: tasks/apply.yml
+        apply:
+          tags: [ apply, k8s ]
+      tags:
+        - always
+    - name: Include cluster_info.yml
+      include_tasks:
+        file: tasks/cluster_info.yml
+        apply:
+          tags: [ cluster_info, k8s ]
+      tags:
+        - always
+    - name: Include crd.yml
+      include_tasks:
+        file: tasks/crd.yml
+        apply:
+          tags: [ crd, k8s ]
+      tags:
+        - always
+    - name: Include delete.yml
+      include_tasks:
+        file: tasks/delete.yml
+        apply:
+          tags: [ delete, k8s ]
+      tags:
+        - always
+    - name: Include exec.yml
+      include_tasks:
+        file: tasks/exec.yml
+        apply:
+          tags: [ exec, k8s ]
+      tags:
+        - always
+    - name: Include full.yml
+      include_tasks:
+        file: tasks/full.yml
+        apply:
+          tags: [ full, k8s ]
+      tags:
+        - always
+    - name: Include gc.yml
+      include_tasks:
+        file: tasks/gc.yml
+        apply:
+          tags: [ gc, k8s ]
+      tags:
+        - always
+    - name: Include info.yml
+      include_tasks:
+        file: tasks/info.yml
+        apply:
+          tags: [ info, k8s ]
+      tags:
+        - always
+    - name: Include lists.yml
+      include_tasks:
+        file: tasks/lists.yml
+        apply:
+          tags: [ lists, k8s ]
+      tags:
+        - always
+    - name: Include log.yml
+      include_tasks:
+        file: tasks/log.yml
+        apply:
+          tags: [ log, k8s ]
+      tags:
+        - always
+    - name: Include rollback.yml
+      include_tasks:
+        file: tasks/rollback.yml
+        apply:
+          tags: [ rollback, k8s ]
+      tags:
+        - always
+    - name: Include scale.yml
+      include_tasks:
+        file: tasks/scale.yml
+        apply:
+          tags: [ scale, k8s ]
+      tags:
+        - always
+    - name: Include template.yml
+      include_tasks:
+        file: tasks/template.yml
+        apply:
+          tags: [ template, k8s ]
+      tags:
+        - always
+    - name: Include waiter.yml
+      include_tasks:
+        file: tasks/waiter.yml
+        apply:
+          tags: [ waiter, k8s ]
+      tags:
+        - always
 
   roles:
-    - helm
+    - role: helm
+      tags:
+        - helm
 
   post_tasks:
     - name: Ensure namespace exists

molecule/default/tasks/full.yml

@@ -333,10 +333,6 @@
       register: k8s_info_testing6
       failed_when: not k8s_info_testing6.resources or k8s_info_testing6.resources[0].status.phase != "Active"
 
-    - include_tasks: crd.yml
-    - include_tasks: lists.yml
-    - include_tasks: append_hash.yml
-
   always:
     - name: Delete all namespaces
       k8s:

molecule/default/tasks/info.yml

@@ -2,7 +2,6 @@
 - block:
     - set_fact:
         wait_namespace: wait
-        k8s_pod_name: pod-info-1
         multi_pod_one: multi-pod-1
         multi_pod_two: multi-pod-2
 
@@ -200,6 +199,9 @@
         that:
           - "{{ lookup('pipe', 'date +%s') }} - {{ start }} > 30"
 
+  vars:
+    k8s_pod_name: pod-info-1
+
   always:
     - name: Remove namespace
       k8s:

molecule/default/tasks/lists.yml

@@ -1,139 +1,147 @@
 ---
-- name: Ensure testing1 namespace exists
-  k8s:
-    api_version: v1
-    kind: Namespace
-    name: testing1
-
 - block:
-    - name: Create configmaps
+    - name: Ensure testing1 namespace exists
       k8s:
-        namespace: testing1
-        definition:
-          apiVersion: v1
-          kind: ConfigMapList
-          items: '{{ configmaps }}'
-
-    - name: Get ConfigMaps
-      k8s_info:
         api_version: v1
-        kind: ConfigMap
+        kind: Namespace
-        namespace: testing1
+        name: testing1
-        label_selectors:
-          - app=test
-      register: cms
 
-    - name: All three configmaps should exist
+    - block:
-      assert:
+        - name: Create configmaps
-        that: item.data.a is defined
+          k8s:
-      with_items: '{{ cms.resources }}'
+            namespace: testing1
+            definition:
+              apiVersion: v1
+              kind: ConfigMapList
+              items: '{{ configmaps }}'
 
-    - name: Delete configmaps
+        - name: Get ConfigMaps
+          k8s_info:
+            api_version: v1
+            kind: ConfigMap
+            namespace: testing1
+            label_selectors:
+              - app=test
+          register: cms
+
+        - name: All three configmaps should exist
+          assert:
+            that: item.data.a is defined
+          with_items: '{{ cms.resources }}'
+
+        - name: Delete configmaps
+          k8s:
+            state: absent
+            namespace: testing1
+            definition:
+              apiVersion: v1
+              kind: ConfigMapList
+              items: '{{ configmaps }}'
+
+        - name: Get ConfigMaps
+          k8s_info:
+            api_version: v1
+            kind: ConfigMap
+            namespace: testing1
+            label_selectors:
+              - app=test
+          register: cms
+
+        - name: All three configmaps should not exist
+          assert:
+            that: not cms.resources
+      vars:
+        configmaps:
+          - metadata:
+              name: list-example-1
+              labels:
+                app: test
+            data:
+              a: first
+          - metadata:
+              name: list-example-2
+              labels:
+                app: test
+            data:
+              a: second
+          - metadata:
+              name: list-example-3
+              labels:
+                app: test
+            data:
+              a: third
+
+    - block:
+        - name: Create list of arbitrary resources
+          k8s:
+            namespace: testing1
+            definition:
+              apiVersion: v1
+              kind: List
+              namespace: testing1
+              items: '{{ resources }}'
+
+        - name: Get the created resources
+          k8s_info:
+            api_version: '{{ item.apiVersion }}'
+            kind: '{{ item.kind }}'
+            namespace: testing1
+            name: '{{ item.metadata.name }}'
+          register: list_resources
+          with_items: '{{ resources }}'
+
+        - name: All resources should exist
+          assert:
+            that: ((list_resources.results | sum(attribute="resources", start=[])) | length) == (resources | length)
+
+        - name: Delete list of arbitrary resources
+          k8s:
+            state: absent
+            namespace: testing1
+            definition:
+              apiVersion: v1
+              kind: List
+              namespace: testing1
+              items: '{{ resources }}'
+
+        - name: Get the resources
+          k8s_info:
+            api_version: '{{ item.apiVersion }}'
+            kind: '{{ item.kind }}'
+            namespace: testing1
+            name: '{{ item.metadata.name }}'
+          register: list_resources
+          with_items: '{{ resources }}'
+
+        - name: The resources should not exist
+          assert:
+            that: not ((list_resources.results | sum(attribute="resources", start=[])) | length)
+      vars:
+        resources:
+          - apiVersion: v1
+            kind: ConfigMap
+            metadata:
+              name: list-example-4
+            data:
+              key: value
+          - apiVersion: v1
+            kind: Service
+            metadata:
+              name: list-example-svc
+              labels:
+                app: test
+            spec:
+              selector:
+                app: test
+              ports:
+                - protocol: TCP
+                  targetPort: 8000
+                  name: port-8000-tcp
+                  port: 8000
+  always:
+    - name: Remove testing1 namespace
       k8s:
+        kind: Namespace
+        name: testing1
         state: absent
-        namespace: testing1
+      ignore_errors: yes
-        definition:
-          apiVersion: v1
-          kind: ConfigMapList
-          items: '{{ configmaps }}'
-
-    - name: Get ConfigMaps
-      k8s_info:
-        api_version: v1
-        kind: ConfigMap
-        namespace: testing1
-        label_selectors:
-          - app=test
-      register: cms
-
-    - name: All three configmaps should not exist
-      assert:
-        that: not cms.resources
-  vars:
-    configmaps:
-      - metadata:
-          name: list-example-1
-          labels:
-            app: test
-        data:
-          a: first
-      - metadata:
-          name: list-example-2
-          labels:
-            app: test
-        data:
-          a: second
-      - metadata:
-          name: list-example-3
-          labels:
-            app: test
-        data:
-          a: third
-
-- block:
-    - name: Create list of arbitrary resources
-      k8s:
-        namespace: testing1
-        definition:
-          apiVersion: v1
-          kind: List
-          namespace: testing1
-          items: '{{ resources }}'
-
-    - name: Get the created resources
-      k8s_info:
-        api_version: '{{ item.apiVersion }}'
-        kind: '{{ item.kind }}'
-        namespace: testing1
-        name: '{{ item.metadata.name }}'
-      register: list_resources
-      with_items: '{{ resources }}'
-
-    - name: All resources should exist
-      assert:
-        that: ((list_resources.results | sum(attribute="resources", start=[])) | length) == (resources | length)
-
-    - name: Delete list of arbitrary resources
-      k8s:
-        state: absent
-        namespace: testing1
-        definition:
-          apiVersion: v1
-          kind: List
-          namespace: testing1
-          items: '{{ resources }}'
-
-    - name: Get the resources
-      k8s_info:
-        api_version: '{{ item.apiVersion }}'
-        kind: '{{ item.kind }}'
-        namespace: testing1
-        name: '{{ item.metadata.name }}'
-      register: list_resources
-      with_items: '{{ resources }}'
-
-    - name: The resources should not exist
-      assert:
-        that: not ((list_resources.results | sum(attribute="resources", start=[])) | length)
-  vars:
-    resources:
-      - apiVersion: v1
-        kind: ConfigMap
-        metadata:
-          name: list-example-4
-        data:
-          key: value
-      - apiVersion: v1
-        kind: Service
-        metadata:
-          name: list-example-svc
-          labels:
-            app: test
-        spec:
-          selector:
-            app: test
-          ports:
-            - protocol: TCP
-              targetPort: 8000
-              name: port-8000-tcp
-              port: 8000