Attention is currently required from: laforge, pespin.
6 comments:
File config/sys.config:
Patch Set #8, Line 24: %% {enft_kpi_enable, true}, %% whether to enable the NFT KPI module (default: false)
where does the "e" from "enft" come from? At first sight looks like "nft" would be much more compreh […]
It's because I named the library `enftables` (Erlang NIFs for libnftables), and this `e` prefix made it here too (the code is using `enftables`, so the module name is `enft_kpi` and thus param names).
File include/s1gw_metrics.hrl:
Patch Set #8, Line 40: %% NOTE: these counters shall not be listed in ?S1GW_COUNTERS,
So simply this can be announced as "Per-eNB counters"?
Right. Other counters will be made per-eNB too at some point.
I am adding a NOTE here because a NOTE below states that new entries need to be added to `s1gw_metric:init()` (actually `?S1GW_COUNTERS` or `?S1GW_GAUGES`). I don't think it hurts?
File rebar.config:
Patch Set #8, Line 13: {git, "https://gitea.osmocom.org/vyanitskiy/enftables.git", {branch, "fixeria/json"}}},
can we have this in a "osmocom/master" or "master" branch instead?
The whole repository currently lives in my personal Gitea profile and will be migrated to https://gitea.osmocom.org/erlang/. I'll also merge this branch into `master`. I'll do this once the customer completes initial testing.
File src/enft_kpi.erl:
Patch Set #8, Line 142: R1 = [nft_expr_match_ip_proto("udp", ?OP_NEQ), nft_expr_accept()],
sounds like you may want to introduce atoms in the nft lib to express OP_NEQ and OP_EQ instead.
Actually, a lot of this stuff can be moved to `enftables`. But for now it can live here.
Patch Set #8, Line 341: case enb_add_nft_counters(ES0#{addr => Addr}, Cfg) of
fyi this can take some time, and afaict you are doing it synchronously (handle_call), which means th […]
Good point. I'll migrate this to use `cast` instead of `call`.
File src/osmo_s1gw_sup.erl:
Patch Set #8, Line 81: EnftKpi = {enft_kpi, {enft_kpi, start_link, [enft_kpi_cfg()]},
I?m still wondering why this is called "enft" but above it's called "Pfcp" and not "Epfcp" or "Sctp" […]
See the other comment.
To view, visit change 40281. To unsubscribe, or for help writing mail filters, visit settings.