Change in ...osmo-gsm-tester[master]: bts-trx: Improve logging and trap SIGTERM in ssh_sigkiller.sh

pespin gerrit-no-reply at lists.osmocom.org
Wed Aug 14 17:25:52 UTC 2019


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 <pespin at sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge at gnumonks.org>
Gerrit-Reviewer: pespin <pespin at sysmocom.de>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20190814/f8f1bdbe/attachment.html>


More information about the gerrit-log mailing list