Attention is currently required from: osmith, pespin.
fixeria has posted comments on this change by fixeria. ( https://gerrit.osmocom.org/c/erlang/osmo-s1gw/+/42365?usp=email )
Change subject: doc/manuals: document the metrics ......................................................................
Patch Set 5:
(3 comments)
File doc/manuals/chapters/metrics.adoc:
https://gerrit.osmocom.org/c/erlang/osmo-s1gw/+/42365/comment/290def47_e2029... : PS4, Line 5: https://github.com/Feuerlabs/exometer_core%5Bexometer_core] library. Two
"OsmoS1GW exposes internal metrics using the […]
I believe it's important to mention it here, so I've removed the library reference in `configuration.adoc`. I've also mentioned the StatsD reporter plugin here.
https://gerrit.osmocom.org/c/erlang/osmo-s1gw/+/42365/comment/d7c22b20_0ebf6... : PS4, Line 25:
is this explanation of how the stats are stored internally useful to the user?
Right, this is more relevant for developers. I've reworked this section explaining the hierarchical nature of the metric names.
https://gerrit.osmocom.org/c/erlang/osmo-s1gw/+/42365/comment/5685d006_8f130... : PS4, Line 68: | `s1ap.proxy.in_pkt.erab_setup_req` | E-RAB SETUP REQUEST PDUs
How about: […]
Done