<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Patch Set 17: Code-Review+2</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Note: We must not merge this before we have merged the related osmo-msc code. The reason for this is that the tests will try to access the SGs interface always on startup. So all tests will start to fail.</p><p><a href="https://gerrit.osmocom.org/11488">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/11488/3/msc/MSC_Tests.default">File msc/MSC_Tests.default:</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/11488/3/msc/MSC_Tests.default@25">Patch Set #3, Line 25:</a> <code style="font-family:monospace,monospace"></code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">why did you remove the SGsAP. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">I probably dropped this unintentionally, however it works fine without.</p><p style="white-space: pre-wrap; word-wrap: break-word;">When I add the line again I get this warning:</p><p style="white-space: pre-wrap; word-wrap: break-word;">TC_sgsap_lu(23)@lobotron: Warning: Test port parameter sctp_stack is not supported on port SGsAP.</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/11488/8/msc/MSC_Tests.ttcn">File msc/MSC_Tests.ttcn:</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/11488/8/msc/MSC_Tests.ttcn@3108">Patch Set #8, Line 3108:</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;">testcase TC_sgsap_lu_and_nothing() runs on MTC_CT {<br>     var BSC_ConnHdlr vc_conn;<br>     f_init();<br><br>   vc_conn := f_start_handler(refers(f_tc_sgsap_lu_and_nothing), 3);<br>     vc_conn.done;<br>}<br><br>private function f_tc_sgsap_expl_imsi_det_eps(charstring id, BSC_ConnHdlrPars pars)<br>runs on BSC_ConnHdlr {<br>       f_init_handler(pars);<br> f_sgs_perform_lu();<br>   f_sleep(3.0);<br><br>       var octetstring mme_name := f_enc_dns_hostname(mp_mme_name);<br>  SGsAP.send(ts_SGsAP_EPS_DETACH_IND(g_pars.imsi, mme_name, UE_initiated));<br>     SGsAP.receive(tr_SGsAP_EPS_DETACH_ACK(g_pars.imsi));<br>  f_ctrl_get_exp(IPA_CTRL, "fsm.SGs-UE.id.imsi:" & hex2str(g_pars.imsi) & ".state", "SGs-NULL");<br><br></pre></blockquote></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Same here, what is the purpose of that?!? o_O</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">As long as this is work in progress you can ignore such quirks. I will remove this when everything is ready.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/11488/8/msc/MSC_Tests.ttcn@3186">Patch Set #8, Line 3186:</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;"><br>    /* Wait for the states inside the MSC to settle and check the state<br>    * of the SGs Association */<br><br></pre></blockquote></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Here too... If both parts of this file are tooo unrelated, […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Done</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/11488">change 11488</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/11488"/><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: I38543c35a9e74cea276e58d1d7ef01ef07ffe858 </div>
<div style="display:none"> Gerrit-Change-Number: 11488 </div>
<div style="display:none"> Gerrit-PatchSet: 17 </div>
<div style="display:none"> Gerrit-Owner: Harald Welte <laforge@gnumonks.org> </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: Vadim Yanitskiy <axilirator@gmail.com> </div>
<div style="display:none"> Gerrit-CC: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 18 Dec 2018 16:30:38 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>