From a4367de16c3fd0c81a4bd956e755c7a4eafa1def Mon Sep 17 00:00:00 2001 From: A Holt Date: Mon, 14 Oct 2019 12:46:24 -0400 Subject: [PATCH 1/3] Clarify Kolibri installation / migration requirements, per @benjaoming --- roles/kolibri/tasks/main.yml | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/roles/kolibri/tasks/main.yml b/roles/kolibri/tasks/main.yml index e3fdc8e9b..6415d7222 100644 --- a/roles/kolibri/tasks/main.yml +++ b/roles/kolibri/tasks/main.yml @@ -25,23 +25,27 @@ group: root mode: 0755 +# 2019-10-14: An alternative might be to put KOLIBRI_USER="kolibri" into +# /etc/kolibri/conf.d/iiab.conf - name: Save kolibri_user ({{ kolibri_user }}) to /etc/kolibri/username copy: - content: "{{ kolibri_user }}" + content: "{{ kolibri_user }}" # i.e. kolibri dest: /etc/kolibri/username owner: root group: root mode: 0644 - + +# 2019-10-14: An alternative mentioned by @benjaoming (Benjamin Bach) would be +# to put KOLIBRI_HOME="/library/kolibri" into /etc/kolibri/conf.d/iiab.conf - name: Save kolibri_home (KOLIBRI_HOME="{{ kolibri_home }}") to /etc/kolibri/daemon.conf copy: - content: 'KOLIBRI_HOME="{{ kolibri_home }}"' + content: 'KOLIBRI_HOME="{{ kolibri_home }}"' # i.e. /library/kolibri dest: /etc/kolibri/daemon.conf owner: root group: root mode: 0644 -- name: apt install latest Kolibri .deb from {{ kolibri_deb_url }} (populates {{ kolibri_home }}, migrates database) # i.e. /library/kolibri +- name: apt install latest Kolibri .deb from {{ kolibri_deb_url }} (populates {{ kolibri_home }} / migrates database, based on params set in /etc/kolibri) apt: deb: "{{ kolibri_deb_url }}" # https://learningequality.org/r/kolibri-deb-latest environment: From 6bbfde4012407b104037d20d00a2991597e42b5e Mon Sep 17 00:00:00 2001 From: A Holt Date: Mon, 14 Oct 2019 12:54:29 -0400 Subject: [PATCH 2/3] Is "chmod -R kolibri:www-data /library/kolibri" nec? --- roles/kolibri/tasks/main.yml | 2 ++ 1 file changed, 2 insertions(+) diff --git a/roles/kolibri/tasks/main.yml b/roles/kolibri/tasks/main.yml index 6415d7222..360589266 100644 --- a/roles/kolibri/tasks/main.yml +++ b/roles/kolibri/tasks/main.yml @@ -100,6 +100,8 @@ become_user: "{{ kolibri_user }}" when: kolibri_provision | bool +# 2019-10-14: QUESTION FOR @benjaoming...is this stanza really necessary +# (shouldn't migration & provisiondevice above be sufficient?) - name: chown -R {{ kolibri_user }}:{{ apache_user }} {{ kolibri_home }} for good measure? file: path: "{{ kolibri_home }}" # /library/kolibri From c11495951dfb973b4b2c27cabc6a0474504ad95a Mon Sep 17 00:00:00 2001 From: A Holt Date: Mon, 14 Oct 2019 14:12:17 -0400 Subject: [PATCH 3/3] Comment out chown stanza, per @benjaoming --- roles/kolibri/tasks/main.yml | 18 +++++++++--------- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/roles/kolibri/tasks/main.yml b/roles/kolibri/tasks/main.yml index 360589266..5c2ccd757 100644 --- a/roles/kolibri/tasks/main.yml +++ b/roles/kolibri/tasks/main.yml @@ -100,15 +100,15 @@ become_user: "{{ kolibri_user }}" when: kolibri_provision | bool -# 2019-10-14: QUESTION FOR @benjaoming...is this stanza really necessary -# (shouldn't migration & provisiondevice above be sufficient?) -- name: chown -R {{ kolibri_user }}:{{ apache_user }} {{ kolibri_home }} for good measure? - file: - path: "{{ kolibri_home }}" # /library/kolibri - owner: "{{ kolibri_user }}" # kolibri - group: "{{ apache_user }}" # www-data (on Debian/Ubuntu/Raspbian) - recurse: yes - when: kolibri_provision | bool +# 2019-10-14: This stanza should not be necessary according to @benjaoming +# (Benjamin Bach) especially as migration & provisiondevice were run above. +#- name: chown -R {{ kolibri_user }}:{{ apache_user }} {{ kolibri_home }} for good measure? +# file: +# path: "{{ kolibri_home }}" # /library/kolibri +# owner: "{{ kolibri_user }}" # kolibri +# group: "{{ apache_user }}" # www-data (on Debian/Ubuntu/Raspbian) +# recurse: yes +# when: kolibri_provision | bool # 2019-10-07: Moved to roles/httpd/tasks/main.yml