From fc8d47876079d9463ad22aa3007c26084ca15325 Mon Sep 17 00:00:00 2001
From: barcus <barcus@users.noreply.github.com>
Date: Sat, 26 Mar 2022 12:44:42 +0100
Subject: [PATCH] db_init option (#2)

* add DB_INIT option
* update readme
* general improvment
---
 README.md            | 20 ++++++++++----------
 docker-entrypoint.sh | 21 ++++++++++-----------
 2 files changed, 20 insertions(+), 21 deletions(-)

diff --git a/README.md b/README.md
index c63f37c..f3dff27 100644
--- a/README.md
+++ b/README.md
@@ -31,23 +31,23 @@ Either way, the command is run with postgres system user.
 After successful upgrade, new database is available in `<pg_new_data>`
 folder. In any case source database is not modified.
 
-Docker volumes required:
+PostgreSQL version, database encoding and locale are discovered from
+source database.
 
-* `/data/pg-old` (should contain source db)
-* `/data/pg-new` (will contain target db)
+:warning: Be sure to set PGUSER var with an existing superuser in source DB
 
 See example below.
 
-Variables:
+Docker volumes required:
 
-* PGUSER=postgres (should exist in source database)
+* `/data/pg-old` (should contain source db)
+* `/data/pg-new` (will contain target db)
 
-Optional:
+Variables:
 
-* PG_NEW=14 (default v14)
-* PG_OLD (discovered from source PG_VERSION file)
-* ENCODING (discovered from source $PGUSER database)
-* LOCALE (discovered from source $PGUSER database)
+* PGUSER (default postgres)
+* PG_NEW (default 14)
+* DB_INIT (default true)
 
 ### Example
 
diff --git a/docker-entrypoint.sh b/docker-entrypoint.sh
index fca79ee..ca6c011 100644
--- a/docker-entrypoint.sh
+++ b/docker-entrypoint.sh
@@ -1,17 +1,14 @@
 #!/usr/bin/env bash
 #set -x
 
-if [ -z "${PGUSER}" ] ; then
-  echo 'ERROR: var PGUSER required'
-  exit 1
-fi 
-
-[ -z "${PG_NEW}" ] && PG_NEW=14
-
+: "${PGUSER:=postgres}"
+: "${PG_NEW:=14}"
+: "${DB_INIT:=true}"
 PG_OLD=$(cat /pg_old/data/PG_VERSION)
+
 re='^[0-9]+([.][0-9]+)?$'
 if ! [[ ${PG_OLD} =~ $re ]] ; then
-   echo "ERROR: /pg_old/data/PG_VERSION file not found" >&2; exit 1
+   echo "ERROR: version not found in file /pg_old/data/PG_VERSION" >&2; exit 1
 fi
 
 export PGBINOLD=/usr/lib/postgresql/${PG_OLD}/bin
@@ -61,9 +58,11 @@ export LOCALE=$(echo ${LOCALE}| xargs)
 eval "${PGBINOLD}/pg_ctl -D ${PGDATAOLD} -l logfile stop"
 
 # Init DB PG_NEW  
-[ -z "${ENCODING}" ] && ENCODING="SQL_ASCII"
-[ -z "${LOCALE}" ] && LOCALE="en_US.utf8"
-eval "${PGBINNEW}/initdb --username=${PGUSER} --pgdata=${PGDATANEW} --encoding=${ENCODING} --lc-collate=${LOCALE} --lc-ctype=${LOCALE}"
+if [ "${DB_INIT}" == 'true' ]; then
+  ENCODING="${ENCODING:=SQL_ASCII}"
+  LOCALE="${LOCALE:=en_US.utf8}"
+  eval "${PGBINNEW}/initdb --username=${PGUSER} --pgdata=${PGDATANEW} --encoding=${ENCODING} --lc-collate=${LOCALE} --lc-ctype=${LOCALE}"
+fi
 
 # run pg_upgrade or launch CMD
 if [ "$1" = 'pg_upgrade' ] ;then