Use k8s_info module instead of deprecated k8s_facts in molecule scaffold (#2168)

Author: geerlingguy

CHANGELOG.md

@@ -5,6 +5,7 @@
 
 ### Changed
 - Upgrade minimal Ansible version in the init projects from `2.4` to `2.6`. ([#2107](https://github.com/operator-framework/operator-sdk/pull/2107))
+- Use `k8s_info` module instead of deprecated `k8s_facts` module in molecule test scaffold. ([#2168](https://github.com/operator-framework/operator-sdk/issues/2168))
 
 ### Deprecated
 

doc/ansible/dev/retroactively-owned-resources.md

@@ -133,7 +133,7 @@ This file can be used as-is without user adjustments.
     - name: Import user variables
       include_vars: vars.yml
     - name: Retrieve owning resource
-      k8s_facts:
+      k8s_info:
         api_version: "{{ owning_resource.apiVersion }}"
         kind: "{{ owning_resource.kind }}"
         name: "{{ owning_resource.name }}"

internal/scaffold/ansible/molecule_default_asserts.go

@@ -46,7 +46,7 @@ const moleculeDefaultAssertsAnsibleTmpl = `---
     ansible_python_interpreter: '{{ ansible_playbook_python }}'
   tasks:
     - name: Get all pods in {{ namespace }}
-      k8s_facts:
+      k8s_info:
         api_version: v1
         kind: Pod
         namespace: '{{ namespace }}'

internal/scaffold/ansible/molecule_test_cluster_playbook.go

@@ -60,7 +60,7 @@ const moleculeTestClusterPlaybookAnsibleTmpl = `---
       msg: "{{ lookup('k8s', group='[[.Resource.FullGroup]]', api_version='[[.Resource.Version]]', kind='[[.Resource.Kind]]', namespace=namespace, resource_name=custom_resource.metadata.name) }}"
 
   - name: Wait 2m for reconciliation to run
-    k8s_facts:
+    k8s_info:
       api_version: '[[.Resource.Version]]'
       kind: '[[.Resource.Kind]]'
       namespace: '{{ namespace }}'

internal/scaffold/ansible/molecule_test_local_playbook.go

@@ -77,7 +77,7 @@ const moleculeTestLocalPlaybookAnsibleTmpl = `---
       when: hostvars[groups.k8s.0].build_cmd.changed
 
     - name: Wait 30s for Operator Deployment to terminate
-      k8s_facts:
+      k8s_info:
         api_version: '{{ definition.apiVersion }}'
         kind: '{{ definition.kind }}'
         namespace: '{{ namespace }}'
@@ -102,7 +102,7 @@ const moleculeTestLocalPlaybookAnsibleTmpl = `---
         definition: '{{ custom_resource }}'
 
     - name: Wait 2m for reconciliation to run
-      k8s_facts:
+      k8s_info:
         api_version: '{{ custom_resource.apiVersion }}'
         kind: '{{ custom_resource.kind }}'
         namespace: '{{ namespace }}'

test/ansible-inventory/molecule/test-local/playbook.yml

@@ -35,7 +35,7 @@
     when: hostvars[groups.k8s.0].build_cmd.changed
 
   - name: Wait 30s for Operator Deployment to terminate
-    k8s_facts:
+    k8s_info:
       api_version: '{{ definition.apiVersion }}'
       kind: '{{ definition.kind }}'
       namespace: '{{ namespace }}'
@@ -60,7 +60,7 @@
       definition: "{{ custom_resource }}"
 
   - name: Wait 2m for reconciliation to run
-    k8s_facts:
+    k8s_info:
       api_version: '{{ custom_resource.apiVersion }}'
       kind: '{{ custom_resource.kind }}'
       namespace: '{{ namespace }}'

test/ansible-memcached/asserts.yml

@@ -135,7 +135,7 @@
           definition: '{{ custom_resource }}'
 
       - name: Wait for the custom resource to be deleted
-        k8s_facts:
+        k8s_info:
           api_version: '{{ custom_resource.apiVersion }}'
           kind: '{{ custom_resource.kind }}'
           namespace: '{{ namespace }}'