Change in ...osmo-ttcn3-hacks[master]: Revert "bts: f_init_pcu: sleep before connect"

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

osmith gerrit-no-reply at lists.osmocom.org
Wed Sep 4 09:49:45 UTC 2019


osmith has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/15401


Change subject: Revert "bts: f_init_pcu: sleep before connect"
......................................................................

Revert "bts: f_init_pcu: sleep before connect"

This reverts commit c089b415f5e3bbf5eb2de435b925581ca42d8872. The
additional sleep caused other tests to break, probably because it
triggered race conditions:
* TC_pcu_socket_connect_multi
* TC_pcu_socket_connect_si3gprs
* TC_si_sched_13_2bis_2ter_2quater

Adjust TC_pcu_socket_verify_info_ind test case error message to mention
OS#4179. This test is flapping now, most of the time the BTS sends a
CellID 0 because it did not receive the real CellID from the BSC yet.

Related: OS#4179
Change-Id: I2115c337f4601a4614b140715323c42803b003ee
---
M bts/BTS_Tests.ttcn
1 file changed, 3 insertions(+), 4 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/01/15401/1

diff --git a/bts/BTS_Tests.ttcn b/bts/BTS_Tests.ttcn
index 15948a3..2c11ca2 100644
--- a/bts/BTS_Tests.ttcn
+++ b/bts/BTS_Tests.ttcn
@@ -324,9 +324,6 @@
 		pcu_conn_id := -1;
 		return;
 	}
-
-	f_sleep(0.5); /* workaround for OS#4179 */
-
 	pcu_conn_id := f_pcuif_connect(pt, mp_pcu_socket);
 
 	T.start;
@@ -4876,7 +4873,9 @@
 	var uint16_t cell_id_si3 := si3.payload.si3.cell_id;
 	var uint16_t cell_id_pcu := g_pcu_last_info.u.info_ind.cell_id;
 	if (cell_id_si3 != cell_id_pcu) {
-		setverdict(fail, "Expected cell_id ", cell_id_si3, " got: ", cell_id_pcu);
+		setverdict(fail, "Expected cell_id '", cell_id_si3, "' and got '", cell_id_pcu, "'. This either means,",
+				 " that the BTS is sending the wrong cell_id, or that the BTS sent it too early",
+				 " (OS#4179)");
 	}
 
 	/* Verify LAC */

-- 
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/15401
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: I2115c337f4601a4614b140715323c42803b003ee
Gerrit-Change-Number: 15401
Gerrit-PatchSet: 1
Gerrit-Owner: osmith <osmith at sysmocom.de>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20190904/82444730/attachment.htm>


More information about the gerrit-log mailing list