Attention is currently required from: neels, pespin.
laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-hnbgw/+/36539?usp=email )
Change subject: per-HNB GTP-U traffic counters via nft ......................................................................
Patch Set 12:
(3 comments)
File src/osmo-hnbgw/hnbgw_hnbap.c:
https://gerrit.osmocom.org/c/osmo-hnbgw/+/36539/comment/9af84649_52b5f700 PS6, Line 571: hnb_persistent_connected(ctx->persistent);
It really depends on the specific event you want to track here. […]
I agree pespin's wording is more clear.
File src/osmo-hnbgw/hnbgw_vty.c:
https://gerrit.osmocom.org/c/osmo-hnbgw/+/36539/comment/09c3a806_e0667d5b PS12, Line 885: "nft-kpi [TABLE_NAME]", shouldn't this be in some kind of #ifdef block to suppress offering those vty commands in case the program was not compiled with nftables support?
File src/osmo-hnbgw/nft_kpi.c:
https://gerrit.osmocom.org/c/osmo-hnbgw/+/36539/comment/7158d6e7_658478a9 PS9, Line 994: nft_kpi_get_counters_schedule();
I really think we want to at least do as I described, which is really quick to implement (adding one […]
I agree with neels that his approach implements what we discussed (for the reasons we discussed and he stated). Yes, it's not ideal from the user point of view, but let's not be perfect at the cost of dragging this another week.