osmith has submitted this change. ( https://gerrit.osmocom.org/c/docker-playground/+/36858?usp=email ) Change subject: Revert "ttcn3-docker-run.sh: Fix dumpcap failing to access /data inside docker" ...................................................................... Revert "ttcn3-docker-run.sh: Fix dumpcap failing to access /data inside docker" This reverts commit 2b9c8e3db8cf77c5ce5c3f888365f983f3ee8d76, as it causes a lot of testsuites to fail with e.g. + network_replace_subnet_in_configs + set +x Applying SUBNET=161 to: /home/osmocom-build/jenkins/workspace/ttcn3-ggsn-test/logs/ggsn-all/osmo-ggsn.cfg Applying SUBNET=161 to: /home/osmocom-build/jenkins/workspace/ttcn3-ggsn-test/logs/ggsn-tester-all/GGSN_Tests.cfg sed: couldn't open temporary file /home/osmocom-build/jenkins/workspace/ttcn3-ggsn-test/logs/ggsn-tester-all/sedsByWIc: Permission denied Let's discuss how to fix the dumpcap problem instead in OS#6455. Change-Id: Ic98e8206d8251060b777fc7d0b4497ba3e1caf7f --- M common/ttcn3-docker-run.sh 1 file changed, 20 insertions(+), 3 deletions(-) Approvals: Jenkins Builder: Verified osmith: Looks good to me, approved diff --git a/common/ttcn3-docker-run.sh b/common/ttcn3-docker-run.sh index 60339c4..c8c871e 100755 --- a/common/ttcn3-docker-run.sh +++ b/common/ttcn3-docker-run.sh @@ -31,9 +31,6 @@ done fi -# Make sure /data is owned to the user running inside docker. -# jenkins-commons.sh fix_perms() will fix back the permissions once done. -chown -R "$(id -u):$(id -g)" /data cd /data # Use TEST_NAME to only run one test instead of all. Set it like this: -- To view, visit https://gerrit.osmocom.org/c/docker-playground/+/36858?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: Ic98e8206d8251060b777fc7d0b4497ba3e1caf7f Gerrit-Change-Number: 36858 Gerrit-PatchSet: 1 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-MessageType: merged