<p><a href="https://gerrit.osmocom.org/c/osmo-gsm-tester/+/21523">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/osmo-gsm-tester/+/21523/2/sysmocom/handover-suites.conf">File sysmocom/handover-suites.conf:</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/+/21523/2/sysmocom/handover-suites.conf@1">Patch Set #2, Line 1:</a> <code style="font-family:monospace,monospace">- handover_2G:nanobts+bts1-oc2g</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Why not adding it to the defaults-suites. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">happening in another commit</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-gsm-tester/+/21523/2/sysmocom/suites/handover_2G/handover.py">File sysmocom/suites/handover_2G/handover.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/+/21523/2/sysmocom/suites/handover_2G/handover.py@91">Patch Set #2, Line 91:</a> <code style="font-family:monospace,monospace">with bsc.ctrl() as bsc_ctrl:</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">so basically you keep the object until the end of the test. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">the entire point of 'with' is that the CTRL socket is also closed again when the test is done, whatever the reason. Otherwise we may end up leaking open sockets en masse.</p><p style="white-space: pre-wrap; word-wrap: break-word;">What other way would you suggest?</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/+/21523/2/sysmocom/suites/handover_2G/handover.py@118">Patch Set #2, Line 118:</a> <code style="font-family:monospace,monospace">            wait_no_raise(counters.changed, timeout=20)</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">try: wait() except: pass […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">i think at first i just wanted some supplementary info of the counters and not fail on them if they didn't change, but wait() is sufficient indeed.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-gsm-tester/+/21523">change 21523</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/+/21523"/><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: I0b2671304165a1aaae2b386af46fbd8b098e3bd8 </div>
<div style="display:none"> Gerrit-Change-Number: 21523 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-CC: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Sun, 06 Dec 2020 22:17:31 +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>