Upgrade: Add PATH for patronictl; Override become_method in Molecule tests

automation/molecule/default/converge.yml

@@ -6,6 +6,7 @@
   tasks:
     - name: Set variables for PostgreSQL Cluster deployment test
       ansible.builtin.set_fact:
+        ansible_become_method: su # Override become_method
         firewall_enabled_at_boot: false
         firewall_enable_ipv6: false # Added to prevent test failures in CI.
         swap_file_create: false # Added to prevent test failures in CI.

automation/molecule/default/prepare.yml

@@ -23,7 +23,6 @@
     - name: "Install netaddr dependency on controlling host"
       ansible.builtin.pip:
         name: netaddr
-      become: false
       environment:
         PIP_BREAK_SYSTEM_PACKAGES: "1"
 

automation/molecule/pg_upgrade/converge.yml

@@ -6,6 +6,7 @@
   tasks:
     - name: Set variables for Patroni cluster deployment test
       ansible.builtin.set_fact:
+        ansible_become_method: su # Override become_method
         firewall_enabled_at_boot: false
         firewall_enable_ipv6: false # Added to prevent test failures in CI.
         swap_file_create: false # Added to prevent test failures in CI.

automation/molecule/postgrespro/converge.yml

@@ -6,6 +6,7 @@
   tasks:
     - name: Set variables for molecule
       ansible.builtin.set_fact:
+        ansible_become_method: su # Override become_method
         firewall_enable_ipv6: false # Added to prevent test failures in CI.
         swap_file_create: false # Added to prevent test failures in CI.
         sysctl_set: false # Added to prevent test failures in CI.

automation/roles/upgrade/tasks/update_config.yml

@@ -258,6 +258,8 @@
 
 - name: "Remove 'standby_cluster' parameter from DCS (if exists)"
   ansible.builtin.command: patronictl -c {{ patroni_config_file }} edit-config -s standby_cluster=null --force
+  environment:
+    PATH: "{{ ansible_env.PATH }}:/usr/bin:/usr/local/bin"
   when: inventory_hostname in groups['primary']
 
 # Copy the pg_hba.conf file to a new PostgreSQL to save pg_hba rules.