pespin has submitted this change. (
https://gerrit.osmocom.org/c/erlang/osmo-epdg/+/36116?usp=email )
Change subject: epdg_ue_fsm: Increase timeout waiting for gtp response
......................................................................
epdg_ue_fsm: Increase timeout waiting for gtp response
Answers can take a while specially during CreateSession, where several
nodes are involved, like PGW, AAA, HSS, PCRF, etc.
Change-Id: I3c8e7c7551bb3d48c46c0195c02cb44ce222a17f
---
M src/epdg_ue_fsm.erl
1 file changed, 15 insertions(+), 1 deletion(-)
Approvals:
Jenkins Builder: Verified
laforge: Looks good to me, but someone else must approve
pespin: Looks good to me, approved
diff --git a/src/epdg_ue_fsm.erl b/src/epdg_ue_fsm.erl
index 439cb3f..94f2a39 100644
--- a/src/epdg_ue_fsm.erl
+++ b/src/epdg_ue_fsm.erl
@@ -49,6 +49,8 @@
state_wait_create_session_resp/3, state_wait_delete_session_resp/3,
state_wait_swm_session_termination_answer/3, state_active/3]).
+-define(TIMEOUT_VAL_WAIT_GTP_ANSWER, 10000).
+
-record(ue_fsm_data, {
imsi,
apn = "internet" :: string(),
@@ -274,7 +276,7 @@
{keep_state, Data}.
state_wait_create_session_resp(enter, _OldState, Data) ->
- {keep_state, Data, [{state_timeout,5000,create_session_timeout}]};
+ {keep_state, Data,
[{state_timeout,?TIMEOUT_VAL_WAIT_GTP_ANSWER,create_session_timeout}]};
state_wait_create_session_resp({call, From},
{received_gtpc_create_session_response, Result}, Data) ->
lager:info("ue_fsm state_authenticated
event=received_gtpc_create_session_response, ~p~n", [Data]),
--
To view, visit https://gerrit.osmocom.org/c/erlang/osmo-epdg/+/36116?usp=email
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: erlang/osmo-epdg
Gerrit-Branch: master
Gerrit-Change-Id: I3c8e7c7551bb3d48c46c0195c02cb44ce222a17f
Gerrit-Change-Number: 36116
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pes...@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <lafo...@osmocom.org>
Gerrit-Reviewer: pespin <pes...@sysmocom.de>
Gerrit-MessageType: merged
