<p>Patch set 2:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #ffd4d4;">Code-Review -1</span></p><p><a href="https://gerrit.osmocom.org/11516">View Change</a></p><p>3 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/11516/2/src/osmo_ms_driver/starter.py">File src/osmo_ms_driver/starter.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/11516/2/src/osmo_ms_driver/starter.py@64">Patch Set #2, Line 64:</a> </p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><pre style="font-family: monospace,monospace; white-space: pre-wrap;">       #pcap_recorder.PcapRecorder(self.suite_run, self.run_dir.new_dir('pcap'), None,<br>        #                           'host %s and port not 22' % self.addr())<br></pre></blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">I think starting 10k tcpdump is not that desirable.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/11516/2/src/osmo_ms_driver/starter.py@55">Patch Set #2, Line 55:</a> </p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><pre style="font-family: monospace,monospace; white-space: pre-wrap;"> inst = util.Dir(os.path.abspath(self.suite_run.trial.get_inst('osmocom-bb')))<br><br>        binary = inst.child('bin', 'virtphy')<br>        if not os.path.isfile(binary):<br>            raise RuntimeError('Binary missing: %r' % binary)<br>        lib = inst.child('lib')<br>        if not os.path.isdir(lib):<br>            raise RuntimeError('No lib/ in %r' % inst)<br><br>        #pcap_recorder.PcapRecorder(self.suite_run, self.run_dir.new_dir('pcap'), None,<br>        #                           'host %s and port not 22' % self.addr())<br><br>        env = { 'LD_LIBRARY_PATH': util.prepend_library_path(lib) }<br></pre></blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Can this be in a helper returning the binary + env?</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/11516/2/src/osmo_ms_driver/starter.py@141">Patch Set #2, Line 141:</a> </p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><pre style="font-family: monospace,monospace; white-space: pre-wrap;">        self.run_dir = util.Dir(self.suite_run.get_test_run_dir().new_dir(self.name()))<br><br>        inst = util.Dir(os.path.abspath(self.suite_run.trial.get_inst('osmocom-bb')))<br><br>        binary = inst.child('bin', 'mobile')<br>        if not os.path.isfile(binary):<br>            raise RuntimeError('Binary missing: %r' % binary)<br>        lib = inst.child('lib')<br>        if not os.path.isdir(lib):<br>            raise RuntimeError('No lib/ in %r' % inst)<br><br>        #pcap_recorder.PcapRecorder(self.suite_run, self.run_dir.new_dir('pcap'), None,<br>        #                           'host %s and port not 22' % self.addr())<br><br>        env = { 'LD_LIBRARY_PATH': util.prepend_library_path(lib) }<br></pre></blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Utility function please.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/11516">change 11516</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/11516"/><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-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I6748fce6ca25c7d4daedafe87b6b77c471d17bf9 </div>
<div style="display:none"> Gerrit-Change-Number: 11516 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: Pau Espin Pedrol <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Holger Freyther <holger@freyther.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder (1000002) </div>
<div style="display:none"> Gerrit-Reviewer: Pau Espin Pedrol <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 30 Oct 2018 21:22:59 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: Yes </div>