fix: perms and dir structure for physical backups (#650)

Co-authored-by: Paul Cioanca <[email protected]>

Author: darora

docker/all-in-one/Dockerfile

@@ -221,7 +221,7 @@ RUN useradd --create-home --shell /bin/bash postgrest && \
 RUN mkdir -p /etc/wal-g && \
     chown -R adminapi:adminapi /etc/wal-g && \
     chmod g+w /etc/wal-g
-RUN mkdir -p /var/log/wal-g && chmod +x /usr/bin/admin-mgr && chmod ug+s /usr/bin/admin-mgr && touch /etc/wal-g/config.json && chown adminapi:adminapi /etc/wal-g/config.json
+RUN mkdir -p /var/log/wal-g && chown -R postgres:postgres /var/log/wal-g && chmod +x /usr/bin/admin-mgr && chmod ug+s /usr/bin/admin-mgr && touch /etc/wal-g/config.json && chown adminapi:adminapi /etc/wal-g/config.json
 RUN chown -R adminapi:adminapi /etc/adminapi
 
 # Add healthcheck and entrypoint scripts

docker/all-in-one/entrypoint.sh

@@ -119,14 +119,16 @@ fi
 
 # Update pgdata directory
 if [ "${PGDATA_REAL:-}" ]; then
-  mkdir -p "${PGDATA_REAL}"
-  chown -R postgres:postgres "${PGDATA_REAL}"
+    mkdir -p "${PGDATA_REAL}"
+    chown -R postgres:postgres "${PGDATA_REAL}"
+    chmod g+rx "${PGDATA_REAL}"
 fi
 
 if [ "${PGDATA:-}" ]; then
   if [ "${PGDATA_REAL:-}" ]; then
     mkdir -p "$(dirname "${PGDATA}")"
-    ln -s "${PGDATA}" "${PGDATA_REAL}"
+    rm -rf "${PGDATA}"
+    ln -s "${PGDATA_REAL}" "${PGDATA}"
   else
     mkdir -p "$PGDATA"
     chown postgres:postgres "$PGDATA"