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
osmith has uploaded this change for review. ( 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(-)
git pull ssh://gerrit.osmocom.org:29418/docker-playground refs/changes/58/36858/1
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-MessageType: newchange
Attention is currently required from: pespin.
osmith has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/36855?usp=email )
Change subject: SIP_Templates: Fix missing ':' between URI scheme and host
......................................................................
Patch Set 1: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/36855?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: I5369db54901d50cf08d7ffc3e0d1cf3f5091e77b
Gerrit-Change-Number: 36855
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Fri, 17 May 2024 08:47:01 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
Attention is currently required from: pespin.
osmith has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/36853?usp=email )
Change subject: ttcn3-tcpdump-start.sh: Log issue with packet dumper not properly started
......................................................................
Patch Set 2: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/36853?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: I50af5eb16a000c4f3585a3458f9130a349eff097
Gerrit-Change-Number: 36853
Gerrit-PatchSet: 2
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Fri, 17 May 2024 08:46:48 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment