September 2025 - gerrit-log - lists.osmocom.org

[XS] Change in osmo-ttcn3-hacks[master]: mme: Add missing tests to expected-results.xml

by laforge

laforge has submitted this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41080?usp=email ) Change subject: mme: Add missing tests to expected-results.xml ...................................................................... mme: Add missing tests to expected-results.xml Change-Id: I9fbe1dd102268558f4b360dc8e870117e4e17096 --- M mme/expected-results.xml 1 file changed, 7 insertions(+), 1 deletion(-) Approvals: fixeria: Looks good to me, approved Jenkins Builder: Verified diff --git a/mme/expected-results.xml b/mme/expected-results.xml index 370aa0e..6826995 100644 --- a/mme/expected-results.xml +++ b/mme/expected-results.xml @@ -1,7 +1,13 @@ <?xml version="1.0"?> -<testsuite name='Titan' tests='71' failures='8' errors='0' skipped='0' inconc='0' time='MASKED'> +<testsuite name='Titan' tests='10' failures='0' errors='0' skipped='0' inconc='0' time='MASKED'> <testcase classname='MME_Tests' name='TC_s1ap_setup_unknown_global_enb_id_plmn' time='MASKED'/> <testcase classname='MME_Tests' name='TC_s1ap_setup_wrong_tac' time='MASKED'/> <testcase classname='MME_Tests' name='TC_s1ap_setup' time='MASKED'/> <testcase classname='MME_Tests' name='TC_s1ap_attach' time='MASKED'/> + <testcase classname='MME_Tests' name='TC_s1ap_tau_unknown_guti' time='MASKED'/> + <testcase classname='MME_Tests' name='TC_gn_echo_request' time='MASKED'/> + <testcase classname='MME_Tests' name='TC_RIM_RAN_INF' time='MASKED'/> + <testcase classname='MME_Tests' name='TC_s1ap_reset' time='MASKED'/> + <testcase classname='MME_Tests' name='TC_ue_cell_reselect_eutran_to_geran' time='MASKED'/> + <testcase classname='MME_Tests' name='TC_ue_cell_reselect_geran_to_eutran' time='MASKED'/> </testsuite> -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41080?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: osmo-ttcn3-hacks Gerrit-Branch: master Gerrit-Change-Id: I9fbe1dd102268558f4b360dc8e870117e4e17096 Gerrit-Change-Number: 41080 Gerrit-PatchSet: 2 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org>

1 week, 3 days

1
0
0 0

[M] Change in osmo-ttcn3-hacks[master]: library/NAS_EPS_Templates.ttcn: Add templates to handle CS Fallback r...

by laforge

Attention is currently required from: pespin. laforge has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41081?usp=email ) Change subject: library/NAS_EPS_Templates.ttcn: Add templates to handle CS Fallback related procedures ...................................................................... Patch Set 2: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41081?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-ttcn3-hacks Gerrit-Branch: master Gerrit-Change-Id: If8884224e353a9883200c84d737e63c3a3ca9b33 Gerrit-Change-Number: 41081 Gerrit-PatchSet: 2 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Comment-Date: Fri, 12 Sep 2025 11:58:37 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 week, 3 days

1
0
0 0

[S] Change in osmo-ttcn3-hacks[master]: library/s1ap: Support receiving UEContextModificationRequest

by laforge

Attention is currently required from: pespin. laforge has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41075?usp=email ) Change subject: library/s1ap: Support receiving UEContextModificationRequest ...................................................................... Patch Set 2: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41075?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-ttcn3-hacks Gerrit-Branch: master Gerrit-Change-Id: Iadb80eea7281b3bdf3359b0e2956e18bdb208207 Gerrit-Change-Number: 41075 Gerrit-PatchSet: 2 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Comment-Date: Fri, 12 Sep 2025 11:58:28 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 week, 3 days

1
0
0 0

[XS] Change in osmo-ttcn3-hacks[master]: mme: Mark GTP and diameter emulations alive

by laforge

Attention is currently required from: daniel, lynxis lazus, osmith, pespin. laforge has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41074?usp=email ) Change subject: mme: Mark GTP and diameter emulations alive ...................................................................... Patch Set 2: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41074?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-ttcn3-hacks Gerrit-Branch: master Gerrit-Change-Id: I53851dc981e9395e025cd5d2fb0122bea49564ee Gerrit-Change-Number: 41074 Gerrit-PatchSet: 2 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: daniel <dwillmann(a)sysmocom.de> Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-CC: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Attention: osmith <osmith(a)sysmocom.de> Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Attention: daniel <dwillmann(a)sysmocom.de> Gerrit-Attention: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Comment-Date: Fri, 12 Sep 2025 11:58:20 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 week, 3 days

1
0
0 0

[XS] Change in ...osmo-s1gw[master]: s1gw_metrics: export metric types

by laforge

laforge has submitted this change. ( https://gerrit.osmocom.org/c/erlang/osmo-s1gw/+/40907?usp=email ) Change subject: s1gw_metrics: export metric types ...................................................................... s1gw_metrics: export metric types Change-Id: I57f0a8bbc83ec4732742ec16fe09abd2aea8127b --- M src/s1gw_metrics.erl 1 file changed, 4 insertions(+), 0 deletions(-) Approvals: fixeria: Looks good to me, approved laforge: Looks good to me, but someone else must approve pespin: Looks good to me, but someone else must approve Jenkins Builder: Verified diff --git a/src/s1gw_metrics.erl b/src/s1gw_metrics.erl index 8a74774..7ca3352 100644 --- a/src/s1gw_metrics.erl +++ b/src/s1gw_metrics.erl @@ -108,6 +108,10 @@ -type gauge() :: [gauge | _]. -type metric() :: counter() | gauge(). +-export_type([metric/0, + counter/0, + gauge/0]). + -spec register_all(Type, List) -> Result when Type :: exometer:type(), -- To view, visit https://gerrit.osmocom.org/c/erlang/osmo-s1gw/+/40907?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: I57f0a8bbc83ec4732742ec16fe09abd2aea8127b Gerrit-Change-Number: 40907 Gerrit-PatchSet: 3 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>

1 week, 3 days

1
0
0 0

[S] Change in ...osmo-s1gw[master]: pfcp_peer: add fetch_info/0

by laforge

laforge has submitted this change. ( https://gerrit.osmocom.org/c/erlang/osmo-s1gw/+/40902?usp=email ) Change subject: pfcp_peer: add fetch_info/0 ...................................................................... pfcp_peer: add fetch_info/0 This is useful for introspection and the upcoming REST interface. Change-Id: I3fdcf7f6daac09a8c523826995a610c04f5a327b --- M src/pfcp_peer.erl 1 file changed, 26 insertions(+), 1 deletion(-) Approvals: laforge: Looks good to me, approved pespin: Looks good to me, but someone else must approve Jenkins Builder: Verified diff --git a/src/pfcp_peer.erl b/src/pfcp_peer.erl index efcf0d0..f6e9c7d 100644 --- a/src/pfcp_peer.erl +++ b/src/pfcp_peer.erl @@ -48,6 +48,7 @@ session_establish_req/3, session_modify_req/3, session_delete_req/1, + fetch_info/0, shutdown/0]). -include_lib("kernel/include/logger.hrl"). @@ -69,7 +70,14 @@ -type pfcp_msg() :: {pfcp_msg_type(), pfcp_ies()}. -type pfcp_pdu() :: #pfcp{}. --export_type([pfcp_session_rsp/0, +-type peer_info() :: #{state := atom(), + laddr := inet:ip_address(), + raddr := inet:ip_address(), + lrts := pos_integer(), + rrts => pos_integer()}. + +-export_type([peer_info/0, + pfcp_session_rsp/0, pfcp_msg_type/0, pfcp_seq_nr/0, pfcp_seid/0, @@ -139,6 +147,11 @@ gen_statem:call(?MODULE, {?FUNCTION_NAME, SEID}). +-spec fetch_info() -> peer_info(). +fetch_info() -> + gen_statem:call(?MODULE, ?FUNCTION_NAME). + + shutdown() -> gen_statem:stop(?MODULE). @@ -338,6 +351,18 @@ S1 = registry_del({Pid, Ref}, S0), {keep_state, S1}; +handle_event(State, + {call, From}, fetch_info, + #peer_state{} = S) -> + Info = #{state => State, + laddr => S#peer_state.loc_addr, + raddr => S#peer_state.rem_addr, + lrts => S#peer_state.loc_rts, + rrts => S#peer_state.rem_rts}, + %% omit props with Value =:= undefined + Reply = maps:filter(fun(_K, V) -> V =/= undefined end, Info), + {keep_state_and_data, {reply, From, Reply}}; + handle_event(State, Event, EventData, S) -> ?LOG_ERROR("Unexpected event ~p in state ~p: ~p", [Event, State, EventData]), {keep_state, S}. -- To view, visit https://gerrit.osmocom.org/c/erlang/osmo-s1gw/+/40902?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: I3fdcf7f6daac09a8c523826995a610c04f5a327b Gerrit-Change-Number: 40902 Gerrit-PatchSet: 3 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>

1 week, 3 days

1
0
0 0

[S] Change in ...osmo-s1gw[master]: erab_fsm: make fetch_info/1 return a map

by laforge

laforge has submitted this change. ( https://gerrit.osmocom.org/c/erlang/osmo-s1gw/+/40906?usp=email ) Change subject: erab_fsm: make fetch_info/1 return a map ...................................................................... erab_fsm: make fetch_info/1 return a map Maps are a lot easier to work with and have better type hints. Change-Id: I83999fa9346d19260cb44e0bb6a36b7ee1c12ce1 --- M src/erab_fsm.erl M src/s1ap_proxy.erl M test/erab_fsm_test.erl 3 files changed, 26 insertions(+), 15 deletions(-) Approvals: pespin: Looks good to me, but someone else must approve Jenkins Builder: Verified laforge: Looks good to me, approved diff --git a/src/erab_fsm.erl b/src/erab_fsm.erl index cd4fa27..f5d8efe 100644 --- a/src/erab_fsm.erl +++ b/src/erab_fsm.erl @@ -103,6 +103,17 @@ -type iface() :: 'Core' | 'Access'. -type pfcp_ie() :: map(). +-type erab_info() :: #{uid := term(), + state := atom(), + f_teid_u2c => teid_addr(), + f_teid_c2u => teid_addr(), + f_teid_a2u => teid_addr(), + f_teid_u2a => teid_addr(), + seid_loc := pfcp_peer:pfcp_seid(), + seid_rem => pfcp_peer:pfcp_seid()}. + +-export_type([erab_info/0]). + %% ------------------------------------------------------------------ %% public API @@ -205,7 +216,7 @@ gen_statem:cast(Pid, ?FUNCTION_NAME). --spec fetch_info(pid()) -> proplists:proplist(). +-spec fetch_info(pid()) -> erab_info(). fetch_info(Pid) -> gen_statem:call(Pid, ?FUNCTION_NAME). @@ -542,16 +553,16 @@ %% Event handler for all states handle_event(State, {call, From}, fetch_info, #erab_state{} = S) -> - Info = [{state, State}, - {uid, S#erab_state.uid}, - {f_teid_u2c, S#erab_state.u2c}, - {f_teid_c2u, S#erab_state.c2u}, - {f_teid_a2u, S#erab_state.a2u}, - {f_teid_u2a, S#erab_state.u2a}, - {seid_loc, S#erab_state.seid_loc}, - {seid_rem, S#erab_state.seid_rem}], - %% omit props with Value =:= undefined - Reply = lists:filter(fun({_, V}) -> V =/= undefined end, Info), + Info = #{state => State, + uid => S#erab_state.uid, + f_teid_u2c => S#erab_state.u2c, + f_teid_c2u => S#erab_state.c2u, + f_teid_a2u => S#erab_state.a2u, + f_teid_u2a => S#erab_state.u2a, + seid_loc => S#erab_state.seid_loc, + seid_rem => S#erab_state.seid_rem}, + %% omit fields with Value =:= undefined + Reply = maps:filter(fun(_K, V) -> V =/= undefined end, Info), {keep_state_and_data, {reply, From, Reply}}; handle_event(State, {call, From}, EventData, _S) -> diff --git a/src/s1ap_proxy.erl b/src/s1ap_proxy.erl index fd51b2c..658f08c 100644 --- a/src/s1ap_proxy.erl +++ b/src/s1ap_proxy.erl @@ -967,12 +967,12 @@ {ok, Pid} -> %% Ignore F-TEID in the original message, just replace it with C2U F-TEID Info = erab_fsm:fetch_info(Pid), - case proplists:get_value(f_teid_c2u, Info) of - {TEID, TLA} -> + case maps:find(f_teid_c2u, Info) of + {ok, {TEID, TLA}} -> C1 = C0#'E-RABNotToBeModifiedItemBearerModInd'{'transportLayerAddress' = TLA, 'dL-GTP-TEID' = << TEID:32/big >>}, {{ok, C1}, S}; - undefined -> + error -> ?LOG_ERROR("E-RAB-ID ~p :: missing C2U F-TEID", [erab_uid(ERABId, S)]), {{error, missing_f_teid}, S} end; diff --git a/test/erab_fsm_test.erl b/test/erab_fsm_test.erl index a3dfb4d..e49d5cc 100644 --- a/test/erab_fsm_test.erl +++ b/test/erab_fsm_test.erl @@ -35,7 +35,7 @@ erab_fsm_info(Pid, Key) -> Info = erab_fsm:fetch_info(Pid), - proplists:get_value(Key, Info). + maps:get(Key, Info). %% ------------------------------------------------------------------ -- To view, visit https://gerrit.osmocom.org/c/erlang/osmo-s1gw/+/40906?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: I83999fa9346d19260cb44e0bb6a36b7ee1c12ce1 Gerrit-Change-Number: 40906 Gerrit-PatchSet: 3 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>

1 week, 3 days

1
0
0 0

[XS] Change in ...osmo-s1gw[master]: {gtpu_kpi,pfcp_peer}: do not set trap_exit

by laforge

laforge has submitted this change. ( https://gerrit.osmocom.org/c/erlang/osmo-s1gw/+/41033?usp=email ) Change subject: {gtpu_kpi,pfcp_peer}: do not set trap_exit ...................................................................... {gtpu_kpi,pfcp_peer}: do not set trap_exit We do not expect nor handle 'EXIT' messages in these modules, therefore we should not be setting trap_exit. Change-Id: I966f4a53b8eca6763ca30bae0bbb9e025e6a2c4f --- M src/gtpu_kpi.erl M src/pfcp_peer.erl 2 files changed, 0 insertions(+), 2 deletions(-) Approvals: laforge: Looks good to me, approved Jenkins Builder: Verified pespin: Looks good to me, but someone else must approve diff --git a/src/gtpu_kpi.erl b/src/gtpu_kpi.erl index 5aeb749..9432626 100644 --- a/src/gtpu_kpi.erl +++ b/src/gtpu_kpi.erl @@ -146,7 +146,6 @@ %% ------------------------------------------------------------------ init([#{enable := true} = Cfg]) -> - process_flag(trap_exit, true), TName = maps:get(table_name, Cfg, "osmo-s1gw"), Interval = maps:get(interval, Cfg, 3000), %% flush the table, in case it remained diff --git a/src/pfcp_peer.erl b/src/pfcp_peer.erl index a7e9515..efcf0d0 100644 --- a/src/pfcp_peer.erl +++ b/src/pfcp_peer.erl @@ -156,7 +156,6 @@ init([LocAddr, RemAddr]); init([LocAddr, RemAddr]) -> - process_flag(trap_exit, true), s1gw_metrics:gauge_set(?S1GW_GAUGE_PFCP_ASSOCIATED, 0), {ok, Sock} = gen_udp:open(?PFCP_PORT, [binary, {ip, LocAddr}, -- To view, visit https://gerrit.osmocom.org/c/erlang/osmo-s1gw/+/41033?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: I966f4a53b8eca6763ca30bae0bbb9e025e6a2c4f Gerrit-Change-Number: 41033 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>

1 week, 3 days

1
0
0 0

[S] Change in ...osmo-s1gw[master]: sctp_proxy: make fetch_info/1 return a map

by laforge

laforge has submitted this change. ( https://gerrit.osmocom.org/c/erlang/osmo-s1gw/+/40905?usp=email ) Change subject: sctp_proxy: make fetch_info/1 return a map ...................................................................... sctp_proxy: make fetch_info/1 return a map Maps are a lot easier to work with and have better type hints. Change-Id: I24029faa7e44efe34599b46fae731c801ff6191d --- M src/sctp_proxy.erl 1 file changed, 18 insertions(+), 7 deletions(-) Approvals: Jenkins Builder: Verified laforge: Looks good to me, approved pespin: Looks good to me, but someone else must approve diff --git a/src/sctp_proxy.erl b/src/sctp_proxy.erl index b9258c3..830fe91 100644 --- a/src/sctp_proxy.erl +++ b/src/sctp_proxy.erl @@ -53,6 +53,18 @@ -include("s1gw_metrics.hrl"). + +-type conn_info() :: #{state := atom(), + handler := pid(), + mme_addr := inet:ip_address(), + mme_port := inet:port_number(), + enb_aid := gen_sctp:assoc_id(), + mme_aid => gen_sctp:assoc_id() + }. + +-export_type([conn_info/0]). + + %% ------------------------------------------------------------------ %% public API %% ------------------------------------------------------------------ @@ -70,7 +82,7 @@ gen_statem:cast(Pid, {send_data, Data}). --spec fetch_info(pid()) -> proplists:proplist(). +-spec fetch_info(pid()) -> conn_info(). fetch_info(Pid) -> gen_statem:call(Pid, ?FUNCTION_NAME). @@ -207,12 +219,11 @@ handle_event(State, {call, From}, fetch_info, #{handler := Pid, conn_cfg := ConnCfg} = S0) -> S1 = maps:with([enb_aid, mme_aid], S0), - S2 = S1#{state => State, - handler => Pid, - mme_addr => maps:get(raddr, ConnCfg), - mme_port => maps:get(rport, ConnCfg)}, - Reply = proplists:from_map(S2), - {keep_state_and_data, {reply, From, Reply}}; + Info = S1#{state => State, + handler => Pid, + mme_addr => maps:get(raddr, ConnCfg), + mme_port => maps:get(rport, ConnCfg)}, + {keep_state_and_data, {reply, From, Info}}; handle_event(State, {call, From}, EventData, _S) -> ?LOG_ERROR("Unexpected call in state ~p: ~p", [State, EventData]), -- To view, visit https://gerrit.osmocom.org/c/erlang/osmo-s1gw/+/40905?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: I24029faa7e44efe34599b46fae731c801ff6191d Gerrit-Change-Number: 40905 Gerrit-PatchSet: 3 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>

1 week, 3 days

1
0
0 0

[XS] Change in ...osmo-s1gw[master]: sctp_proxy: return handler Pid in fetch_info/0

by laforge

laforge has submitted this change. ( https://gerrit.osmocom.org/c/erlang/osmo-s1gw/+/40904?usp=email ) Change subject: sctp_proxy: return handler Pid in fetch_info/0 ...................................................................... sctp_proxy: return handler Pid in fetch_info/0 Instead of returning the E-RAB list, give the caller access to the associated s1ap_proxy process by returning its Pid. This allows the caller to use API provided by the s1ap_proxy directly. Change-Id: I3911bfc107fea3ca572cf44f1d93b768b3c82b4f --- M src/sctp_proxy.erl 1 file changed, 1 insertion(+), 2 deletions(-) Approvals: laforge: Looks good to me, approved Jenkins Builder: Verified pespin: Looks good to me, but someone else must approve diff --git a/src/sctp_proxy.erl b/src/sctp_proxy.erl index badadcd..b9258c3 100644 --- a/src/sctp_proxy.erl +++ b/src/sctp_proxy.erl @@ -206,10 +206,9 @@ %% Event handler for all states handle_event(State, {call, From}, fetch_info, #{handler := Pid, conn_cfg := ConnCfg} = S0) -> - ERABs = s1ap_proxy:fetch_erab_list(Pid), %% TODO: separate API S1 = maps:with([enb_aid, mme_aid], S0), S2 = S1#{state => State, - erab_list => ERABs, + handler => Pid, mme_addr => maps:get(raddr, ConnCfg), mme_port => maps:get(rport, ConnCfg)}, Reply = proplists:from_map(S2), -- To view, visit https://gerrit.osmocom.org/c/erlang/osmo-s1gw/+/40904?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: I3911bfc107fea3ca572cf44f1d93b768b3c82b4f Gerrit-Change-Number: 40904 Gerrit-PatchSet: 3 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>

1 week, 3 days

1
0
0 0

2025

2024

2023

2022

gerrit-log September 2025