Merge pull request #11 from timlinux/master

Added link to docker hub page in readme

.gitignore

@@ -0,0 +1,2 @@
+.idea
+*.*~

Dockerfile

@@ -20,7 +20,6 @@ RUN apt-get -y install ca-certificates rpl pwgen
 
 # Next line a workaround for https://github.com/dotcloud/docker/issues/963
 RUN apt-get install -y postgresql-9.3-postgis-2.1
-RUN service postgresql start && /bin/su postgres -c "createuser -d -s -r -l docker" && /bin/su postgres -c "psql postgres -c \"ALTER USER docker WITH ENCRYPTED PASSWORD 'docker'\"" && service postgresql stop
 
 # Start with supervisor
 ADD postgres.conf /etc/supervisor/conf.d/postgres.conf
@@ -38,7 +37,4 @@ RUN /setup.sh
 ADD start-postgis.sh /start-postgis.sh
 RUN chmod 0755 /start-postgis.sh
 
-ADD perms_wrapper.sh /perms_wrapper.sh
-RUN chmod 0755 /perms_wrapper.sh
-
-CMD /perms_wrapper.sh
+CMD /start-postgis.sh

README.md

@@ -2,6 +2,8 @@
 
 A simple docker container that runs PostGIS
 
+Visit our page on the docker hub at: https://registry.hub.docker.com/u/kartoza/postgis/
+
 There are a number of other docker postgis containers out there. This one
 differentiates itself by:
 

perms_wrapper.sh

@@ -1,15 +0,0 @@
-#!/bin/bash
-
-# /etc/ssl/private can't be accessed from within container for some reason
-# (@andrewgodwin says it's something AUFS related)  - taken from https://github.com/orchardup/docker-postgresql/blob/master/Dockerfile
-cp -r /etc/ssl /tmp/ssl-copy/
-chmod -R 0700 /etc/ssl
-chown -R postgres /tmp/ssl-copy
-rm -r /etc/ssl
-mv /tmp/ssl-copy /etc/ssl
-
-# needs to be done as root:
-chown -R postgres:postgres /var/lib/postgresql
-
-# everything else needs to be done as non-root (i.e. postgres)
-sudo -u postgres /start-postgis.sh

start-postgis.sh

@@ -8,15 +8,26 @@ POSTGRES="/usr/lib/postgresql/9.3/bin/postgres"
 INITDB="/usr/lib/postgresql/9.3/bin/initdb"
 SQLDIR="/usr/share/postgresql/9.3/contrib/postgis-2.1/"
 
+# /etc/ssl/private can't be accessed from within container for some reason
+# (@andrewgodwin says it's something AUFS related)  - taken from https://github.com/orchardup/docker-postgresql/blob/master/Dockerfile
+cp -r /etc/ssl /tmp/ssl-copy/
+chmod -R 0700 /etc/ssl
+chown -R postgres /tmp/ssl-copy
+rm -r /etc/ssl
+mv /tmp/ssl-copy /etc/ssl
+
+
 # test if DATADIR is existent
 if [ ! -d $DATADIR ]; then
   echo "Creating Postgres data at $DATADIR"
   mkdir -p $DATADIR
 fi
+# needs to be done as root:
+chown -R postgres:postgres $DATADIR
 
 # Note that $USERNAME and $PASS below are optional paramters that can be passed
 # via docker run e.g.
-#docker run --name="postgis" -e USERNAME=qgis -e PASS=qgis -d -v 
+#docker run --name="postgis" -e USERNAME=qgis -e PASS=qgis -d -v
 #/var/docker-data/postgres-dat:/var/lib/postgresql -t qgis/postgis:6
 
 # If you dont specify a user/password in docker run, we will generate one
@@ -30,33 +41,33 @@ if [ ! "$(ls -A $DATADIR)" ]; then
   # Initialise db
   echo "Initializing Postgres Database at $DATADIR"
   #chown -R postgres $DATADIR
-  $INITDB $DATADIR
+  su - postgres -c "$INITDB $DATADIR"
 fi
 
 # Make sure we have a user set up
 if [ -z "$USERNAME" ]; then
   USERNAME=docker
-fi  
+fi
 if [ -z "$PASS" ]; then
   PASS=docker
-fi  
+fi
 # redirect user/pass into a file so we can echo it into
 # docker logs when container starts
 # so that we can tell user their password
 echo "postgresql user: $USERNAME" > /tmp/PGPASSWORD.txt
 echo "postgresql password: $PASS" >> /tmp/PGPASSWORD.txt
-$POSTGRES --single -D $DATADIR -c config_file=$CONF <<< "CREATE USER $USERNAME WITH SUPERUSER ENCRYPTED PASSWORD '$PASS';"
+su - postgres -c "$POSTGRES --single -D $DATADIR -c config_file=$CONF <<< \"CREATE USER $USERNAME WITH SUPERUSER ENCRYPTED PASSWORD '$PASS';\""
 
 trap "echo \"Sending SIGTERM to postgres\"; killall -s SIGTERM postgres" SIGTERM
 
-$POSTGRES -D $DATADIR -c config_file=$CONF &
+su - postgres -c "$POSTGRES -D $DATADIR -c config_file=$CONF &"
 
 # Wait for the db to start up before trying to use it....
 
 sleep 10
 
-RESULT=`psql -l | grep postgis | wc -l`
-if [[ $RESULT == '1' ]]
+RESULT=`su - postgres -c "psql -l | grep postgis | wc -l"`
+if [[ ${RESULT} == '1' ]]
 then
     echo 'Postgis Already There'
 else
@@ -64,29 +75,33 @@ else
     # Note the dockerfile must have put the postgis.sql and spatialrefsys.sql scripts into /root/
     # We use template0 since we want t different encoding to template1
     echo "Creating template postgis"
-    createdb template_postgis -E UTF8 -T template0
+    su - postgres -c "createdb template_postgis -E UTF8 -T template0"
     echo "Enabling template_postgis as a template"
-    psql template1 -c "UPDATE pg_database SET datistemplate = TRUE WHERE datname = 'template_postgis';"
+    CMD="UPDATE pg_database SET datistemplate = TRUE WHERE datname = 'template_postgis';"
+    su - postgres -c "$CMD"
     echo "Loading postgis.sql"
-    psql template_postgis -f $SQLDIR/postgis.sql
+    su - postgres -c "psql template_postgis -f $SQLDIR/postgis.sql"
     echo "Loading spatial_ref_sys.sql"
-    psql template_postgis -f $SQLDIR/spatial_ref_sys.sql
+    su - postgres -c "psql template_postgis -f $SQLDIR/spatial_ref_sys.sql"
 
     # Needed when importing old dumps using e.g ndims for constraints
     echo "Loading legacy sql"
-    psql template_postgis -f $SQLDIR/legacy_minimal.sql
-    psql template_postgis -f $SQLDIR/legacy_gist.sql
+    su - postgres -c "psql template_postgis -f $SQLDIR/legacy_minimal.sql"
+    su - postgres -c "psql template_postgis -f $SQLDIR/legacy_gist.sql"
     echo "Granting on geometry columns"
-    psql template_postgis -c 'GRANT ALL ON geometry_columns TO PUBLIC;'
+    su - postgres -c "psql template_postgis -c 'GRANT ALL ON geometry_columns TO PUBLIC;'"
     echo "Granting on geography columns"
-    psql template_postgis -c 'GRANT ALL ON geography_columns TO PUBLIC;'
+    su - postgres -c "psql template_postgis -c 'GRANT ALL ON geography_columns TO PUBLIC;'"
     echo "Granting on spatial ref sys"
-    psql template_postgis -c 'GRANT ALL ON spatial_ref_sys TO PUBLIC;'
+    su - postgres -c "psql template_postgis -c 'GRANT ALL ON spatial_ref_sys TO PUBLIC;'"
     # Create a default db called 'gis' that you can use to get up and running quickly
     # It will be owned by the docker db user
-    createdb -O docker -T template_postgis gis
+    su - postgres -c "createdb -O docker -T template_postgis gis"
 fi
 # This should show up in docker logs afterwards
-psql -l
+su - postgres -c "psql -l"
 
-wait $!
+PID=`cat /var/run/postgresql/9.3-main.pid`
+kill -9 ${PID}
+echo "Postgres initialisation process completed .... restarting in foreground"
+su - postgres -c "$POSTGRES -D $DATADIR -c config_file=$CONF"