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
>.
--
To view, visit
https://gerrit.osmocom.org/c/erlang/osmo-s1gw/+/38240?usp=email
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: merged
Gerrit-Project: erlang/osmo-s1gw
Gerrit-Branch: master
Gerrit-Change-Id: I229c47148fd7408e44f1339ae2a2909f570c73ec
Gerrit-Change-Number: 38240
Gerrit-PatchSet: 2
Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>