[PATCH] osmo-ttcn3-hacks[master]: ttcn3-tcpdump-start.sh: Wait for tcpdump to start recording

Pau Espin Pedrol gerrit-no-reply at lists.osmocom.org
Thu Feb 15 15:11:32 UTC 2018

Review at  https://gerrit.osmocom.org/6482

ttcn3-tcpdump-start.sh: Wait for tcpdump to start recording

We generate some fake transit and we wait until we catch tcpdump already
saved some packet into the pcap file, this way making sure it is already
recording before starting the test.

The -U flag (--packet-buffered) is added to increase the chances to
sleep less time waiting for stuff being saved into the pcap file.
According to tcpdump manual:
If  the  -w option is specified, make the saved raw packet output ``packet-buffered''; i.e., as each packet is saved, it will
              be written to the output file, rather than being written only when the output buffer fills.

Change-Id: I91cfd84ff5356857a13af1901abfe2204a93f76d
M ttcn3-tcpdump-start.sh
1 file changed, 14 insertions(+), 5 deletions(-)

  git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/82/6482/1

diff --git a/ttcn3-tcpdump-start.sh b/ttcn3-tcpdump-start.sh
index 10e8ab4..0ce88e9 100755
--- a/ttcn3-tcpdump-start.sh
+++ b/ttcn3-tcpdump-start.sh
@@ -1,6 +1,7 @@
 if [ "z$TTCN3_PCAP_PATH" = "z" ]; then
@@ -15,20 +16,28 @@
 # NOTE: This requires you to be root or something like
 # "laforge ALL=NOPASSWD: /usr/sbin/tcpdump, /bin/kill" in your sudoers file
 if [ "$(id -u)" = "0" ]; then
-	CMD=/usr/sbin/tcpdump
-	CMD="sudo /usr/sbin/tcpdump"
+	CMD="sudo $TCPDUMP"
-$CMD -s 0 -n -i any -w "$TTCN3_PCAP_PATH/$TESTCASE.pcap" >$TTCN3_PCAP_PATH/$TESTCASE.pcap.log 2>&1 &
+$CMD -U -s 0 -n -i any -w "$TTCN3_PCAP_PATH/$TESTCASE.pcap" >$TTCN3_PCAP_PATH/$TESTCASE.pcap.log 2>&1 &
 echo $PID > $PIDFILE
-# Wait until tcpdump creates the pcap file to give it some time to start listenting.
+# Wait until tcpdump creates the pcap file and starts recording.
+# We generate some traffic until we see tcpdump catches it.
 # Timeout is 10 seconds.
+ping >/dev/null 2>&1 &
-while [ ! -f "$TTCN3_PCAP_PATH/$TESTCASE.pcap" ] && [ $i -lt 10 ]
+while [ ! -f "$TTCN3_PCAP_PATH/$TESTCASE.pcap" ] ||
+      [ "$($TCPDUMP -r "$TTCN3_PCAP_PATH/$TESTCASE.pcap" 2>/dev/null | wc -l)" -eq 0 ]
 	echo "Waiting for tcpdump to start... $i"
 	sleep 1
+	if [ $i -eq 10 ]; then
+		break
+	fi
+kill $PID

To view, visit https://gerrit.osmocom.org/6482
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I91cfd84ff5356857a13af1901abfe2204a93f76d
Gerrit-PatchSet: 1
Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Owner: Pau Espin Pedrol <pespin at sysmocom.de>

More information about the gerrit-log mailing list