pespin has submitted this change and it was merged. (
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/15190 )
Change subject: bts-trx: Improve logging and trap SIGTERM in ssh_sigkiller.sh
......................................................................
bts-trx: Improve logging and trap SIGTERM in ssh_sigkiller.sh
Trapping SIGTERM is done in order to get extra logging about that event.
Related: OS#4151
Change-Id: I2c7649dee8f5ecdaa08274806ccd8948d197ffdb
---
M src/osmo_gsm_tester/bts_osmotrx.py
1 file changed, 2 insertions(+), 0 deletions(-)
Approvals:
laforge: Looks good to me, but someone else must approve
pespin: Looks good to me, approved
Jenkins Builder: Verified
diff --git a/src/osmo_gsm_tester/bts_osmotrx.py
b/src/osmo_gsm_tester/bts_osmotrx.py
index 93e3c62..b3738d2 100644
--- a/src/osmo_gsm_tester/bts_osmotrx.py
+++ b/src/osmo_gsm_tester/bts_osmotrx.py
@@ -224,12 +224,14 @@
echo "received signal handler $sig, killing $mypid"
kill $mypid
}
+ trap 'sign_handler SIGTERM' SIGTERM
trap 'sign_handler SIGINT' SIGINT
trap 'sign_handler SIGHUP' SIGHUP
"$@" &
mypid=$!
echo "waiting for $mypid"
wait $mypid
+ echo "process $mypid finished"
"""
f.write(r)
st = os.stat(wrapper_script)
--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/15190
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I2c7649dee8f5ecdaa08274806ccd8948d197ffdb
Gerrit-Change-Number: 15190
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-Reviewer: pespin <[email protected]>
Gerrit-MessageType: merged