Attention is currently required from: neels.
iedemam has posted comments on this change. (
https://gerrit.osmocom.org/c/osmo-bsc/+/27081 )
Change subject: stats: new trackers for lchan life duration
......................................................................
Patch Set 18:
(1 comment)
Patchset:
PS18:
Hi Neels,
I appreciate the review. I currently have reduced availability due to some vacation days
but will address the comments and update the patchset early next week.
One clarification before I address the comments: I definitely want to record milliseconds
(if only for clarity...no one uses decaseconds) but don't need to update the timer
each millisecond. Is it possible to keep resolution at milliseconds but only trigger
updates every 1/10th of a second? This would be ideal to me.
Everything else looks good to me. I just need a chance to implement + post back here.
Thanks again,
-Michael
--
To view, visit
https://gerrit.osmocom.org/c/osmo-bsc/+/27081
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: I1b0670c47cb5e0b7776eda89d1e71545ba0e3347
Gerrit-Change-Number: 27081
Gerrit-PatchSet: 18
Gerrit-Owner: iedemam <michael(a)kapsulate.com>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: neels <nhofmeyr(a)sysmocom.de>
Gerrit-CC: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-CC: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: neels <nhofmeyr(a)sysmocom.de>
Gerrit-Comment-Date: Fri, 01 Apr 2022 13:31:57 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment