From 2a8011a7cd9ff41f3dd4823ccd44e2a70507a9d9 Mon Sep 17 00:00:00 2001
From: kr4ut <community@kraut.hosting>
Date: Sun, 23 Jun 2019 18:14:25 +0200
Subject: [PATCH 1/2] Install Python bindings based on interpreter; Closes
 dj-wasabi/ansible-zabbix-server#148

---
 tasks/Debian.yml | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/tasks/Debian.yml b/tasks/Debian.yml
index b3f8bad..578b323 100644
--- a/tasks/Debian.yml
+++ b/tasks/Debian.yml
@@ -10,6 +10,7 @@
       - "http://repo.zabbix.com/zabbix/{{ zabbix_version }}/{{ ansible_distribution.lower() }}/"
       - "{{ ansible_distribution_release }}"
       - "main"
+    zabbix_python_prefix: "python{% if ansible_python_version | version_compare('3', '>=') %}3{% endif %}"
 
 - name: "Debian | Set some facts"
   set_fact:
@@ -96,7 +97,7 @@
 
 - name: "Debian | Install Ansible module dependencies"
   apt:
-    name: python-psycopg2
+    name: "{{ zabbix_python_prefix }}-psycopg2"
     state: present
   register: zabbix_server_dependencies_installed
   until: zabbix_server_dependencies_installed is succeeded
@@ -110,7 +111,7 @@
   apt:
     name:
       - mysql-client
-      - python-mysqldb
+      - "{{ zabbix_python_prefix }}-mysqldb"
     state: present
   register: zabbix_server_dependencies_installed
   until: zabbix_server_dependencies_installed is succeeded

From 35f366389f50634896f00ec87b7e6402a3aba83d Mon Sep 17 00:00:00 2001
From: kr4ut <community@kraut.hosting>
Date: Sat, 3 Aug 2019 17:08:42 +0200
Subject: [PATCH 2/2] Fix lint error use shell only when shell functionality is
 required

---
 tasks/postgresql.yml | 67 ++++++++++++++++++++------------------------
 1 file changed, 30 insertions(+), 37 deletions(-)

diff --git a/tasks/postgresql.yml b/tasks/postgresql.yml
index 12c5c32..0be5721 100644
--- a/tasks/postgresql.yml
+++ b/tasks/postgresql.yml
@@ -1,39 +1,6 @@
 ---
 # task file for postgresql
 
-- name: "Set facts for usage in this playbook"
-  set_fact:
-    postgres_create_command:
-      - "cd {{ datafiles_path }} &&"
-      - "if [ -f create.sql.gz ]; then gunzip create.sql.gz ; fi &&"
-      - "psql -h '{{ zabbix_server_dbhost }}'"
-      - "-U '{{ zabbix_server_dbuser }}'"
-      - "-d '{{ zabbix_server_dbname }}'"
-      - "-p '{{ zabbix_server_dbport }}'"
-      - "-f create.sql && touch /etc/zabbix/schema.done"
-    postgres_schema_command:
-      - "cd {{ datafiles_path }} && "
-      - "if [ -f schema.sql.gz ]; then gunzip schema.sql.gz ; fi &&"
-      - "psql -h '{{ zabbix_server_dbhost }}'"
-      - "-U '{{ zabbix_server_dbuser }}'"
-      - "-d '{{ zabbix_server_dbname }}'"
-      - "-p '{{ zabbix_server_dbport }}'"
-      - "-f schema.sql && touch /etc/zabbix/schema.done"
-    postgres_images_command:
-      - "cd {{ datafiles_path }} &&"
-      - "psql -h '{{ zabbix_server_dbhost }}'"
-      - "-U '{{ zabbix_server_dbuser }}'"
-      - "-d '{{ zabbix_server_dbname }}'"
-      - "-p '{{ zabbix_server_dbport }}'"
-      - "-f images.sql && touch /etc/zabbix/images.done"
-    postgres_data_command:
-      - "cd {{ datafiles_path }} &&"
-      - "psql -h '{{ zabbix_server_dbhost }}'"
-      - "-U '{{ zabbix_server_dbuser }}'"
-      - "-d '{{ zabbix_server_dbname }}'"
-      - "-p '{{ zabbix_server_dbport }}'"
-      - "-f data.sql && touch /etc/zabbix/data.done"
-
 - name: "Set the correct delegated_dbhost (to support postgres db deployment on a remote dbhost)"
   set_fact:
     delegated_dbhost: "{{ zabbix_server_dbhost if (zabbix_server_dbhost != 'localhost') else inventory_hostname }}"
@@ -95,7 +62,14 @@
     - database
 
 - name: "PostgreSQL | Create schema"
-  shell: "{{ postgres_create_command | join(' ') }}"
+  shell: >
+    cd {{ datafiles_path }} &&
+    if [ -f create.sql.gz ]; then gunzip create.sql.gz ; fi &&
+    psql -h '{{ zabbix_server_dbhost }}'
+    -U '{{ zabbix_server_dbuser }}'
+    -d '{{ zabbix_server_dbname }}'
+    -p '{{ zabbix_server_dbport }}'
+    -f create.sql && touch /etc/zabbix/schema.done
   args:
     creates: /etc/zabbix/schema.done
     warn: no
@@ -133,7 +107,14 @@
     - create_dir_or_not.stat.exists
 
 - name: "PostgreSQL | Importing schema file"
-  shell: "{{ postgres_schema_command | join(' ') }}"
+  shell: >
+    cd {{ datafiles_path }} &&
+    if [ -f schema.sql.gz ]; then gunzip schema.sql.gz ; fi &&
+    psql -h '{{ zabbix_server_dbhost }}'
+    -U '{{ zabbix_server_dbuser }}'
+    -d '{{ zabbix_server_dbname }}'
+    -p '{{ zabbix_server_dbport }}'
+    -f schema.sql && touch /etc/zabbix/schema.done
   args:
     creates: /etc/zabbix/schema.done
     warn: no
@@ -146,7 +127,13 @@
     - database
 
 - name: "PostgreSQL | Importing images file"
-  shell: "{{ postgres_images_command | join(' ') }}"
+  shell: >
+    cd {{ datafiles_path }} &&
+    psql -h '{{ zabbix_server_dbhost }}'
+    -U '{{ zabbix_server_dbuser }}'
+    -d '{{ zabbix_server_dbname }}'
+    -p '{{ zabbix_server_dbport }}'
+    -f images.sql && touch /etc/zabbix/images.done
   args:
     creates: /etc/zabbix/images.done
     warn: no
@@ -158,7 +145,13 @@
     - database
 
 - name: "PostgreSQL | Importing data file"
-  shell: "{{ postgres_data_command | join(' ') }}"
+  shell: >
+    cd {{ datafiles_path }} &&
+    psql -h '{{ zabbix_server_dbhost }}'
+    -U '{{ zabbix_server_dbuser }}'
+    -d '{{ zabbix_server_dbname }}'
+    -p '{{ zabbix_server_dbport }}'
+    -f data.sql && touch /etc/zabbix/data.done
   args:
     creates: /etc/zabbix/data.done
     warn: no