<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">(1 comment)</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">I mean, it would be nicer to allow having a ctrl for both sides,<br>but if it fixes the status quo, this patch is ok, right?</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Making the control interface available from both sides requires an<br>additional architecture to route the message back and forth between<br>the two entities.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">I'm not sure I'm following you here.  Why would two components not be<br>able to each independently talk to the CTRL interface?  Where exactly<br>is the problem here?  Where is what kind of "architecture" needed?</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">The patch does not break anything, also having the control<br>interface available at the BSC_ConnectionHandler makes more sense<br>than having it on the MTC_CT.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">For your particular use case right now, this may be true.  For other use cases, it is different.  I put it there intentionally at a time.  As other test suites also show, for some tests it makes sense to access CTRL or VTY from the main test component,and for others from the respective *Handler.  There's no one truth here.</p><p>Patch set 8:<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/11690">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.osmocom.org/11690">change 11690</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/11690"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-ttcn3-hacks </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I204b2e96057d13342f4bd4fdaf08fb7b88b6b11d </div>
<div style="display:none"> Gerrit-Change-Number: 11690 </div>
<div style="display:none"> Gerrit-PatchSet: 8 </div>
<div style="display:none"> Gerrit-Owner: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder (1000002) </div>
<div style="display:none"> Gerrit-Reviewer: Max <msuraev@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 12 Dec 2018 20:57:18 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-HasLabels: Yes </div>