fixeria submitted this change.

View Change



2 is the latest approved patch-set.
No files were changed between the latest approved patch-set and the submitted one.

Approvals: Jenkins Builder: Verified pespin: Looks good to me, approved
sctp_proxy: fix calling gauge_dec() on a counter

A follow-up commit will add type hints to prevent such errors.

Change-Id: Ic1e29061838e9c96cc4687308c6c56db7011ae9f
---
M src/sctp_proxy.erl
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/sctp_proxy.erl b/src/sctp_proxy.erl
index 280e408..c901b80 100644
--- a/src/sctp_proxy.erl
+++ b/src/sctp_proxy.erl
@@ -258,7 +258,7 @@

sctp_send_pending([Data | Pending], S0) ->
S1 = sctp_send(Data, S0),
- s1gw_metrics:gauge_dec(?S1GW_CTR_S1AP_PROXY_UPLINK_PACKETS_QUEUED),
+ s1gw_metrics:gauge_dec(?S1GW_GAUGE_S1AP_PROXY_UPLINK_PACKETS_QUEUED),
sctp_send_pending(Pending, S1);

sctp_send_pending([], S) ->

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

Gerrit-MessageType: merged
Gerrit-Project: erlang/osmo-s1gw
Gerrit-Branch: master
Gerrit-Change-Id: Ic1e29061838e9c96cc4687308c6c56db7011ae9f
Gerrit-Change-Number: 38296
Gerrit-PatchSet: 5
Gerrit-Owner: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>