osmith submitted this change.

View Change

Approvals: Jenkins Builder: Verified osmith: Verified pespin: Looks good to me, but someone else must approve laforge: Looks good to me, approved fixeria: Looks good to me, but someone else must approve
fix_perms: use debian:bullseye

Use debian:bullseye instead of debian-bullseye-build. Building the
latter requires a lot of time, it's definitively not needed for just
running chmod. Use debian:bullseye, since we use it as base for most
images and it's therefore very likely that it's already downloaded
(and if not, it would get downloaded here).

Change-Id: I88f513a0a8ee04628842a7aa1b663c390d2032cd
---
M jenkins-common.sh
1 file changed, 1 insertion(+), 5 deletions(-)

diff --git a/jenkins-common.sh b/jenkins-common.sh
index b393b05..33b1a67 100644
--- a/jenkins-common.sh
+++ b/jenkins-common.sh
@@ -238,16 +238,12 @@
}

fix_perms() {
- if ! docker_image_exists "debian-bullseye-build"; then
- docker_images_require "debian-bullseye-build"
- fi
-
echo Fixing permissions
docker run --rm \
-v $VOL_BASE_DIR:/data \
-v $CACHE_DIR:/cache \
--name ${BUILD_TAG}-cleaner \
- $REPO_USER/debian-bullseye-build \
+ debian:bullseye \
chmod -R a+rX /data/ /cache/
}


1 is the latest approved patch-set. No files were changed between the latest approved patch-set and the submitted one.

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

Gerrit-Project: docker-playground
Gerrit-Branch: master
Gerrit-Change-Id: I88f513a0a8ee04628842a7aa1b663c390d2032cd
Gerrit-Change-Number: 28903
Gerrit-PatchSet: 2
Gerrit-Owner: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged