!LOCAL Add makefile for automating build and upload

Add entrypoint that calls envsubst ahead of starting nginx

Makefile

@@ -0,0 +1,20 @@
+REPOSITORY_PROXY   = vcs.enp.one/skylab/nxcloud-proxy
+REPOSITORY_SERVER  = vcs.enp.one/skylab/nxcloud-server
+
+
+.PHONY: help docs
+# source: https://marmelab.com/blog/2016/02/29/auto-documented-makefile.html
+help: ## List Makefile targets
+	$(info Makefile documentation)
+	@grep -E '^[a-zA-Z_-]+:.*?## .*$$' $(MAKEFILE_LIST) | sort | awk 'BEGIN {FS = ":.*?## "}; {printf "\033[36m%-10s\033[0m %s\n", $$1, $$2}'
+
+
+image:  ## Build image
+	podman build ./php-fpm --tag $(REPOSITORY_SERVER):latest
+	podman build ./nginx --tag $(REPOSITORY_PROXY):latest
+
+push: image  ## Build and publish image
+	podman login $(shell echo $(REPOSITORY_SERVER) | cut -d '/' -f 1)
+	podman push $(REPOSITORY_SERVER):latest
+	podman login $(shell echo $(REPOSITORY_PROXY) | cut -d '/' -f 1)
+	podman push $(REPOSITORY_PROXY):latest

README.md

@@ -61,6 +61,12 @@ Simply run this command from the root of the cloned repo:
 
 this will take a while.
 
+### Building nginx
+This will embed an optimized configuration for serving Nextcloud files and PHP-FPM resources. Run this
+command from the root of the cloned repo:
+
+`docker compose build nginx`
+
 ### Installing Nextcloud
 
 Run `docker compose up -d`. If something doesn't work try debugging it yourself of open an issue with the php-fpm and nginx logs attached.
@@ -82,9 +88,6 @@ Edit `/your/nextcloud/root/nextcloud/config/config.php` and add the following op
 ),
 ```
 
-### Editing nginx.conf
-You may also have to replace `example.com` with your own domain or multiple domains in the nginx.conf file.
-
 ### Enabling system cron (optional)
 Nextcloud must perform background tasks. The best way to do that is to use cron. However, on docker this is not easily doable. Here the host will perform the cronjobs required.
 

docker-compose.yml

@@ -2,55 +2,71 @@
 services:
 
   nginx:
-    container_name: nginx-nextcloud
+    build: ./nginx
-    image: nginx:latest
     ports:
-      - 80:80
+      - 8080:80
       - 443:443
     volumes:
-      - ${NEXTCLOUD_DIR}:/var/www/html
+      - ./appdata/application:/var/www/html
-      - ${DATA_DIR}:/data
+      - ./appdata/data:/data
-      - ./nginx.conf:/etc/nginx/nginx.conf:ro
+    environment:
+      - NEXTCLOUD_PHP_FPM_HOST=nxcloud-server-1:9000
+      - NEXTCLOUD_DOMAIN=localhost
+      - NEXTCLOUD_MAX_UPLOAD_SIZE=4G
     networks:
       - nextcloud
     depends_on:
-      - php-fpm-nextcloud
+      - server
-      - redis-nextcloud
+      - redis
-      - mariadb-nextcloud
+      - mariadb
 
-  php-fpm-nextcloud:
+  server:
-    container_name: php-fpm-nextcloud
+    build:
-    build: ./php-fpm
+      context: ./php-fpm
+      tags:
+        - vcs.enp.one/skylab/nextcloud-server:latest
     volumes:
-      - ${NEXTCLOUD_DIR}:/var/www/html
+      - ./appdata/application:/var/www/html
-      - ${DATA_DIR}:/data
+      - ./appdata/data:/data
     networks:
       - nextcloud
 
-  mariadb-nextcloud:
+  mariadb:
-    container_name: mariadb-nextcloud
     image: mariadb
     command: --transaction-isolation=READ-COMMITTED --binlog-format=ROW
-    restart: always
     volumes:
-      - ./mariadb:/var/lib/mysql
+      - ./appdata/database:/var/lib/mysql
     environment:
-      - MYSQL_ROOT_PASSWORD=${MARIADB_ROOT_PASS}
+      - MARIADB_RANDOM_ROOT_PASSWORD=true
-      - MYSQL_PASSWORD=${MARIADB_PASS}
+      - MYSQL_PASSWORD=nextcloud
       - MYSQL_DATABASE=nextcloud
       - MYSQL_USER=nextcloud
     networks:
       - nextcloud
 
-  redis-nextcloud:
+  redis:
-    container_name: redis-nextcloud
-#    image: redis:latest
-# keydb is a fork and drop-in replacement for Redis
     image: eqalpha/keydb
-    restart: unless-stopped
     networks:
       - nextcloud
 
+  cron:
+    image: vcs.enp.one/skylab/nextcloud-server:latest
+    command:
+      - -c
+      - "'while true; do php --define apc.enable_cli=1 /var/www/html/cron.php; sleep 300; done'"
+    networks:
+      - nextcloud
+    user: www-data
+    entrypoint:
+      - /bin/bash
+    volumes:
+      - ./appdata/application:/var/www/html
+      - ./appdata/data:/data
+    depends_on:
+      - server
+      - redis
+      - mariadb
+
 networks:
   nextcloud:
     driver: bridge

nginx/Dockerfile

@@ -0,0 +1,10 @@
+FROM docker.io/library/nginx:latest
+
+ENV NEXTCLOUD_DOMAIN=example.com
+ENV NEXTCLOUD_PHP_FPM_HOST=server:9000
+ENV NEXTCLOUD_MAX_UPLOAD_SIZE=512M
+
+ADD nginx.conf.template /nginx.conf.template
+ADD docker-entrypoint.sh /docker-entrypoint.sh
+
+ENTRYPOINT ["sh", "-c", "/docker-entrypoint.sh"]

nginx/docker-entrypoint.sh

@@ -0,0 +1,16 @@
+#/usr/bin/env bash
+
+# This implementation is adapted from the solution proposed here:
+# https://stackoverflow.com/questions/21866477/nginx-use-environment-variables
+
+# envsubst will take the input and replace all references to environment
+# variables with their corresponding value. Because nginx uses the same
+# '$' prefix for its internal variables, we should explicitly define the
+# variables we want to replace rather than replacing all env vars.
+envsubst '
+$NEXTCLOUD_PHP_FPM_HOST
+$NEXTCLOUD_DOMAIN
+$NEXTCLOUD_MAX_UPLOAD_SIZE
+' < /nginx.conf.template > /etc/nginx/nginx.conf
+
+exec nginx -g 'daemon off;'

nginx/nginx.conf.template

@@ -2,7 +2,7 @@ events { worker_connections 1024; }
 http {
 
     upstream php-handler {
-        server php-fpm-nextcloud:9000;
+        server ${NEXTCLOUD_PHP_FPM_HOST};
         #server unix:/run/php/php8.2-fpm.sock;
     }
 
@@ -16,7 +16,7 @@ http {
         listen 80;
         listen [::]:80;
         # INFO: Set this to your domain
-        server_name example.com;
+        server_name ${NEXTCLOUD_DOMAIN};
 
         # Prevent nginx HTTP Server Detection
         server_tokens off;
@@ -34,7 +34,7 @@ http {
         add_header Strict-Transport-Security "max-age=15768000; includeSubDomains; preload" always;
 
         # set max upload size and increase upload timeout:
-        client_max_body_size 512M;
+        client_max_body_size ${NEXTCLOUD_MAX_UPLOAD_SIZE};
         client_body_timeout 300s;
         fastcgi_buffers 64 4K;
 
@@ -71,10 +71,10 @@ http {
         # and include that list explicitly or add the file extension
         # only for Nextcloud like below:
         include mime.types;
-        types {
+        #types {
-            text/javascript js mjs;
+        #    text/javascript js mjs;
-            application/wasm wasm;
+        #    application/wasm wasm;
-        }
+        #}
 
         # Specify how to handle directories -- specifying `/index.php$request_uri`
         # here as the fallback means that Nginx always exhibits the desired behaviour