neels has submitted this change. ( https://gerrit.osmocom.org/c/osmo-bsc/+/22268 )
Change subject: show bug: add test_dyn_ts_balance_congestion.ho_vty ...................................................................... show bug: add test_dyn_ts_balance_congestion.ho_vty Related: SYS#5298 Change-Id: I34471fbd490a95253bd0709308a42cde2af6481c --- M tests/handover/handover_tests.ok A tests/handover/test_dyn_ts_balance_congestion.ho_vty 2 files changed, 32 insertions(+), 0 deletions(-) Approvals: Jenkins Builder: Verified laforge: Looks good to me, approved pespin: Looks good to me, but someone else must approve diff --git a/tests/handover/handover_tests.ok b/tests/handover/handover_tests.ok index e1e35a9..2658f8b 100644 --- a/tests/handover/handover_tests.ok +++ b/tests/handover/handover_tests.ok @@ -17,6 +17,7 @@ pass test_congestion_no_oscillation2.ho_vty pass test_disabled_ho_and_as.ho_vty pass test_dyn_ts_amr_tch_f_to_h_congestion_assignment.ho_vty +pass test_dyn_ts_balance_congestion.ho_vty pass test_dyn_ts_congestion_tch_f_vs_tch_h.ho_vty pass test_dyn_ts_congestion_tch_f_vs_tch_h_2.ho_vty pass test_dyn_ts_favor_half_used_tch_h_as_target.ho_vty diff --git a/tests/handover/test_dyn_ts_balance_congestion.ho_vty b/tests/handover/test_dyn_ts_balance_congestion.ho_vty new file mode 100644 index 0000000..ad9d6a5 --- /dev/null +++ b/tests/handover/test_dyn_ts_balance_congestion.ho_vty @@ -0,0 +1,31 @@ +# To balance congestion, consider cross effects between TCH/F and TCH/H when occupying a dynamic timeslot in the target: +# when balancing of TCH/F congestion would take up a dyn TS in the target, reducing TCH/H availability, the handover +# should not cause worse TCH/H congestion than in the source cell. + +create-bts trx-count 1 timeslots c+s4 TCH/F TCH/F TCH/F TCH/F TCH/F TCH/F PDCH +create-bts trx-count 1 timeslots c+s4 TCH/F TCH/F TCH/F TCH/F dyn dyn PDCH + +# for this test, avoid changing a TCH/F to a TCH/H by using a non-AMR codec +codec tch/f FR + +network + bts 0 + handover2 min-free-slots tch/f 2 + bts 1 + handover2 min-free-slots tch/f 4 + handover2 min-free-slots tch/h 4 + +set-ts-use trx 0 0 states * TCH/F TCH/F TCH/F TCH/F TCH/F TCH/F * +set-ts-use trx 1 0 states * TCH/F TCH/F TCH/F TCH/F pdch pdch * + +meas-rep lchan * * * * rxlev 40 rxqual 0 ta 0 neighbors 30 +expect-no-chan + +# bts 0 is full for TCH/F. Looking at TCH/F, by target_overbooked_after_ho==75% < current_overbooked_before_ho==100%, a +# congestion balancing to bts 1 would be performed. But the TCH/F on the target cell would occupy a dynamic timeslot. +# That would reduce the TCH/H free slots by two and cause TCH/H being overbooked by 50%. On the source cell, TCH/H is +# not congested. No handover is performed because 50% in the target is more congestion for TCH/H than 0% in the source +# cell. +congestion-check +# FAIL: should not increase TCH/H congestion by occupying a dyn TS +expect-ho from lchan 0 0 1 0 to lchan 1 0 5 0 -- To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/22268 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-bsc Gerrit-Branch: master Gerrit-Change-Id: I34471fbd490a95253bd0709308a42cde2af6481c Gerrit-Change-Number: 22268 Gerrit-PatchSet: 4 Gerrit-Owner: neels <nhofm...@sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <lafo...@osmocom.org> Gerrit-Reviewer: neels <nhofm...@sysmocom.de> Gerrit-Reviewer: pespin <pes...@sysmocom.de> Gerrit-MessageType: merged
