Merge pull request #10 from uclahs-cds/ts_docker_permissions

Updated docker permissions
uclahs-cds · Mar 5, 2021 · 74f0e4a · 74f0e4a
2 parents d9b1072 + dbe12e8
commit 74f0e4a
Showing 1 changed file with 8 additions and 4 deletions.
diff --git a/pipeline/config/methods.config b/pipeline/config/methods.config
@@ -105,7 +105,11 @@ params {
 // Enable docker
 docker {
     enabled = true
-    sudo = params.sge_scheduler // Set to true if run on SGE
-    runOptions = "-u \$(id -u):\$(id -g)"
-    runOptions = "\$(for i in `id --real --groups`; do echo -n \"--group-add=\$i \"; done)"
-}
+    sudo = (params.sge_scheduler) ? true : false // Set to true if run on SGE
+
+    // Pass user's UID/GID and group IDs to Docker
+    uid_and_gid = "-u \$(id -u):\$(id -g)"
+    all_group_ids = "\$(for i in `id --real --groups`; do echo -n \"--group-add=\$i \"; done)"
+
+    runOptions = "${uid_and_gid} ${all_group_ids}"
+}