fixeria submitted this change.

Approvals:
  Jenkins Builder: Verified
  pespin: Looks good to me, approved

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(-)

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 change 38428. To unsubscribe, or for help writing mail filters, visit settings.