<p><a href="https://gerrit.osmocom.org/c/osmo-gsm-tester/+/21302">View Change</a></p><p>4 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-gsm-tester/+/21302/3/src/osmo_gsm_tester/obj/android_host.py">File src/osmo_gsm_tester/obj/android_host.py:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-gsm-tester/+/21302/3/src/osmo_gsm_tester/obj/android_host.py@38">Patch Set #3, Line 38:</a> <code style="font-family:monospace,monospace">                ['find', str(self.remote_run_dir),</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">What do you say to this solution? sudo is not necessary anymore and I only remove the files that have the specific extensions. The only other solution here would be, as you have proposed, to delete the complete folder but it would be more overhead in terms of number of commands to be used.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-gsm-tester/+/21302/3/src/osmo_gsm_tester/obj/android_host.py@45">Patch Set #3, Line 45:</a> <code style="font-family:monospace,monospace">        popen_args_clear_diag_logs = \</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">I could not find any other way to do this that applies to all Android devices we have here to test. The path is hard coded and is non standard, so there shouldn't be anything in there besides the 3 files created by diag_mdlog. Can we leave it like that or should I investigate again to find another way?</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-gsm-tester/+/21302/2/src/osmo_gsm_tester/obj/bitrate_monitor.py">File src/osmo_gsm_tester/obj/bitrate_monitor.py:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-gsm-tester/+/21302/2/src/osmo_gsm_tester/obj/bitrate_monitor.py@58">Patch Set #2, Line 58:</a> <code style="font-family:monospace,monospace">            self.rem_host.remote_user = 'root'</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">ACK, but then why not simply setting "root" as ssh_user in resources. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">I can't do that because the commands to be executed on the machine running the ADB instance and hosting the Android UEs need the remote_user='jenkins'. The problematic here is that in case of remote execution the slave unit has the same ip as the Android UEs. The only difference is the port of the ssh server.</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-gsm-tester/+/21302/2/sysmocom/suites/4g/iperf3_dl.py">File sysmocom/suites/4g/iperf3_dl.py:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-gsm-tester/+/21302/2/sysmocom/suites/4g/iperf3_dl.py@26">Patch Set #2, Line 26:</a> <code style="font-family:monospace,monospace">proc = iperf3cli.prepare_test_proc(iperf3cli.DIR_DL, ue.netns(), bitrate=max_rate, ue=ue)</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">What about extending RunNode with a new "run_type" = "adb" and then add a new schema property "adb_s […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Very good idea! Makes it much cleaner and I don't have to touch the iperf3_dl/ul tests.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-gsm-tester/+/21302">change 21302</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.osmocom.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.osmocom.org/c/osmo-gsm-tester/+/21302"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-gsm-tester </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I79a5d803e869a868d4dac5e0d4c2feb38038dc5c </div>
<div style="display:none"> Gerrit-Change-Number: 21302 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </div>
<div style="display:none"> Gerrit-Owner: ninjab3s <nils.fuerste@softwareradiosystems.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 30 Nov 2020 14:00:05 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Comment-In-Reply-To: ninjab3s <nils.fuerste@softwareradiosystems.com> </div>
<div style="display:none"> Comment-In-Reply-To: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>