Attention is currently required from: fixeria, pespin. msuraev has posted comments on this change. ( https://gerrit.osmocom.org/c/osmocom-bb/+/31298 )
Change subject: mobile: add optional statsd support ......................................................................
Patch Set 1:
(2 comments)
Commit Message:
https://gerrit.osmocom.org/c/osmocom-bb/+/31298/comment/39a1856b_e586fc48 PS1, Line 7: statsd support
You're not adding any actual stats items, but adding the VTY commands only. […]
No such plans at the moment.
File src/host/layer23/include/osmocom/bb/common/l23_app.h:
https://gerrit.osmocom.org/c/osmocom-bb/+/31298/comment/82367cc9_77037949 PS1, Line 17: L23_OPT_STAT
Why making this optional command line option? If some app indicates L23_OPT_VTY, then I guess it mak […]
I'm fine with adding stats commands to all apps with vty as well. What others think - how fine-grained should it be?
-
msuraev