osmith has submitted this change. ( https://gerrit.osmocom.org/c/docker-playground/+/38227?usp=email )

Change subject: debian-bookworm-titan: chown 1000 for deps ......................................................................

debian-bookworm-titan: chown 1000 for deps

Change the ownership of the "deps" directory to a not yet created user with the ID 1000. This user gets created when the gerrit verifications use the container for building. Changing the ownership of the deps directory allows using them and updating them as needed (e.g. if the patch in review changes a commit of one of the deps).

Chowning here once results in faster jobs, compared to if we did it every time the gerrit verifications job ran.

Creating the user here would result in conflicts with other containers depending on debian-bookworm-titan that may create their own user with the same UID.

Related: osmo-ci I4bea336097e8180b19b3afa41c0b984d93c21bd1 Change-Id: Id51885d0929f4966b1a4a181b3453c06d7956b53 --- M debian-bookworm-titan/Dockerfile 1 file changed, 3 insertions(+), 1 deletion(-)

Approvals: Jenkins Builder: Verified osmith: Looks good to me, approved laforge: Looks good to me, but someone else must approve

diff --git a/debian-bookworm-titan/Dockerfile b/debian-bookworm-titan/Dockerfile index c4af2ef..0ce6c43 100644 --- a/debian-bookworm-titan/Dockerfile +++ b/debian-bookworm-titan/Dockerfile @@ -71,6 +71,7 @@ apt-get clean

# Clone osmo-ttcn3-hacks and deps, invalidate cache if deps change (OS#5017) +# chown: gerrit-verifications will potentially update the deps as UID=1000 RUN git clone https://gerrit.osmocom.org/osmo-ttcn3-hacks && \ make -j8 -C /osmo-ttcn3-hacks deps ADD https://gerrit.osmocom.org/plugins/gitiles/osmo-ttcn3-hacks/+/refs/heads/mas... /tmp/deps-Makefile @@ -78,4 +79,5 @@ cd /osmo-ttcn3-hacks && \ git pull && \ make -j8 deps; \ - fi + fi && \ + chown -R 1000:1000 /osmo-ttcn3-hacks/deps