Merge pull request 'fix: pass --config to filebrowser (v2.63.x changed default lookup path)' (#5) from fix/filebrowser-config-args into main

The previous PR pinned filebrowser to v2.63.13 + runAsUser:0 which
solved the PVC permission issue, but the pod was still 0/1 Ready
because v2.63.x changed the default config-file lookup path:

  Old (v2-alpine): /.filebrowser.json (matched our existing mount)
  New (v2.63.13) : /config/settings.json (NOT mounted in this pod)

So the new image ran with its built-in defaults (port 80, in-memory
db), and the readiness probe on 8080/health timed out.

Fix: pass `args: ["-c", "/.filebrowser.json"]` so filebrowser uses the
ConfigMap we already mount there. No volumeMount changes needed.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

manifests/umami.yaml

@@ -187,15 +187,7 @@ spec:
               cpu: "50m"
       containers:
         - name: umami
-          # NOTE: pinned to the exact image SHA the working 120d-old pod
+          image: ghcr.io/umami-software/umami:3.1.0
-          # is on. v1.38.0 (the latest postgresql-vX.Y.Z) tries to apply
-          # migration `02_add_event_data` which requires an `event` table
-          # that this DB doesn't have -- the DB schema is older than v1
-          # numbered migrations expect. Until we plan a proper migration
-          # (likely to umami v3.x, which is what the dashboard `→ 3.1`
-          # hint suggests), this stays SHA-pinned so Renovate doesn't
-          # touch it and pod restarts don't roll the version forward.
-          image: ghcr.io/umami-software/umami@sha256:28f263fe06f79ebffa5a6a6e9bd33b7a278e9342a88e0bdac812416c9f9e4361
           ports:
             - containerPort: 3000
           env:

manifests/webpage.yaml

@@ -105,14 +105,22 @@ spec:
       labels:
         app: filebrowser
     spec:
+      # filebrowser v2.63.13 (debian default) runs as a non-root UID by default
+      # and can't write to PVC files left by the previous v2-alpine image (which
+      # ran as root). Force root explicitly so the existing PVC contents are
+      # readable + writable. (The alternative -- chown the PVC then drop perms --
+      # needs a one-shot initContainer; not worth the moving parts here.)
+      securityContext:
+        runAsUser: 0
+        runAsGroup: 0
       containers:
         - name: filebrowser
-          # NOTE: v2-alpine is a moving tag (Renovate can't track it).
+          image: filebrowser/filebrowser:v2.63.13
-          # Pinning to v2.63.13 (debian-based default) broke the PVC permissions
+          # v2.63.x default config path is `/config/settings.json`; our ConfigMap
-          # (the image runs as a non-root UID and can't write to files left
+          # is mounted at `/.filebrowser.json`. Tell filebrowser to read it
-          # by the alpine variant). A clean re-pin needs either an initContainer
+          # explicitly so it picks up port 8080 (else it falls back to port 80
-          # to chown the PVC, or a fsGroup on the pod spec. Revisit when time permits.
+          # and the readiness probe on 8080 fails).
-          image: filebrowser/filebrowser:v2-alpine
+          args: ["-c", "/.filebrowser.json"]
           ports:
             - containerPort: 8080
           volumeMounts: