<p>neels <strong>submitted</strong> this change.</p><p><a href="https://gerrit.osmocom.org/c/osmo-bsc/+/22268">View Change</a></p><div style="white-space:pre-wrap">Approvals:
  Jenkins Builder: Verified
  laforge: Looks good to me, approved
  pespin: Looks good to me, but someone else must approve

</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">show bug: add test_dyn_ts_balance_congestion.ho_vty<br><br>Related: SYS#5298<br>Change-Id: I34471fbd490a95253bd0709308a42cde2af6481c<br>---<br>M tests/handover/handover_tests.ok<br>A tests/handover/test_dyn_ts_balance_congestion.ho_vty<br>2 files changed, 32 insertions(+), 0 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/tests/handover/handover_tests.ok b/tests/handover/handover_tests.ok</span><br><span>index e1e35a9..2658f8b 100644</span><br><span>--- a/tests/handover/handover_tests.ok</span><br><span>+++ b/tests/handover/handover_tests.ok</span><br><span>@@ -17,6 +17,7 @@</span><br><span> pass test_congestion_no_oscillation2.ho_vty</span><br><span> pass test_disabled_ho_and_as.ho_vty</span><br><span> pass test_dyn_ts_amr_tch_f_to_h_congestion_assignment.ho_vty</span><br><span style="color: hsl(120, 100%, 40%);">+pass test_dyn_ts_balance_congestion.ho_vty</span><br><span> pass test_dyn_ts_congestion_tch_f_vs_tch_h.ho_vty</span><br><span> pass test_dyn_ts_congestion_tch_f_vs_tch_h_2.ho_vty</span><br><span> pass test_dyn_ts_favor_half_used_tch_h_as_target.ho_vty</span><br><span>diff --git a/tests/handover/test_dyn_ts_balance_congestion.ho_vty b/tests/handover/test_dyn_ts_balance_congestion.ho_vty</span><br><span>new file mode 100644</span><br><span>index 0000000..ad9d6a5</span><br><span>--- /dev/null</span><br><span>+++ b/tests/handover/test_dyn_ts_balance_congestion.ho_vty</span><br><span>@@ -0,0 +1,31 @@</span><br><span style="color: hsl(120, 100%, 40%);">+# To balance congestion, consider cross effects between TCH/F and TCH/H when occupying a dynamic timeslot in the target:</span><br><span style="color: hsl(120, 100%, 40%);">+# when balancing of TCH/F congestion would take up a dyn TS in the target, reducing TCH/H availability, the handover</span><br><span style="color: hsl(120, 100%, 40%);">+# should not cause worse TCH/H congestion than in the source cell.</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+create-bts trx-count 1 timeslots c+s4 TCH/F TCH/F TCH/F TCH/F TCH/F TCH/F PDCH</span><br><span style="color: hsl(120, 100%, 40%);">+create-bts trx-count 1 timeslots c+s4 TCH/F TCH/F TCH/F TCH/F dyn   dyn   PDCH</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+# for this test, avoid changing a TCH/F to a TCH/H by using a non-AMR codec</span><br><span style="color: hsl(120, 100%, 40%);">+codec tch/f FR</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+network</span><br><span style="color: hsl(120, 100%, 40%);">+ bts 0</span><br><span style="color: hsl(120, 100%, 40%);">+  handover2 min-free-slots tch/f 2</span><br><span style="color: hsl(120, 100%, 40%);">+ bts 1</span><br><span style="color: hsl(120, 100%, 40%);">+  handover2 min-free-slots tch/f 4</span><br><span style="color: hsl(120, 100%, 40%);">+  handover2 min-free-slots tch/h 4</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+set-ts-use trx 0 0 states        *    TCH/F TCH/F TCH/F TCH/F TCH/F TCH/F *</span><br><span style="color: hsl(120, 100%, 40%);">+set-ts-use trx 1 0 states        *    TCH/F TCH/F TCH/F TCH/F pdch  pdch  *</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+meas-rep lchan * * * * rxlev 40 rxqual 0 ta 0 neighbors 30</span><br><span style="color: hsl(120, 100%, 40%);">+expect-no-chan</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+# bts 0 is full for TCH/F. Looking at TCH/F, by target_overbooked_after_ho==75% < current_overbooked_before_ho==100%, a</span><br><span style="color: hsl(120, 100%, 40%);">+# congestion balancing to bts 1 would be performed. But the TCH/F on the target cell would occupy a dynamic timeslot.</span><br><span style="color: hsl(120, 100%, 40%);">+# 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</span><br><span style="color: hsl(120, 100%, 40%);">+# not congested. No handover is performed because 50% in the target is more congestion for TCH/H than 0% in the source</span><br><span style="color: hsl(120, 100%, 40%);">+# cell.</span><br><span style="color: hsl(120, 100%, 40%);">+congestion-check</span><br><span style="color: hsl(120, 100%, 40%);">+# FAIL: should not increase TCH/H congestion by occupying a dyn TS</span><br><span style="color: hsl(120, 100%, 40%);">+expect-ho from lchan 0 0 1 0 to lchan 1 0 5 0</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bsc/+/22268">change 22268</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.osmocom.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.osmocom.org/c/osmo-bsc/+/22268"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-bsc </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I34471fbd490a95253bd0709308a42cde2af6481c </div>
<div style="display:none"> Gerrit-Change-Number: 22268 </div>
<div style="display:none"> Gerrit-PatchSet: 4 </div>
<div style="display:none"> Gerrit-Owner: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Reviewer: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: merged </div>