fixeria submitted this change.

View Change

Approvals: laforge: Looks good to me, but someone else must approve pespin: Looks good to me, approved Jenkins Builder: Verified
gtpu_kpi: use ?ENV_DEFAULT_GTPU_KPI_{TABLE_NAME,INTERVAL}

Change-Id: Ia8194966b20da1adc26c02f416b8ec0508191d98
---
M src/gtpu_kpi.erl
1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/src/gtpu_kpi.erl b/src/gtpu_kpi.erl
index 9432626..309f554 100644
--- a/src/gtpu_kpi.erl
+++ b/src/gtpu_kpi.erl
@@ -49,6 +49,7 @@

-include_lib("kernel/include/logger.hrl").

+-include("osmo_s1gw.hrl").
-include("s1gw_metrics.hrl").

-define(GTPU_PORT, 2152).
@@ -146,8 +147,8 @@
%% ------------------------------------------------------------------

init([#{enable := true} = Cfg]) ->
- TName = maps:get(table_name, Cfg, "osmo-s1gw"),
- Interval = maps:get(interval, Cfg, 3000),
+ TName = maps:get(table_name, Cfg, ?ENV_DEFAULT_GTPU_KPI_TABLE_NAME),
+ Interval = maps:get(interval, Cfg, ?ENV_DEFAULT_GTPU_KPI_INTERVAL),
%% flush the table, in case it remained
%% it may not exist, so we ignore the result
nft_flush_table(TName),

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

Gerrit-MessageType: merged
Gerrit-Project: erlang/osmo-s1gw
Gerrit-Branch: master
Gerrit-Change-Id: Ia8194966b20da1adc26c02f416b8ec0508191d98
Gerrit-Change-Number: 42484
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>