srs_andre has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/17562 )


Change subject: srs_ue: run srsUE as normal RemoteProcess
......................................................................

srs_ue: run srsUE as normal RemoteProcess

running with RemoteProcessFixIgnoreSIGHUP was needed bc
srsUE used to ignore SIGHUP that is delivered to the process
when the SSH session is terminated causing the UE to not
quit properly. This isn't the case anymore so we can use
RemoteProcess for spawning the UE.

Furthermore, the wrapper script didn't pass along stdin
to it's child process, i.e., srsUE. Therefore the console
trace can't be seen if the run through the wrapper.

Change-Id: Iab4dafc58167620362a36ec9e904b8d49a7d5fc1
---
M src/osmo_gsm_tester/srs_ue.py
1 file changed, 1 insertion(+), 1 deletion(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-gsm-tester 
refs/changes/62/17562/1

diff --git a/src/osmo_gsm_tester/srs_ue.py b/src/osmo_gsm_tester/srs_ue.py
index ffd847b..467fe9b 100644
--- a/src/osmo_gsm_tester/srs_ue.py
+++ b/src/osmo_gsm_tester/srs_ue.py
@@ -172,7 +172,7 @@
                 '--general.metrics_csv_filename=' + self.remote_metrics_file,
                 self.additional_args)

-        self.process = 
self.rem_host.RemoteProcessFixIgnoreSIGHUP(srsUE.BINFILE, 
util.Dir(srsUE.REMOTE_DIR), args)
+        self.process = self.rem_host.RemoteProcess(srsUE.BINFILE, args)
         #self.process = 
self.rem_host.RemoteProcessFixIgnoreSIGHUP(srsUE.BINFILE, remote_run_dir, args, 
remote_lib)
         self.suite_run.remember_to_stop(self.process)
         self.process.launch()

--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/17562
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: Iab4dafc58167620362a36ec9e904b8d49a7d5fc1
Gerrit-Change-Number: 17562
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre <an...@softwareradiosystems.com>
Gerrit-MessageType: newchange

Reply via email to