fixeria has submitted this change. (
https://gerrit.osmocom.org/c/erlang/osmo-s1gw/+/42484?usp=email )
Change subject: gtpu_kpi: use ?ENV_DEFAULT_GTPU_KPI_{TABLE_NAME,INTERVAL}
......................................................................
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(-)
Approvals:
laforge: Looks good to me, but someone else must approve
pespin: Looks good to me, approved
Jenkins Builder: Verified
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 https://gerrit.osmocom.org/c/erlang/osmo-s1gw/+/42484?usp=email
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings?usp=email
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 <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <[email protected]>
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-Reviewer: pespin <[email protected]>
