Attention is currently required from: neels, pespin.

laforge has posted comments on this change by neels. ( 
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/37882?usp=email )

Change subject: ctrl, hnbgw: access rate counter groups by given ID instead of 
index
......................................................................


Patch Set 4: Code-Review+1

(1 comment)

Patchset:

PS4:
There's now also the proposal to revert to the original behaviour (always use 
numeric index 0): https://gerrit.osmocom.org/c/osmo-hnbgw/+/38002

Ideally libosmocore should enforce the "unique ID" rule only if there are no 
unique names being set.  However, our API first creates the objcet with numeric 
ID, and then optionally sets the name in a second call - so there's no point 
where we can print a related error/warning message 😞



--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/37882?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings?usp=email

Gerrit-MessageType: comment
Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: I70e74e7554482df67aa1d90bc04314124dea444f
Gerrit-Change-Number: 37882
Gerrit-PatchSet: 4
Gerrit-Owner: neels <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-CC: pespin <[email protected]>
Gerrit-Attention: neels <[email protected]>
Gerrit-Attention: pespin <[email protected]>
Gerrit-Comment-Date: Wed, 04 Sep 2024 06:57:21 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes

Reply via email to