laforge has submitted this change. ( https://gerrit.osmocom.org/c/erlang/osmo-s1gw/+/38240?usp=email )

Change subject: s1ap_proxy_test: re-generate E-RAB RELEASE.{cmd,rsp} PDU blobs ......................................................................

s1ap_proxy_test: re-generate E-RAB RELEASE.{cmd,rsp} PDU blobs

These PDU blobs contained the right E-RAB-ID=6, but different and unexpected {MME,ENB}-UE-S1AP-ID values. As a result of this, the E-RAB {7,9,6} that we establish in test_e_rab_setup/1 is never released properly, and this is why we're seeing these two errors:

E-RAB {1,0,6} is not registered (s1ap_proxy:handle_ie/2) E-RAB {2,1,6} is not registered (s1ap_proxy:handle_ie/2)

The new S1AP PDU blobs were re-generated using ttcn3-s1gw-test.

Change-Id: I229c47148fd7408e44f1339ae2a2909f570c73ec --- M test/s1ap_proxy_test.erl 1 file changed, 6 insertions(+), 10 deletions(-)

Approvals: laforge: Looks good to me, approved pespin: Looks good to me, but someone else must approve Jenkins Builder: Verified

diff --git a/test/s1ap_proxy_test.erl b/test/s1ap_proxy_test.erl index 9534980..5f6167d 100644 --- a/test/s1ap_proxy_test.erl +++ b/test/s1ap_proxy_test.erl @@ -168,14 +168,10 @@ %% [eNB <- MME] E-RAB RELEASE COMMAND %% TODO: make E-RAB IDs configurable e_rab_release_cmd_pdu() -> - << 16#00, 16#07, 16#00, 16#37, 16#00, 16#00, 16#05, 16#00, - 16#00, 16#00, 16#02, 16#00, 16#01, 16#00, 16#08, 16#00, - 16#02, 16#00, 16#00, 16#00, 16#42, 16#00, 16#0a, 16#18, - 16#3e, 16#80, 16#00, 16#00, 16#60, 16#3e, 16#80, 16#00, - 16#00, 16#00, 16#21, 16#40, 16#07, 16#00, 16#00, 16#23, - 16#40, 16#02, 16#0c, 16#40, 16#00, 16#1a, 16#40, 16#0b, - 16#0a, 16#27, 16#d3, 16#76, 16#43, 16#e7, 16#07, 16#72, - 16#47, 16#cd, 16#24 + << 16#00, 16#07, 16#00, 16#1a, 16#00, 16#00, 16#03, 16#00, + 16#00, 16#00, 16#02, 16#00, 16#07, 16#00, 16#08, 16#00, + 16#02, 16#00, 16#09, 16#00, 16#21, 16#40, 16#07, 16#00, + 16#00, 16#23, 16#40, 16#02, 16#0c, 16#40 >>.

@@ -183,8 +179,8 @@ %% TODO: make E-RAB IDs configurable e_rab_release_rsp_pdu() -> << 16#20, 16#07, 16#00, 16#19, 16#00, 16#00, 16#03, 16#00, - 16#00, 16#40, 16#02, 16#00, 16#02, 16#00, 16#08, 16#40, - 16#02, 16#00, 16#01, 16#00, 16#45, 16#40, 16#06, 16#00, + 16#00, 16#00, 16#02, 16#00, 16#07, 16#00, 16#08, 16#00, + 16#02, 16#00, 16#09, 16#00, 16#45, 16#40, 16#06, 16#00, 16#00, 16#0f, 16#40, 16#01, 16#0c >>.