Merge pull request #246 from turbo124/master

Fixes for crons and folder permissions

alpine/Dockerfile_v5

@@ -53,6 +53,7 @@ RUN set -eux; \
     libzip-dev \
     oniguruma-dev \
     git \
+    busybox-suid \
     zip; \
     docker-php-ext-configure gd --with-freetype --with-jpeg; \
     docker-php-ext-install -j$(nproc) \
@@ -95,15 +96,10 @@ RUN addgroup -S "$INVOICENINJA_USER" && \
     addgroup "$INVOICENINJA_USER" www-data; \
     chown -R "$INVOICENINJA_USER":"$INVOICENINJA_USER" /var/www/app
 
+
 # Install Composer
 RUN curl -sS https://getcomposer.org/installer | php -- --install-dir=/usr/local/bin --filename=composer; 
 
-## Set up the cronjob and run cron daemon
-RUN echo "* * * * * run-parts /etc/periodic/1min" >> /etc/crontabs/root
-COPY ./config/cron/cronjob_v5.sh /etc/periodic/1min/invoiceninja_cronjob
-RUN chown $INVOICENINJA_USER /etc/periodic/1min/invoiceninja_cronjob && \
-    crond -l 2 -b
-
 USER $INVOICENINJA_USER
 
 RUN /usr/local/bin/composer install --no-dev --no-suggest --no-progress --quiet

docker-compose.yml

@@ -42,6 +42,7 @@ services:
       - DB_DATABASE1=ninja
       - PHANTOMJS_PDF_GENERATION=false
       - SNAPPDF_EXECUTABLE_PATH="/usr/bin/chromium-browser"
+      - SNAPPDF_SKIP_DOWNLOAD=true
     volumes:
       # Configure your mounted directories, make sure the folder 'public' and 'storage'
       # exist, before mounting them
@@ -70,6 +71,24 @@ services:
     networks:
       - invoiceninja
 
+  # V5 crons only
+  cron:
+    image: invoiceninja/invoiceninja:5
+    volumes:
+      -  public:/var/www/app/public
+      -  storage:/var/www/app/storage
+    entrypoint: |
+      /bin/sh -c 'sh -s <<EOF
+      trap "break;exit" SIGHUP SIGINT SIGTERM
+      sleep 60s
+      while /bin/true; do
+        ./artisan schedule:run >> /dev/null 2>&1
+        sleep 60s
+      done
+      EOF'
+    networks:
+      - invoiceninja
+
   # THIS IS ONLY A VALID CONFIGURATION FOR IN 4. DO NOT USE FOR IN 5.
   # cron:
   #   image: invoiceninja/invoiceninja:alpine-4