Attention is currently required from: neels.
pespin has posted comments on this change by pespin. (
https://gerrit.osmocom.org/c/osmo-msc/+/40633?usp=email )
Change subject: ran_peer: Introuce stats msc.ran_peers.{total,active}
......................................................................
Patch Set 4:
(2 comments)
File include/osmocom/msc/gsm_data.h:
https://gerrit.osmocom.org/c/osmo-msc/+/40633/comment/67d27499_f1864a66?usp… :
PS4, Line 127: [MSC_STAT_RAN_PEERS_TOTAL] = { "msc.ran_peers.total",
"Total RAN peers (BSC, RNC)", OSMO_STAT_ITEM_NO_UNIT, 4, 0},
(maybe add "seen"? like "Total RAN
peers seen since startup" ... […]
Done
File src/libmsc/ran_peer.c:
https://gerrit.osmocom.org/c/osmo-msc/+/40633/comment/76651dc2_8240905e?usp… :
PS4, Line 95: osmo_stat_item_dec(osmo_stat_item_group_get_item(net->statg,
MSC_STAT_RAN_PEERS_TOTAL), 1);
you already count the state change in onleave(), so
just make sure to change the state before/on dea […]
Currently this FSM has a
DISCARDING state exactly for the purpose of not being used 😄
I was even tempted submitting a patch removing it, but I didn't want to spend even
more time changing existing stuff.
--
To view, visit
https://gerrit.osmocom.org/c/osmo-msc/+/40633?usp=email
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: osmo-msc
Gerrit-Branch: master
Gerrit-Change-Id: I36e40510c9a95f0c9cf5f32d2a7baab840aa8660
Gerrit-Change-Number: 40633
Gerrit-PatchSet: 4
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: neels <nhofmeyr(a)sysmocom.de>
Gerrit-Attention: neels <nhofmeyr(a)sysmocom.de>
Gerrit-Comment-Date: Mon, 14 Jul 2025 15:53:46 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: neels <nhofmeyr(a)sysmocom.de>