Attention is currently required from: laforge, pespin.

View Change

1 comment:

• Patchset:

  • Patch Set #2:

    you are missing the use of array indexes in this API verifying rate counters via the ctrl interface. […]

    I thought some more about this, and a nicer API would actually be one where the object identification is also part of the array entry. It could be a string, then callers could enter "msc.0" or "hnb.by_name.A1B2C3".

    That would be nice, but I currently prefer not to spend time on our testing API cosmetics, and just use this patch that is quickest to success.

    There is another compound to this: when we revert this patch to osmo-hnbgw
    https://gerrit.osmocom.org/c/osmo-hnbgw/+/38002
    then the object identification behavior does not change, and we don't currently need these patches here applied at all to fix our test suite.

    (The general mismatch of using "obj.N" for volatile objects still exists, but it would not show in the test suite, because osmo-hnbgw will always snap back to hnb.0.)

To view, visit change 37882. To unsubscribe, or for help writing mail filters, visit settings.