Attention is currently required from: neels, pespin. laforge has posted comments on this change by neels. ( https://gerrit.osmocom.org/c/libosmocore/+/37881?usp=email ) Change subject: CTRL: get rate_ctr_group by the id from rate_ctr_group_set_name() ...................................................................... Patch Set 4: (3 comments) Patchset: PS2: * I agree on the "can be optimized later without ABI breakage" * I agree in general it would be better to perform tests of interfaces that our users actually use (stats reporter), but even if that is set to 1s reporting interval, it would mean an (up to) one second latency every time we'd want to check if some counte has incremented as per the tests expectations. We really don't want to slow down our test suite. The CTRL interface has the advantage that it's a request-response interface that gives us the definitive value right now. Patchset: PS4: fine with everything except the one issue about splitting the patch File src/core/rate_ctr.c: https://gerrit.osmocom.org/c/libosmocore/+/37881/comment/9852a592_788870fb?… : PS2, Line 284: osmo_quote_str(name, -1)); this should probably be a separate commit. experience with changes like these shows that we have existing applications breaking with this change, so we might want the capability to test (or revert) that change separately. -- To view, visit https://gerrit.osmocom.org/c/libosmocore/+/37881?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: libosmocore Gerrit-Branch: master Gerrit-Change-Id: I841a12f76e6fcb2bd7aecb5f4b1707d9ec927137 Gerrit-Change-Number: 37881 Gerrit-PatchSet: 4 Gerrit-Owner: neels <nhofmeyr(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: neels <nhofmeyr(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-CC: laforge <laforge(a)osmocom.org> Gerrit-Attention: neels <nhofmeyr(a)sysmocom.de> Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Comment-Date: Fri, 23 Aug 2024 06:43:36 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: neels <nhofmeyr(a)sysmocom.de> Comment-In-Reply-To: laforge <laforge(a)osmocom.org>