[S] Change in docker-playground[master]: jenkins-common: fix_perms: chown files too

osmith has submitted this change. ( https://gerrit.osmocom.org/c/docker-playground/+/36512?usp=email ) Change subject: jenkins-common: fix_perms: chown files too ...................................................................... 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(-) Approvals: laforge: Looks good to me, but someone else must approve Jenkins Builder: Verified pespin: Looks good to me, approved 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 https://gerrit.osmocom.org/c/docker-playground/+/36512?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: docker-playground Gerrit-Branch: master Gerrit-Change-Id: I43af48bd87e27abad5ad52e734256d78651f67c3 Gerrit-Change-Number: 36512 Gerrit-PatchSet: 4 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-MessageType: merged