Attention is currently required from: msuraev.
osmith has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-manuals/+/30715 )

Change subject: rate counter: add StatsD note
......................................................................


Patch Set 2:

(5 comments)

File common/chapters/counters-overview.adoc:

https://gerrit.osmocom.org/c/osmo-gsm-manuals/+/30715/comment/b6e9fb54_fbe21bc1
PS2, Line 127: does
please use the same max line height as the rest of the file, it seems to be < 
80 characters


https://gerrit.osmocom.org/c/osmo-gsm-manuals/+/30715/comment/ef86da2d_bed078f6
PS2, Line 129: StatsD
* throughout the file it's written as "statsd"
* missing "the"


https://gerrit.osmocom.org/c/osmo-gsm-manuals/+/30715/comment/49c1ec0a_887fce54
PS2, Line 130: via
maybe add a colon at the end, so it's a full sentence? "via:"

same below before the other URL


https://gerrit.osmocom.org/c/osmo-gsm-manuals/+/30715/comment/85c005cd_b20dce89
PS2, Line 130: Netdata
the Netdata dashboard


https://gerrit.osmocom.org/c/osmo-gsm-manuals/+/30715/comment/99bc2a63_57c9f266
PS2, Line 131: statsD isavailable
statsd is available



--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-manuals/+/30715
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-manuals
Gerrit-Branch: master
Gerrit-Change-Id: I124503c3707bbe005abbfb4245abe2829c6ff57c
Gerrit-Change-Number: 30715
Gerrit-PatchSet: 2
Gerrit-Owner: msuraev <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: osmith <[email protected]>
Gerrit-Attention: msuraev <[email protected]>
Gerrit-Comment-Date: Thu, 22 Dec 2022 14:42:57 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment

Reply via email to