Change in osmo-ttcn3-hacks[master]: MSC_ConnectionHandler: stop properly on failure in f_establish_fully()
Harald Welte has posted comments on this change. ( https://gerrit.osmocom.org/10904 ) Change subject: MSC_ConnectionHandler: stop properly on failure in f_establish_fully() .. Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/10904 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-ttcn3-hacks Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ib4ac7bcbac35a4ae454d1806f3fbb727834d18b7 Gerrit-Change-Number: 10904 Gerrit-PatchSet: 1 Gerrit-Owner: dexter Gerrit-Reviewer: Harald Welte Gerrit-Reviewer: Jenkins Builder (102) Gerrit-Reviewer: Vadim Yanitskiy Gerrit-Comment-Date: Thu, 13 Sep 2018 13:58:53 + Gerrit-HasComments: No Gerrit-HasLabels: Yes
Change in osmo-ttcn3-hacks[master]: MSC_ConnectionHandler: stop properly on failure in f_establish_fully()
Harald Welte has submitted this change and it was merged. ( https://gerrit.osmocom.org/10904 ) Change subject: MSC_ConnectionHandler: stop properly on failure in f_establish_fully() .. MSC_ConnectionHandler: stop properly on failure in f_establish_fully() When the altstep in f_establish_fully() gets an unexpected ASSIGNMENT FAIL or COMPLETE it should stop completely like it is already implemented in many other altsteps. Change-Id: Ib4ac7bcbac35a4ae454d1806f3fbb727834d18b7 --- M bsc/MSC_ConnectionHandler.ttcn 1 file changed, 10 insertions(+), 0 deletions(-) Approvals: Harald Welte: Looks good to me, approved Vadim Yanitskiy: Looks good to me, but someone else must approve Jenkins Builder: Verified diff --git a/bsc/MSC_ConnectionHandler.ttcn b/bsc/MSC_ConnectionHandler.ttcn index 102c30e..9f67f1b 100644 --- a/bsc/MSC_ConnectionHandler.ttcn +++ b/bsc/MSC_ConnectionHandler.ttcn @@ -939,18 +939,28 @@ (not st.is_assignment and (st.modify_done or not exp_modify))) and exp_compl] BSSAP.receive(tr_BSSMAP_AssignmentComplete) { setverdict(fail, "Received non-matching ASSIGNMENT COMPLETE"); + all component.stop; + mtc.stop; } [exp_compl] BSSAP.receive(tr_BSSMAP_AssignmentFail) { setverdict(fail, "Received unexpected ASSIGNMENT FAIL"); + all component.stop; + mtc.stop; } [not exp_compl] BSSAP.receive(tr_BSSMAP_AssignmentComplete) { setverdict(fail, "Received unexpected ASSIGNMENT COMPLETE"); + all component.stop; + mtc.stop; } [not exp_compl] BSSAP.receive(tr_BSSMAP_AssignmentFail) { setverdict(fail, "Received non-matching ASSIGNMENT FAIL"); + all component.stop; + mtc.stop; } [] T.timeout { setverdict(fail, "Timeout waiting for ASSIGNMENT COMPLETE"); + all component.stop; + mtc.stop; } } log("g_media ", g_media); -- To view, visit https://gerrit.osmocom.org/10904 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-ttcn3-hacks Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ib4ac7bcbac35a4ae454d1806f3fbb727834d18b7 Gerrit-Change-Number: 10904 Gerrit-PatchSet: 2 Gerrit-Owner: dexter Gerrit-Reviewer: Harald Welte Gerrit-Reviewer: Jenkins Builder (102) Gerrit-Reviewer: Vadim Yanitskiy
Change in osmo-ttcn3-hacks[master]: MSC_ConnectionHandler: stop properly on failure in f_establish_fully()
Vadim Yanitskiy has posted comments on this change. ( https://gerrit.osmocom.org/10904 ) Change subject: MSC_ConnectionHandler: stop properly on failure in f_establish_fully() .. Patch Set 1: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/10904 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-ttcn3-hacks Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ib4ac7bcbac35a4ae454d1806f3fbb727834d18b7 Gerrit-Change-Number: 10904 Gerrit-PatchSet: 1 Gerrit-Owner: dexter Gerrit-Reviewer: Jenkins Builder (102) Gerrit-Reviewer: Vadim Yanitskiy Gerrit-Comment-Date: Wed, 12 Sep 2018 17:51:23 + Gerrit-HasComments: No Gerrit-HasLabels: Yes
Change in osmo-ttcn3-hacks[master]: MSC_ConnectionHandler: stop properly on failure in f_establish_fully()
dexter has uploaded this change for review. ( https://gerrit.osmocom.org/10904 Change subject: MSC_ConnectionHandler: stop properly on failure in f_establish_fully() .. MSC_ConnectionHandler: stop properly on failure in f_establish_fully() When the altstep in f_establish_fully() gets an unexpected ASSIGNMENT FAIL or COMPLETE it should stop completely like it is already implemented in many other altsteps. Change-Id: Ib4ac7bcbac35a4ae454d1806f3fbb727834d18b7 --- M bsc/MSC_ConnectionHandler.ttcn 1 file changed, 10 insertions(+), 0 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/04/10904/1 diff --git a/bsc/MSC_ConnectionHandler.ttcn b/bsc/MSC_ConnectionHandler.ttcn index 102c30e..9f67f1b 100644 --- a/bsc/MSC_ConnectionHandler.ttcn +++ b/bsc/MSC_ConnectionHandler.ttcn @@ -939,18 +939,28 @@ (not st.is_assignment and (st.modify_done or not exp_modify))) and exp_compl] BSSAP.receive(tr_BSSMAP_AssignmentComplete) { setverdict(fail, "Received non-matching ASSIGNMENT COMPLETE"); + all component.stop; + mtc.stop; } [exp_compl] BSSAP.receive(tr_BSSMAP_AssignmentFail) { setverdict(fail, "Received unexpected ASSIGNMENT FAIL"); + all component.stop; + mtc.stop; } [not exp_compl] BSSAP.receive(tr_BSSMAP_AssignmentComplete) { setverdict(fail, "Received unexpected ASSIGNMENT COMPLETE"); + all component.stop; + mtc.stop; } [not exp_compl] BSSAP.receive(tr_BSSMAP_AssignmentFail) { setverdict(fail, "Received non-matching ASSIGNMENT FAIL"); + all component.stop; + mtc.stop; } [] T.timeout { setverdict(fail, "Timeout waiting for ASSIGNMENT COMPLETE"); + all component.stop; + mtc.stop; } } log("g_media ", g_media); -- To view, visit https://gerrit.osmocom.org/10904 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-ttcn3-hacks Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: Ib4ac7bcbac35a4ae454d1806f3fbb727834d18b7 Gerrit-Change-Number: 10904 Gerrit-PatchSet: 1 Gerrit-Owner: dexter