Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

debian bullseye support #88

Closed
pmauduit opened this issue Aug 27, 2021 · 3 comments
Closed

debian bullseye support #88

pmauduit opened this issue Aug 27, 2021 · 3 comments

Comments

@pmauduit
Copy link
Member

Apart from fiddling with #87 (which is not due to the upgrade), here are the only modification I had to do, in order to have the playbook running successfully on bullseye:

diff --git a/requirements.yaml b/requirements.yaml
index d736664..1f3b3bb 100644
--- a/requirements.yaml
+++ b/requirements.yaml
@@ -1,3 +1,3 @@
 # ansible roles from galaxy
-- elastic.elasticsearch,7.13.4
+- elastic.elasticsearch,v7.13.4
 - geerlingguy.kibana,4.0.1
diff --git a/roles/postgresql/tasks/main.yml b/roles/postgresql/tasks/main.yml
index c610aad..da71805 100644
--- a/roles/postgresql/tasks/main.yml
+++ b/roles/postgresql/tasks/main.yml
@@ -10,15 +10,15 @@
 
 - name: installing dependencies
   apt:
-    pkg: ['postgis', 'postgresql-11-postgis-2.5', 'postgresql-11-postgis-2.5-scripts', 'postgresql-contrib']
+    pkg: ['postgis', 'postgresql-13-postgis-3', 'postgresql-13-postgis-3-scripts', 'postgresql-contrib']
     state: present
     update_cache: yes
 # postgresql-11-postgis-2.5-scripts #for postgis.control
 # postgresql-contrib #for dblink extension
 
-- name: install python-psycopg2 for ansible psql modules
+- name: install python3-psycopg2 for ansible psql modules
   apt:
-    name: python-psycopg2
+    name: python3-psycopg2
     state: present
 
 - name: create georchestra user

Note: the requirements.yaml with the version prefixed with a 'v' will probably be needed on previous debian version as well. Please also note that I haven't tested at runtime yet.

@landryb
Copy link
Member

landryb commented Aug 30, 2021

i'm fine with the change, dunno if we need to still support buster (in which case the list of pkgs for postgis needs to be a variable or a when based on ansible_distribution_version)

@pmauduit
Copy link
Member Author

I was wondering indeed, from my PoV, I think it won't be a hassle to manage both versions

@pmauduit
Copy link
Member Author

PR above has been merged, closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants