Max has posted comments on this change. ( https://gerrit.osmocom.org/3895 )

Change subject: TS alloc: expand tests log
......................................................................


Patch Set 28:

(1 comment)

Should have replied year ago but thanks to underwhelming pace of OsmoPCU 
development this patch is still relevant :)

https://gerrit.osmocom.org/#/c/3895/27/tests/alloc/AllocTest.err
File tests/alloc/AllocTest.err:

https://gerrit.osmocom.org/#/c/3895/27/tests/alloc/AllocTest.err@196
PS27, Line 196: [DL] algo A <single> (suggested TRX: -1): failed to find a 
usable TRX (TFI exhausted)
> why does the old log line statte "multi" and the new one "single"? This looks 
> suspicious. […]
Erm, old log doesn't state "multi" - at least I don't see where. In fact the 
old log does not give any info about single/multi allocation being used and 
that's exactly what this patch is trying to address.



--
To view, visit https://gerrit.osmocom.org/3895
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-pcu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I4b78951a79ddbc0745b39d091080a4e0e247d3c5
Gerrit-Change-Number: 3895
Gerrit-PatchSet: 28
Gerrit-Owner: Max <[email protected]>
Gerrit-Reviewer: Harald Welte <[email protected]>
Gerrit-Reviewer: Jenkins Builder (1000002)
Gerrit-Reviewer: Max <[email protected]>
Gerrit-Comment-Date: Tue, 26 Mar 2019 12:59:24 +0000
Gerrit-HasComments: Yes
Gerrit-HasLabels: No

Reply via email to