[S] Change in osmo-ttcn3-hacks[master]: ttcn3-tcpdump-start.sh: Log command used to record packets

2024-05-21 Thread laforge
laforge has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/36866?usp=email )

Change subject: ttcn3-tcpdump-start.sh: Log command used to record packets
..

ttcn3-tcpdump-start.sh: Log command used to record packets

Change-Id: I8bebe53e630ba43e368700dc857b0e00d3cc7c56
---
M ttcn3-tcpdump-start.sh
1 file changed, 10 insertions(+), 0 deletions(-)

Approvals:
  osmith: Looks good to me, but someone else must approve
  laforge: Looks good to me, approved
  Jenkins Builder: Verified




diff --git a/ttcn3-tcpdump-start.sh b/ttcn3-tcpdump-start.sh
index 8564cd5..8b75d0e 100755
--- a/ttcn3-tcpdump-start.sh
+++ b/ttcn3-tcpdump-start.sh
@@ -85,6 +85,7 @@
 # Log stderr to CMD_OUTFILE and a dedicated error log file
 tee $CMD_OUTFILE < $FIFO > $CMD_OUTFILE_ERR &
 CMD_STR="$CMD -s 1520 -n -i any -w \"$TTCN3_PCAP_PATH/$TESTCASE.pcap\" 
>$CMD_OUTFILE 2>$FIFO &"
+echo "$CMD_STR"
 eval $CMD_STR
 # $CMD -s 1520 -n -i any -w \"$TTCN3_PCAP_PATH/$TESTCASE.pcap\" >$CMD_OUTFILE &
 PID=$!

--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/36866?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: I8bebe53e630ba43e368700dc857b0e00d3cc7c56
Gerrit-Change-Number: 36866
Gerrit-PatchSet: 3
Gerrit-Owner: pespin 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: osmith 
Gerrit-MessageType: merged


[S] Change in osmo-ttcn3-hacks[master]: ttcn3-tcpdump-start.sh: Log command used to record packets

2024-05-21 Thread laforge
Attention is currently required from: pespin.

laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/36866?usp=email )

Change subject: ttcn3-tcpdump-start.sh: Log command used to record packets
..


Patch Set 2: Code-Review+2


--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/36866?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: I8bebe53e630ba43e368700dc857b0e00d3cc7c56
Gerrit-Change-Number: 36866
Gerrit-PatchSet: 2
Gerrit-Owner: pespin 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: osmith 
Gerrit-Attention: pespin 
Gerrit-Comment-Date: Tue, 21 May 2024 09:02:57 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


[S] Change in osmo-ttcn3-hacks[master]: ttcn3-tcpdump-start.sh: Log command used to record packets

2024-05-21 Thread osmith
Attention is currently required from: pespin.

osmith has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/36866?usp=email )

Change subject: ttcn3-tcpdump-start.sh: Log command used to record packets
..


Patch Set 2: Code-Review+1

(1 comment)

File ttcn3-tcpdump-start.sh:

https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/36866/comment/f1c3f31f_13dbb6c8
PS1, Line 88: echo $CMD_STR
> should be […]
Done



--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/36866?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: I8bebe53e630ba43e368700dc857b0e00d3cc7c56
Gerrit-Change-Number: 36866
Gerrit-PatchSet: 2
Gerrit-Owner: pespin 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: osmith 
Gerrit-Attention: pespin 
Gerrit-Comment-Date: Tue, 21 May 2024 08:35:50 +
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Comment-In-Reply-To: osmith 
Gerrit-MessageType: comment


[S] Change in osmo-ttcn3-hacks[master]: ttcn3-tcpdump-start.sh: Log command used to record packets

2024-05-21 Thread pespin
Attention is currently required from: pespin.

Hello Jenkins Builder, osmith,

I'd like you to reexamine a change. Please visit

https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/36866?usp=email

to look at the new patch set (#2).

The following approvals got outdated and were removed:
Verified+1 by Jenkins Builder


Change subject: ttcn3-tcpdump-start.sh: Log command used to record packets
..

ttcn3-tcpdump-start.sh: Log command used to record packets

Change-Id: I8bebe53e630ba43e368700dc857b0e00d3cc7c56
---
M ttcn3-tcpdump-start.sh
1 file changed, 10 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks 
refs/changes/66/36866/2
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/36866?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: I8bebe53e630ba43e368700dc857b0e00d3cc7c56
Gerrit-Change-Number: 36866
Gerrit-PatchSet: 2
Gerrit-Owner: pespin 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: osmith 
Gerrit-Attention: pespin 
Gerrit-MessageType: newpatchset


[S] Change in osmo-ttcn3-hacks[master]: ttcn3-tcpdump-start.sh: Log command used to record packets

2024-05-21 Thread osmith
Attention is currently required from: pespin.

osmith has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/36866?usp=email )

Change subject: ttcn3-tcpdump-start.sh: Log command used to record packets
..


Patch Set 1:

(1 comment)

File ttcn3-tcpdump-start.sh:

https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/36866/comment/9fbbccb0_2a801040
PS1, Line 88: echo $CMD_STR
should be
```
echo "$CMD_STR"
```

Otherwise `>$CMD_OUTFILE 2>$FIFO &` will be applied here AFAIK, redirect stdout 
and stderr, and run in background.



--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/36866?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: I8bebe53e630ba43e368700dc857b0e00d3cc7c56
Gerrit-Change-Number: 36866
Gerrit-PatchSet: 1
Gerrit-Owner: pespin 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: osmith 
Gerrit-Attention: pespin 
Gerrit-Comment-Date: Tue, 21 May 2024 07:08:23 +
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment


[S] Change in osmo-ttcn3-hacks[master]: ttcn3-tcpdump-start.sh: Log command used to record packets

2024-05-21 Thread osmith
Attention is currently required from: pespin.

osmith has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/36866?usp=email )

Change subject: ttcn3-tcpdump-start.sh: Log command used to record packets
..


Patch Set 1: -Code-Review


--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/36866?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: I8bebe53e630ba43e368700dc857b0e00d3cc7c56
Gerrit-Change-Number: 36866
Gerrit-PatchSet: 1
Gerrit-Owner: pespin 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: osmith 
Gerrit-Attention: pespin 
Gerrit-Comment-Date: Tue, 21 May 2024 07:06:35 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


[S] Change in osmo-ttcn3-hacks[master]: ttcn3-tcpdump-start.sh: Log command used to record packets

2024-05-21 Thread osmith
Attention is currently required from: pespin.

osmith has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/36866?usp=email )

Change subject: ttcn3-tcpdump-start.sh: Log command used to record packets
..


Patch Set 1: Code-Review+1


--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/36866?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: I8bebe53e630ba43e368700dc857b0e00d3cc7c56
Gerrit-Change-Number: 36866
Gerrit-PatchSet: 1
Gerrit-Owner: pespin 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: osmith 
Gerrit-Attention: pespin 
Gerrit-Comment-Date: Tue, 21 May 2024 07:05:56 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


[S] Change in osmo-ttcn3-hacks[master]: ttcn3-tcpdump-start.sh: Log command used to record packets

2024-05-17 Thread pespin
pespin has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/36866?usp=email )


Change subject: ttcn3-tcpdump-start.sh: Log command used to record packets
..

ttcn3-tcpdump-start.sh: Log command used to record packets

Change-Id: I8bebe53e630ba43e368700dc857b0e00d3cc7c56
---
M ttcn3-tcpdump-start.sh
1 file changed, 10 insertions(+), 0 deletions(-)



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

diff --git a/ttcn3-tcpdump-start.sh b/ttcn3-tcpdump-start.sh
index 8564cd5..5b6c9ea 100755
--- a/ttcn3-tcpdump-start.sh
+++ b/ttcn3-tcpdump-start.sh
@@ -85,6 +85,7 @@
 # Log stderr to CMD_OUTFILE and a dedicated error log file
 tee $CMD_OUTFILE < $FIFO > $CMD_OUTFILE_ERR &
 CMD_STR="$CMD -s 1520 -n -i any -w \"$TTCN3_PCAP_PATH/$TESTCASE.pcap\" 
>$CMD_OUTFILE 2>$FIFO &"
+echo $CMD_STR
 eval $CMD_STR
 # $CMD -s 1520 -n -i any -w \"$TTCN3_PCAP_PATH/$TESTCASE.pcap\" >$CMD_OUTFILE &
 PID=$!

--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/36866?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: I8bebe53e630ba43e368700dc857b0e00d3cc7c56
Gerrit-Change-Number: 36866
Gerrit-PatchSet: 1
Gerrit-Owner: pespin 
Gerrit-MessageType: newchange