neels has submitted this change. ( https://gerrit.osmocom.org/c/osmo-bsc/+/22264 )
Change subject: fixate test_dyn_ts_congestion_tch_f_vs_tch_h.ho_vty ...................................................................... fixate test_dyn_ts_congestion_tch_f_vs_tch_h.ho_vty The test shows a cascade of failures. When we fix the first failure, it will change the sequence of events that follow after that. But it will still be interesting to evaluate all the situations currently shown. Hence fixate each stage's initial situation, by duplicating the expect-ts-use with an identical set-ts-use. Then, when each individual scenario gets fixed, subsequent scenarios still remain unchanged. Change-Id: Ifeaec39ecb64b476ff1438cf987ba0403489c43b --- M tests/handover/test_dyn_ts_congestion_tch_f_vs_tch_h.ho_vty 1 file changed, 24 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/test_dyn_ts_congestion_tch_f_vs_tch_h.ho_vty b/tests/handover/test_dyn_ts_congestion_tch_f_vs_tch_h.ho_vty index 1273d75..bc2c884 100644 --- a/tests/handover/test_dyn_ts_congestion_tch_f_vs_tch_h.ho_vty +++ b/tests/handover/test_dyn_ts_congestion_tch_f_vs_tch_h.ho_vty @@ -25,6 +25,12 @@ expect-ts-use trx 0 0 states * - - - pdch pdch pdch * expect-ts-use trx 1 0 states * TCH/F TCH/F TCH/F TCH/HH TCH/F pdch * + + +### + +set-ts-use trx 0 0 states * - - - pdch pdch pdch * +set-ts-use trx 1 0 states * TCH/F TCH/F TCH/F TCH/HH TCH/F pdch * meas-rep lchan * * * * rxlev 40 rxqual 0 ta 0 neighbors 30 congestion-check @@ -34,12 +40,24 @@ expect-ts-use trx 0 0 states * - - - pdch pdch pdch * expect-ts-use trx 1 0 states * TCH/F TCH/F TCH/F TCH/H- TCH/F TCH/F * + +### + +set-ts-use trx 0 0 states * - - - pdch pdch pdch * +set-ts-use trx 1 0 states * TCH/F TCH/F TCH/F TCH/H- TCH/F TCH/F * + congestion-check # more FAIL: TCH/H moves to worse bts 0 due to congestion expect-ho from lchan 1 0 4 0 to lchan 0 0 4 0 expect-ts-use trx 0 0 states * - - - TCH/H- pdch pdch * expect-ts-use trx 1 0 states * TCH/F TCH/F TCH/F pdch TCH/F TCH/F * + +### + +set-ts-use trx 0 0 states * - - - TCH/H- pdch pdch * +set-ts-use trx 1 0 states * TCH/F TCH/F TCH/F pdch TCH/F TCH/F * + congestion-check expect-no-chan @@ -53,6 +71,12 @@ expect-ts-use trx 0 0 states * - - - pdch pdch pdch * expect-ts-use trx 1 0 states * TCH/F TCH/F TCH/F TCH/F TCH/F TCH/F * + +### + +set-ts-use trx 0 0 states * - - - pdch pdch pdch * +set-ts-use trx 1 0 states * TCH/F TCH/F TCH/F TCH/F TCH/F TCH/F * + congestion-check expect-no-chan # Stable situation now only because TCH/F has min-free-slots set to 0 -- To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/22264 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-bsc Gerrit-Branch: master Gerrit-Change-Id: Ifeaec39ecb64b476ff1438cf987ba0403489c43b Gerrit-Change-Number: 22264 Gerrit-PatchSet: 2 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