laforge submitted this change.

View Change


Approvals: laforge: Looks good to me, approved fixeria: Looks good to me, but someone else must approve Jenkins Builder: Verified
gsup: Log connect failures

Change-Id: I273d37aa75cfacbc7c3302d035c2391d9bdd9b2e
---
M src/gsup_client.erl
1 file changed, 12 insertions(+), 2 deletions(-)

diff --git a/src/gsup_client.erl b/src/gsup_client.erl
index d18138a..50a9bf6 100644
--- a/src/gsup_client.erl
+++ b/src/gsup_client.erl
@@ -83,9 +83,10 @@
true = ipa_proto:register_stream(Socket, ?IPAC_PROTO_EXT_GSUP, {process_id, self()}),
ipa_proto:unblock(Socket),
{ok, #gsupc_state{socket=Socket, ipa_pid=IpaPid}};
- {error, econnrefused} ->
+ {error, Reason} ->
+ lager:error("Connecting to GSUP HLR on IP ~s port ~p failed: ~p~n", [Address, Port, Reason]),
timer:sleep(5000),
- {stop, connrefused}
+ {stop, Reason}
end.



To view, visit change 34198. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: erlang/osmo_dia2gsup
Gerrit-Branch: master
Gerrit-Change-Id: I273d37aa75cfacbc7c3302d035c2391d9bdd9b2e
Gerrit-Change-Number: 34198
Gerrit-PatchSet: 4
Gerrit-Owner: pespin <pespin@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: daniel <dwillmann@sysmocom.de>
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: lynxis lazus <lynxis@fe80.eu>
Gerrit-MessageType: merged