From 5aca56d2ddafad347dd670f4004ebf367c2c84ef Mon Sep 17 00:00:00 2001
From: Bo Lorentsen <druppy@users.noreply.github.com>
Date: Fri, 11 Mar 2022 05:29:51 +0100
Subject: [PATCH] docker: check "/data" mount ownership before forcing it
 (#6553)

Co-authored-by: bl <bl@moch.dk>
---
 docker/start.sh | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/docker/start.sh b/docker/start.sh
index b37acfe27..0ac47da93 100755
--- a/docker/start.sh
+++ b/docker/start.sh
@@ -31,8 +31,12 @@ cleanup() {
 }
 
 create_volume_subfolder() {
-    # Modify the owner of /data dir, make $USER(git) user have permission to create sub-dir in /data.
-    chown -R "$USER:$USER" /data
+    # only change ownership if needed, if using an nfs mount this could be expensive
+    if [ "$USER:$USER" != "$(stat /data -c '%U:%G')" ]
+    then
+        # Modify the owner of /data dir, make $USER(git) user have permission to create sub-dir in /data.
+        chown -R "$USER:$USER" /data
+    fi
 
     # Create VOLUME subfolder
     for f in /data/gogs/data /data/gogs/conf /data/gogs/log /data/git /data/ssh; do