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

…tests (#832)
vitabaks · Dec 11, 2024 · 2fe25cd · 2fe25cd
1 parent 03157f7
commit 2fe25cd
Show file tree

Hide file tree

Showing 5 changed files with 5 additions and 1 deletion.
diff --git a/automation/molecule/default/converge.yml b/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.

diff --git a/automation/molecule/default/prepare.yml b/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"
 

diff --git a/automation/molecule/pg_upgrade/converge.yml b/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.

diff --git a/automation/molecule/postgrespro/converge.yml b/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.

diff --git a/automation/roles/upgrade/tasks/update_config.yml b/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.