iamgini · June 2, 2020 10:54 · Jun 2, 2020
diff --git a/gistfile1.txt b/gistfile1.txt
@@ -0,0 +1,58 @@
+
+[Sam Doran](https://github.com/samdoran) contributed this Ansible role for PostgreSQL Repliction, especially for Database Streaming Replication for Ansible Tower Cluster. Some of the issues are already fixed and good to go.
+
+
+<!-- TOC orderedlist:false -->
+
+- [existing_pg_dir - undefined variable](#existingpgdir---undefined-variable)
+- [Typo in Variable value](#typo-in-variable-value)
+- [Issue with Loop as only item entry for replica](#issue-with-loop-as-only-item-entry-for-replica)
+
+<!-- /TOC -->
+
+## existing_pg_dir - undefined variable
+
+```
+TASK [packages_el : Check for old Postgres data] ***********************************************************************************
+fatal: [node05-db.lab.local]: FAILED! => {"msg": "The task includes an option with an undefined variable. The error was: 'existing_pg_dir' is undefined\n\nThe error appears to be in '/home/ansible/ansible-tower-setup-3.6.4-1/roles/packages_el/tasks/install_postgres.yml': line 2, column 3, but may\nbe elsewhere in the file depending on the exact syntax problem.\n\nThe offending line appears to be:\n\n---\n- name: Check for old Postgres data\n  ^ here\n"}
+```
+
+FIX:
+Added `existing_pg_dir: ""` in `import_role - packages_el` section
+
+
+## Typo in Variable value
+
+```
+TASK [samdoran.pgsql_replication : Add trust in pg_hba.conf] ***********************************************************************
+fatal: [node04-db.lab.local]: FAILED! => {"msg": "An unhandled exception occurred while templating '{{ hostvars[groups[pgsqlrep_group_name][0]]['ansible_facts]['all_ipv4_addresses'][-1] }}'. Error was a <class 'ansible.errors.AnsibleError'>, original message: template error while templating string: expected token ',', got 'all_ipv4_addresses'. String: {{ hostvars[groups[pgsqlrep_group_name][0]][\'ansible_facts]['all_ipv4_addresses'][-1] }}"}
+```
+
+**Fix:**
+
+Before : missing `'` at the end of ansible_facts (in replication sample playbook)
+
+```
+pgsqlrep_replica_address: "{{ hostvars[groups[pgsqlrep_group_name][0]]['ansible_facts']['all_ipv4_addresses'][-1] }}"
+```
+
+After : 
+
+```
+pgsqlrep_replica_address: "{{ hostvars[groups[pgsqlrep_group_name][0]]['ansible_facts']['all_ipv4_addresses'][-1] }}"
+```
+
+## Issue with Loop as only item entry for replica
+```
+TASK [samdoran.pgsql_replication : Add trust in pg_hba.conf] **************************************************************************
+fatal: [node04-db.lab.local]: FAILED! => {"msg": "Invalid data passed to 'loop', it requires a list, got this instead: 10.6.1.207. Hint: If you passed a list/dict of just one element, try adding wantlist=True to your lookup invocation or use q/query instead of lookup."}
+```
+
+Fix: 
+Replaced loop with with_items
+
+```
+#loop: "\{\{ pgsqlrep_replica_address \}\}"
+  with_items: "\{\{ pgsqlrep_replica_address \}\}"
+  notify: restart postgresql
+```