Attention is currently required from: Hoernchen, pespin.
fixeria has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/27602 )
Change subject: ran emulation: allow multiple reset attempts
......................................................................
Patch Set 1: Code-Review-1
(3 comments)
Patchset:
PS1:
CR-1 due to missing 'mp_' prefix.
File library/RAN_Emulation.ttcnpp:
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/27602/comment/4ea2ef22_7356…
PS1, Line 526: while (attempts > 0) {
> This is usually implemented in TTCN3 by means of "repeat;" keyword in the alt.
IMO, the loop approach is easier to read. Not only you would need to 'repeat', but also restart the timer and send the ts_BSSMAP_Reset again in the timeout altstep. But personally I would rather use 'for' here, it's a bit shorter:
for (var integer i := 0; i < g_ran_ops.bssap_reset_retries; i := i + 1) { ... }
Not critical, just a matter of preferences. You can keep this code as is.
File msc/MSC_Tests.ttcn:
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/27602/comment/4f7665eb_dd38…
PS1, Line 143: bssap_reset_retries
All module parameters star with 'mp_', please align the naming.
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/27602
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: I9f6db2a24e984eef31e76f9d42a80eb6a1bb6928
Gerrit-Change-Number: 27602
Gerrit-PatchSet: 1
Gerrit-Owner: Hoernchen <ewild(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-CC: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: Hoernchen <ewild(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Wed, 30 Mar 2022 13:50:42 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Comment-In-Reply-To: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: comment
Attention is currently required from: Hoernchen.
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/27602 )
Change subject: ran emulation: allow multiple reset attempts
......................................................................
Patch Set 1:
(1 comment)
File library/RAN_Emulation.ttcnpp:
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/27602/comment/5572de80_283b…
PS1, Line 526: while (attempts > 0) {
This is usually implemented in TTCN3 by means of "repeat;" keyword in the alt.
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/27602
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: I9f6db2a24e984eef31e76f9d42a80eb6a1bb6928
Gerrit-Change-Number: 27602
Gerrit-PatchSet: 1
Gerrit-Owner: Hoernchen <ewild(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: Hoernchen <ewild(a)sysmocom.de>
Gerrit-Comment-Date: Wed, 30 Mar 2022 12:50:52 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Attention is currently required from: pespin.
neels has posted comments on this change. ( https://gerrit.osmocom.org/c/libosmo-sccp/+/27599 )
Change subject: Add ss7 instances to osmo_ss7_instances in creation order
......................................................................
Patch Set 2: Code-Review+1
(1 comment)
Patchset:
PS2:
should not cause downstream test fallout. did you check that?
--
To view, visit https://gerrit.osmocom.org/c/libosmo-sccp/+/27599
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: libosmo-sccp
Gerrit-Branch: master
Gerrit-Change-Id: Id4d0a20cc5b0811b505b2d1051d496f8bd17d54c
Gerrit-Change-Number: 27599
Gerrit-PatchSet: 2
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: neels <nhofmeyr(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Wed, 30 Mar 2022 12:02:17 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
neels has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bsc/+/27595 )
Change subject: hodec2: show reported ho oscillation from bad rxqual
......................................................................
Patch Set 3: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/27595
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: I51f3101901492ae742d8e56d6eed2725408216cb
Gerrit-Change-Number: 27595
Gerrit-PatchSet: 3
Gerrit-Owner: neels <nhofmeyr(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: neels <nhofmeyr(a)sysmocom.de>
Gerrit-Comment-Date: Wed, 30 Mar 2022 11:56:16 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
Hoernchen has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/27603 )
Change subject: msc tests: fix test so they don't depend on previous test runs
......................................................................
msc tests: fix test so they don't depend on previous test runs
Change-Id: Iebaecd28a426b15baf4729f40b46dd33da79cbae
---
M msc/MSC_Tests.ttcn
1 file changed, 7 insertions(+), 7 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/03/27603/1
diff --git a/msc/MSC_Tests.ttcn b/msc/MSC_Tests.ttcn
index 73029d8..0e4a2a2 100644
--- a/msc/MSC_Tests.ttcn
+++ b/msc/MSC_Tests.ttcn
@@ -1170,7 +1170,7 @@
f_expect_mm_info();
/* wait for normal teardown */
- f_expect_clear();
+ f_expect_clear(verify_vlr_cell_id := false);
/* Now the same IMSI is still attached in the VLR, and a LU with an unknown TMSI reveals the same IMSI only
* later during ID Response. osmo-msc first creates a new vlr_subscr for the unknown TMSI, and as soon as the
@@ -1207,7 +1207,7 @@
f_expect_mm_info();
/* wait for normal teardown */
- f_expect_clear();
+ f_expect_clear(verify_vlr_cell_id := false);
}
testcase TC_attached_imsi_lu_unknown_tmsi() runs on MTC_CT {
var BSC_ConnHdlr vc_conn;
@@ -4410,7 +4410,7 @@
mtc.stop;
}
}
- f_expect_clear();
+ f_expect_clear(verify_vlr_cell_id:=false);
}
testcase TC_lu_with_invalid_mcc_mnc() runs on MTC_CT {
var BSC_ConnHdlr vc_conn;
@@ -6386,7 +6386,7 @@
f_msc_lu_hlr();
f_mm_imei();
f_expect_lu_reject();
- f_expect_clear();
+ f_expect_clear(verify_vlr_cell_id:=false);
}
testcase TC_lu_imsi_auth_tmsi_check_imei_nack() runs on MTC_CT {
var BSC_ConnHdlr vc_conn;
@@ -6416,7 +6416,7 @@
f_msc_lu_hlr();
f_mm_imei();
f_expect_lu_reject();
- f_expect_clear();
+ f_expect_clear(verify_vlr_cell_id:=false);
}
testcase TC_lu_imsi_auth_tmsi_check_imei_err() runs on MTC_CT {
var BSC_ConnHdlr vc_conn;
@@ -6507,7 +6507,7 @@
f_mm_imei_early();
f_expect_lu_reject();
- f_expect_clear();
+ f_expect_clear(verify_vlr_cell_id:=false);
}
testcase TC_lu_imsi_auth_tmsi_check_imei_early_nack() runs on MTC_CT {
var BSC_ConnHdlr vc_conn;
@@ -6535,7 +6535,7 @@
f_mm_imei_early();
f_expect_lu_reject();
- f_expect_clear();
+ f_expect_clear(verify_vlr_cell_id:=false);
}
testcase TC_lu_imsi_auth_tmsi_check_imei_early_err() runs on MTC_CT {
var BSC_ConnHdlr vc_conn;
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/27603
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: Iebaecd28a426b15baf4729f40b46dd33da79cbae
Gerrit-Change-Number: 27603
Gerrit-PatchSet: 1
Gerrit-Owner: Hoernchen <ewild(a)sysmocom.de>
Gerrit-MessageType: newchange
pespin has submitted this change. ( https://gerrit.osmocom.org/c/osmo-gsm-tester/+/27601 )
Change subject: contrib: Fix osmocom-bb-fimware no longer available in order prebuilt pkg repo
......................................................................
contrib: Fix osmocom-bb-fimware no longer available in order prebuilt pkg repo
roox: "please use the Tumbleweed repo instead if you want something that is always available"
Change-Id: I88843e7ed35c42e9b16aaff00c9c6a3973d0d6ca
---
M contrib/jenkins-build-osmocom-bb.sh
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
pespin: Looks good to me, approved; Verified
diff --git a/contrib/jenkins-build-osmocom-bb.sh b/contrib/jenkins-build-osmocom-bb.sh
index 827fc94..aa2e778 100755
--- a/contrib/jenkins-build-osmocom-bb.sh
+++ b/contrib/jenkins-build-osmocom-bb.sh
@@ -5,7 +5,7 @@
name="osmocom-bb"
. "$(dirname "$0")/jenkins-build-common.sh"
-FW_RPM_URL="http://download.opensuse.org/repositories/home:/mnhauke:/osmocom:/nightly/S…"
+FW_RPM_URL="http://download.opensuse.org/repositories/home:/mnhauke:/osmocom:/nightly/o…"
build_repo libosmocore --disable-doxygen
build_repo_dir osmocom-bb src/host/virt_phy ${SANITIZE_FLAGS}
--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/27601
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I88843e7ed35c42e9b16aaff00c9c6a3973d0d6ca
Gerrit-Change-Number: 27601
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: merged
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-gsm-tester/+/27601 )
Change subject: contrib: Fix osmocom-bb-fimware no longer available in order prebuilt pkg repo
......................................................................
Patch Set 1: Verified+1 Code-Review+2
--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/27601
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I88843e7ed35c42e9b16aaff00c9c6a3973d0d6ca
Gerrit-Change-Number: 27601
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Tue, 29 Mar 2022 16:03:22 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment