Merge pull request #1228 from openml/bugfix/docker_writing_logs

Allow the docker user to write logs

Author: josvandervelde

docker/Dockerfile

@@ -4,16 +4,17 @@ RUN docker-php-source extract \
 	&& docker-php-ext-install mysqli \
 	&& docker-php-source delete
 
-# Java is necessery for the direct calls to the evaluation engine (in Api_splits.php)
+# Java is necessary for the direct calls to the evaluation engine (in Api_splits.php)
 RUN apt update \
     && apt install -y --no-install-recommends openjdk-11-jre 
 
 COPY . /var/www/openml
 RUN mv /var/www/openml/openml_OS/config/BASE_CONFIG-BLANK.php /var/www/openml/openml_OS/config/BASE_CONFIG.php
 
-RUN mkdir -p /var/www/openml/logs
-RUN mkdir /data
-
+RUN mkdir -p /var/www/openml/logs && \
+    mkdir -p /var/www/data/log && \
+    mkdir /data && \
+    chown -R www-data:www-data /var/www/data
 
 COPY docker/config/*.load /etc/apache2/mods-enabled/
 COPY docker/config/api.conf /etc/apache2/sites-enabled/000-default.conf