osmith submitted this change.

View Change


Approvals: Jenkins Builder: Verified osmith: Looks good to me, approved
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(-)

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 change 36858. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: docker-playground
Gerrit-Branch: master
Gerrit-Change-Id: Ic98e8206d8251060b777fc7d0b4497ba3e1caf7f
Gerrit-Change-Number: 36858
Gerrit-PatchSet: 1
Gerrit-Owner: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: osmith <osmith@sysmocom.de>
Gerrit-MessageType: merged