<p><a href="https://gerrit.osmocom.org/c/osmo-gsm-tester/+/21302">View Change</a></p><p>7 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/2//COMMIT_MSG">Commit Message:</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//COMMIT_MSG@32">Patch Set #2, Line 32:</a> <code style="font-family:monospace,monospace">Infrastructure explaination:</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">You probably want to add all this documentarion into the UserManual too (osmo-gsm-tester. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Sure! My college Bedran is currently working on a detailed description. We will add it to the manual as well.</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/2//COMMIT_MSG@55">Patch Set #2, Line 55:</a> <code style="font-family:monospace,monospace">dropbearmulti dropbear -F -E -p 130 -R -T /data/local/tmp/authorized_keys  -U 0 -G 0 -N root -A</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">what's this dropbearmulti?</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Its a single binary version of dropbear. https://github.com/mkj/dropbear/blob/master/MULTI. We will explain this in the part to be added to the manual.</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/2//COMMIT_MSG@87">Patch Set #2, Line 87:</a> <code style="font-family:monospace,monospace">  ue_serial: '8d9d79a9'</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">what's this for? can you check if you can reuse any of the existing fields? like "path" ?</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">This is the USB serial of the device used by ADB to identify UEs. path is not used by the AndroidUE class, so it should be possible to reuse this variable here.</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;">You should ideally use sudo with sudoers file limit use of certain commands, that what we usually di […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Unfortunately sudo is not available under Android and we did not find a way to add users to the Android OS. For these reasons we used the workaround with setting remote_user to 'root'.</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/2/src/osmo_gsm_tester/obj/bitrate_monitor.py@79">Patch Set #2, Line 79:</a> <code style="font-family:monospace,monospace">        self.testenv.remember_to_stop(self.rx_monitor_proc)</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">As I shared before, return run_androidue_cmd without calling launch() on it, then AFTER calling reme […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">I understand, I rework all occurrences in the upcoming patches.</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/2/src/osmo_gsm_tester/obj/bitrate_monitor.py@88">Patch Set #2, Line 88:</a> <code style="font-family:monospace,monospace">        brate_rx_raw = self.rx_monitor_proc.get_stdout().split('\n')</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">if you run the process locally you may find out the output you are looking for is in stderr, be care […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Ack</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;">ue.path() instead of pasing ue? Or ue.serial_device()? […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">In case an AndroidUE is used I need to know two values, the ADB serial of the UE for local execution and the ssh port for remote execution. Further, I have to distinguish between AndroidUE modems and all the others. To keep the number of changes low I suggest to add a new function to the base class which returns the appropriate value depending on local or remote execution. What do you think about this?</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: 2 </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: Wed, 25 Nov 2020 11:35:55 +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: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>