osmith submitted this change.

View Change

jenkins-common: fix_perms: chown files too

Ensure the user that runs docker owns files in cache and data dirs
during fix_perms, if an old job created them as root.

Change-Id: I43af48bd87e27abad5ad52e734256d78651f67c3
---
M jenkins-common.sh
1 file changed, 16 insertions(+), 1 deletion(-)

diff --git a/jenkins-common.sh b/jenkins-common.sh
index 59224a0..f22158b 100644
--- a/jenkins-common.sh
+++ b/jenkins-common.sh
@@ -386,7 +386,10 @@
 			-v $CACHE_DIR:/cache \
 			--name ${BUILD_TAG}-cleaner \
 			"debian:$DEBIAN_DEFAULT" \
-			chmod -R a+rX /data/ /cache/
+			sh -e -x -c "
+				chmod -R a+rX /data/ /cache/
+				chown -R $(id -u):$(id -g) /data /cache
+			"
 }
 
 collect_logs() {

To view, visit change 36512. To unsubscribe, or for help writing mail filters, visit settings.