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've been thinking about the main purpose for this: our test suite. […]

* 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?u... : 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.