<p>Patch set 4:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #d4ffd4;">Code-Review +2</span></p><p><a href="https://gerrit.osmocom.org/13042">View Change</a></p><p>2 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/13042/4/src/osmo_gsm_tester/ms_osmo_mobile.py">File src/osmo_gsm_tester/ms_osmo_mobile.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/13042/4/src/osmo_gsm_tester/ms_osmo_mobile.py@33">Patch Set #4, Line 33:</a> <code style="font-family:monospace,monospace">            return "00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00"</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Is this needed for later using imsi_ki_gen afaiu?</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/13042/4/src/osmo_gsm_tester/suite.py">File src/osmo_gsm_tester/suite.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/13042/4/src/osmo_gsm_tester/suite.py@347">Patch Set #4, Line 347:</a> <code style="font-family:monospace,monospace">    def all_modems(self):</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">I precisely wanted to avoid having more functions by having the generic one, since suite.py is already quite big (some stuff should be moved somewhere else, but of course not related). I'd prefer you using all_resources(suite.modem) instead and removing it, but fine.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/13042">change 13042</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/13042"/><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: Ic9fd560c2924731e4fd6eea5aaf3ad565cb4ef52 </div>
<div style="display:none"> Gerrit-Change-Number: 13042 </div>
<div style="display:none"> Gerrit-PatchSet: 4 </div>
<div style="display:none"> Gerrit-Owner: Holger Freyther <holger@freyther.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, 05 Mar 2019 17:40:22 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: Yes </div>