laforge has submitted this change. (
https://gerrit.osmocom.org/c/osmo-ci/+/15760 )
Change subject: docker: chown -R build:build /usr/local
......................................................................
docker: chown -R build:build /usr/local
Prepare to run osmo-python-tests in docker. I've noticed that its
contrib/jenkins.sh tries to write to /usr/local, which makes sense for
the "update-osmo-python-on-slaves" job to actually deploy new versions.
However, the same script gets called in gerrit-verifications too, and
there we should definitively not install a version that is still in
review to whichever random buildslave that is executing the
gerrit-verification job! The follow-up patch for building most Osmocom
projects in docker will fix this bug.
Related: OS#3726
Change-Id: Ie1de8a6d436de0b286ed3372166eceb408bab084
---
M docker/Dockerfile_osmocom_jenkins.amd64
1 file changed, 3 insertions(+), 0 deletions(-)
Approvals:
laforge: Looks good to me, approved
pespin: Looks good to me, but someone else must approve
osmith: Verified
diff --git a/docker/Dockerfile_osmocom_jenkins.amd64
b/docker/Dockerfile_osmocom_jenkins.amd64
index 58e1ee9..6b43911 100644
--- a/docker/Dockerfile_osmocom_jenkins.amd64
+++ b/docker/Dockerfile_osmocom_jenkins.amd64
@@ -84,3 +84,6 @@
DEBIAN_FRONTEND=noninteractive apt-get install -y /tmp/ulfius/*.deb && \
cd ~ && \
rm -r /tmp/ulfius
+
+# osmo-python-tests' contrib/jenkins.sh writes to /usr/local as user
+RUN chown -R build:build /usr/local
--
To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/15760
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-ci
Gerrit-Branch: master
Gerrit-Change-Id: Ie1de8a6d436de0b286ed3372166eceb408bab084
Gerrit-Change-Number: 15760
Gerrit-PatchSet: 3
Gerrit-Owner: osmith <osm...@sysmocom.de>
Gerrit-Reviewer: daniel <dwillm...@sysmocom.de>
Gerrit-Reviewer: laforge <lafo...@osmocom.org>
Gerrit-Reviewer: neels <nhofm...@sysmocom.de>
Gerrit-Reviewer: osmith <osm...@sysmocom.de>
Gerrit-Reviewer: pespin <pes...@sysmocom.de>
Gerrit-MessageType: merged
