From a4367de16c3fd0c81a4bd956e755c7a4eafa1def Mon Sep 17 00:00:00 2001
From: A Holt <holta@users.noreply.github.com>
Date: Mon, 14 Oct 2019 12:46:24 -0400
Subject: [PATCH] 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: