fix: harden PostgreSQL first-run initialization

Bump the packaging revision to mldm2 and move partial PGDATA directories aside before initdb so interrupted first starts do not leave an unusable cluster.

Author: mildman1848

Dockerfile

@@ -6,8 +6,8 @@ FROM ghcr.io/linuxserver/baseimage-alpine:${LSIO_BASE_VERSION}
 ARG LSIO_BASE_VERSION
 ARG BUILD_DATE
 ARG APP_VERSION=16.14
-ARG IMAGE_REVISION=mldm1
-ARG VERSION=16.14-mldm1
+ARG IMAGE_REVISION=mldm2
+ARG VERSION=16.14-mldm2
 ARG VCS_REF
 
 LABEL build_version="Mildman1848 PostgreSQL version:- ${VERSION} Upstream:- ${APP_VERSION} Revision:- ${IMAGE_REVISION} Build-date:- ${BUILD_DATE}" \

Makefile

@@ -3,7 +3,7 @@ SHELL := /usr/bin/env bash
 
 IMAGE_NAME ?= postgresql
 APP_VERSION ?= 16.14
-IMAGE_REVISION ?= mldm1
+IMAGE_REVISION ?= mldm2
 VERSION ?= $(APP_VERSION)-$(IMAGE_REVISION)
 IMAGE_TAG ?= $(VERSION)
 REGISTRY ?= ghcr.io/mildman1848

root/usr/local/bin/start-app

@@ -12,10 +12,18 @@ sql_escape() {
   printf "%s" "${value//\'/\'\'}"
 }
 
-mkdir -p "$PGDATA" /run/postgresql
+mkdir -p "$(dirname "$PGDATA")" /run/postgresql
 chown -R abc:abc /config /run/postgresql
 
 if [[ ! -s "$PGDATA/PG_VERSION" ]]; then
+  if [[ -d "$PGDATA" ]] && find "$PGDATA" -mindepth 1 -maxdepth 1 | grep -q .; then
+    failed_dir="${PGDATA}.failed-init.$(date -u +%Y%m%d%H%M%S)"
+    echo "Warning: $PGDATA exists but has no PG_VERSION; moving partial init to $failed_dir" >&2
+    mv "$PGDATA" "$failed_dir"
+  fi
+
+  mkdir -p "$PGDATA"
+  chown -R abc:abc "$PGDATA"
   echo "Initializing PostgreSQL data directory at $PGDATA"
   s6-setuidgid abc initdb -D "$PGDATA" --username="$POSTGRES_USER"