fixeria has uploaded this change for review. ( https://gerrit.osmocom.org/c/erlang/osmo_dia2gsup/+/38428?usp=email )
Change subject: diameter: assert that diameter:start_service/2 returns ok
......................................................................
diameter: assert that diameter:start_service/2 returns ok
I introduced a regression in 1545311, which would have been spotted
much earlier if we had such an assertion in place. Add it.
Change-Id: Ief13353d280e3eb80a25b5c9a9effdf537bcc704
Related: 1545311 "diameter: fix unused macro param in ?SERVICE"
---
M src/osmo_dia2gsup.erl
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://gerrit.osmocom.org:29418/erlang/osmo_dia2gsup refs/changes/28/38428/1
diff --git a/src/osmo_dia2gsup.erl b/src/osmo_dia2gsup.erl
index 26376a4..b8d7a34 100644
--- a/src/osmo_dia2gsup.erl
+++ b/src/osmo_dia2gsup.erl
@@ -97,7 +97,7 @@
init(State) ->
% DIAMETER side
ProdName = atom_to_list(?MODULE),
- diameter:start_service(?SVC_NAME, ?SERVICE(ProdName)),
+ ok = diameter:start_service(?SVC_NAME, ?SERVICE(ProdName)),
Ip = application:get_env(osmo_dia2gsup, diameter_ip, "127.0.0.8"),
Port = application:get_env(osmo_dia2gsup, diameter_port, 3868),
--
To view, visit https://gerrit.osmocom.org/c/erlang/osmo_dia2gsup/+/38428?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: newchange
Gerrit-Project: erlang/osmo_dia2gsup
Gerrit-Branch: master
Gerrit-Change-Id: Ief13353d280e3eb80a25b5c9a9effdf537bcc704
Gerrit-Change-Number: 38428
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de>
Attention is currently required from: laforge.
pespin has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/upf-benchmark/+/38354?usp=email )
Change subject: Rename osmo-pfcp-tool to osmo-upf-genload
......................................................................
Patch Set 1:
(1 comment)
Patchset:
PS1:
> we already have rtp-load-gen (osmo-mgw) gtp-load-gen (separate git repo), so it might make sense to […]
Ack I'll rename it.
--
To view, visit https://gerrit.osmocom.org/c/upf-benchmark/+/38354?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: upf-benchmark
Gerrit-Branch: master
Gerrit-Change-Id: Ie36379b5dcf4ab86276dfda9e25410fe41db1d93
Gerrit-Change-Number: 38354
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Attention: laforge <laforge(a)osmocom.org>
Gerrit-Comment-Date: Mon, 14 Oct 2024 15:08:04 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: laforge <laforge(a)osmocom.org>
Comment-In-Reply-To: pespin <pespin(a)sysmocom.de>