April 2024 - gerrit-log - lists.osmocom.org

[M] Change in osmo-ttcn3-hacks[master]: sgsn: add TC_attach_rau_invalid_old_rai
by pespin 22 Apr '24

22 Apr '24

Attention is currently required from: fixeria. pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/36625?usp=email ) Change subject: sgsn: add TC_attach_rau_invalid_old_rai ...................................................................... Patch Set 2: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/36625?usp=email 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: I006adabdad26c0c48e1ebe99421bcab7295dc147 Gerrit-Change-Number: 36625 Gerrit-PatchSet: 2 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Comment-Date: Mon, 22 Apr 2024 14:02:43 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

1 0

[M] Change in osmo-ttcn3-hacks[master]: sgsn: TC_sgsn_context_req_in: match PDP Context IE
by pespin 22 Apr '24

22 Apr '24

Attention is currently required from: daniel, fixeria, laforge. pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/36601?usp=email ) Change subject: sgsn: TC_sgsn_context_req_in: match PDP Context IE ...................................................................... Patch Set 6: (1 comment) File sgsn/SGSN_Tests.ttcn: https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/36601/comment/6ebdc670_7f77… PS6, Line 3748: f_pdp_ctx_act(apars); > [packet 478] GGSN (the testsuite) responds with Create PDP Context Response, but not including the End user address IE; This looks wrong to me. The EUA should be there. I see no reason why the GGSN would not assign the EUA. > The testsuite (virtual GGSN) assigns no IP address, and osmo-sgsn happily accepts that. I don't know if this is right or wrong, you're saying it's not right? Ack. One would need to closely read the specs to see if that's allowed, but for sure it's not what we want in these test cases. It's possibly simply some missing code in the ttcn3 code emulating the ggsn. Go fix that :) -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/36601?usp=email 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: I965c0200ea380c55dc0cd8d3f8e5110883ce2646 Gerrit-Change-Number: 36601 Gerrit-PatchSet: 6 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-CC: daniel <dwillmann(a)sysmocom.de> Gerrit-Attention: laforge <laforge(a)osmocom.org> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Attention: daniel <dwillmann(a)sysmocom.de> Gerrit-Comment-Date: Mon, 22 Apr 2024 14:02:04 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: laforge <laforge(a)osmocom.org> Comment-In-Reply-To: fixeria <vyanitskiy(a)sysmocom.de> Comment-In-Reply-To: pespin <pespin(a)sysmocom.de> Gerrit-MessageType: comment

1 0

[M] Change in docker-playground[master]: ttcn3-sgsn-test: add config files for running SNS testcases
by pespin 22 Apr '24

22 Apr '24

Attention is currently required from: fixeria, laforge, osmith. pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/docker-playground/+/36623?usp=email ) Change subject: ttcn3-sgsn-test: add config files for running SNS testcases ...................................................................... Patch Set 2: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/c/docker-playground/+/36623?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: docker-playground Gerrit-Branch: master Gerrit-Change-Id: I767ec223585002d6420eda27ed338722f764c902 Gerrit-Change-Number: 36623 Gerrit-PatchSet: 2 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)gnumonks.org> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: osmith <osmith(a)sysmocom.de> Gerrit-Attention: laforge <laforge(a)gnumonks.org> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Comment-Date: Mon, 22 Apr 2024 13:59:29 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

1 0

[M] Change in docker-playground[master]: ttcn3-sgsn-test: add config files for running SNS testcases
by pespin 22 Apr '24

22 Apr '24

Attention is currently required from: fixeria, laforge, osmith. pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/docker-playground/+/36623?usp=email ) Change subject: ttcn3-sgsn-test: add config files for running SNS testcases ...................................................................... Patch Set 2: (1 comment) File ttcn3-sgsn-test/sns/SGSN_Tests.cfg: https://gerrit.osmocom.org/c/docker-playground/+/36623/comment/916a140e_35a… PS2, Line 46: SGSN_Tests_NS.control > "you are assing" -- lol, what?!? 😄 […] Yeah yeah, I found out NS vs SNS :) -- To view, visit https://gerrit.osmocom.org/c/docker-playground/+/36623?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: docker-playground Gerrit-Branch: master Gerrit-Change-Id: I767ec223585002d6420eda27ed338722f764c902 Gerrit-Change-Number: 36623 Gerrit-PatchSet: 2 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)gnumonks.org> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-CC: pespin <pespin(a)sysmocom.de> Gerrit-Attention: osmith <osmith(a)sysmocom.de> Gerrit-Attention: laforge <laforge(a)gnumonks.org> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Comment-Date: Mon, 22 Apr 2024 13:59:19 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: fixeria <vyanitskiy(a)sysmocom.de> Comment-In-Reply-To: pespin <pespin(a)sysmocom.de> Gerrit-MessageType: comment

1 0

[M] Change in osmo-ttcn3-hacks[master]: sgsn: add TC_attach_rau_invalid_old_rai
by fixeria 22 Apr '24

22 Apr '24

Hello Jenkins Builder, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/36625?usp=email to look at the new patch set (#2). The following approvals got outdated and were removed: Verified+1 by Jenkins Builder Change subject: sgsn: add TC_attach_rau_invalid_old_rai ...................................................................... sgsn: add TC_attach_rau_invalid_old_rai This testcase currently crashes osmo-sgsn. Change-Id: I006adabdad26c0c48e1ebe99421bcab7295dc147 Related: OS#6441 --- M sgsn/SGSN_Tests.ttcn 1 file changed, 57 insertions(+), 0 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/25/36625/2 -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/36625?usp=email 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: I006adabdad26c0c48e1ebe99421bcab7295dc147 Gerrit-Change-Number: 36625 Gerrit-PatchSet: 2 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-MessageType: newpatchset

1 0

[M] Change in osmo-ttcn3-hacks[master]: sgsn: add TC_attach_rau_invalid_old_rai
by fixeria 22 Apr '24

22 Apr '24

fixeria has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/36625?usp=email ) Change subject: sgsn: add TC_attach_rau_invalid_old_rai ...................................................................... sgsn: add TC_attach_rau_invalid_old_rai This testcase currently crashes osmo-sgsn. Change-Id: I006adabdad26c0c48e1ebe99421bcab7295dc147 --- M sgsn/SGSN_Tests.ttcn 1 file changed, 56 insertions(+), 0 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/25/36625/1 diff --git a/sgsn/SGSN_Tests.ttcn b/sgsn/SGSN_Tests.ttcn index 0371e29..d55a264 100644 --- a/sgsn/SGSN_Tests.ttcn +++ b/sgsn/SGSN_Tests.ttcn @@ -2755,6 +2755,49 @@ f_cleanup(); } +private function f_TC_attach_rau_invalid_old_rai(charstring id) runs on BSSGP_ConnHdlr { + timer T; + + /* perform regular attach */ + f_TC_attach(id); + + /* perform RAU via cell A */ + f_routing_area_update(g_pars.ra); + + f_bssgp_client_unregister(g_pars.imsi); + f_bssgp_client_register(g_pars.imsi, g_pars.tlli, BSSGP_PROC[1]); + + /* perform RAU via cell B, but indicating the new RAI as the old RAI */ + var RoutingAreaIdentificationV old_ra := f_cellid_to_RAI(g_pars.bssgp_cell_id[1]); + f_send_l3(ts_GMM_RAU_REQ(f_mi_get_lv(), GPRS_UPD_T_RA, old_ra), 1); + + T.start(2.0); + alt { + [] BSSGP[1].receive(tr_GMM_RAU_REJECT) { + setverdict(pass); + T.stop; + } + [] BSSGP[1].receive(tr_LLC_XID_MT_CMD(?, ?)) { repeat; } + [] T.timeout { + setverdict(fail, "Timeout waiting for RAU Reject"); + mtc.stop; + } + } + + /* perform RAU via cell B, this time with proper old RAI of cell A */ + f_routing_area_update(old_ra := g_pars.ra, ran_index := 1); + + f_detach_mo(c_GMM_DTT_MO_GPRS, true, true, 1); +} +testcase TC_attach_rau_invalid_old_rai() runs on test_CT { + var BSSGP_ConnHdlr vc_conn; + f_init(); + f_sleep(1.0); + vc_conn := f_start_handler(refers(f_TC_attach_rau_invalid_old_rai), testcasename(), g_gb, 138); + vc_conn.done; + f_cleanup(); +} + private function f_TC_attach_gmm_attach_req_while_gmm_attach(charstring id) runs on BSSGP_ConnHdlr { var integer count_req := 0; var MobileIdentityLV mi; @@ -3984,6 +4027,8 @@ /* At the end, may crash osmo-sgsn, see OS#3957, OS#4245 */ execute( TC_attach_req_id_req_ra_update() ); + /* At the end, may crash osmo-sgsn */ + execute( TC_attach_rau_invalid_old_rai() ); } -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/36625?usp=email 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: I006adabdad26c0c48e1ebe99421bcab7295dc147 Gerrit-Change-Number: 36625 Gerrit-PatchSet: 1 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-MessageType: newchange

1 0

[M] Change in meta-telephony[201705]: Remove User= and Group= from systemd service files
by fixeria 22 Apr '24

22 Apr '24

Attention is currently required from: osmith. fixeria has posted comments on this change. ( https://gerrit.osmocom.org/c/meta-telephony/+/36618?usp=email ) Change subject: Remove User= and Group= from systemd service files ...................................................................... Patch Set 2: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/meta-telephony/+/36618?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: meta-telephony Gerrit-Branch: 201705 Gerrit-Change-Id: I68a8fba034aeb0e2923b2958b8d6842410d8b23c Gerrit-Change-Number: 36618 Gerrit-PatchSet: 2 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: osmith <osmith(a)sysmocom.de> Gerrit-Comment-Date: Mon, 22 Apr 2024 13:07:48 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

1 0

[M] Change in osmo-ttcn3-hacks[master]: sgsn: TC_sgsn_context_req_in: match PDP Context IE
by fixeria 22 Apr '24

22 Apr '24

Attention is currently required from: daniel, laforge, pespin. fixeria has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/36601?usp=email ) Change subject: sgsn: TC_sgsn_context_req_in: match PDP Context IE ...................................................................... Patch Set 6: (1 comment) File sgsn/SGSN_Tests.ttcn: https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/36601/comment/68b78d5c_0321… PS6, Line 3748: f_pdp_ctx_act(apars); > @axilirator@gmail. […] I believe PCAPs would do a better job explaining what's going on. https://osmocom.org/attachments/download/8165/SGSN_Tests.TC_sgsn_context_re… -- here is a PCAP file recorded with the previous patch revision (5). * [packet 456] MS sends GMM `Activate PDP Context Request` with the `Requested PDP Address` IE being empty (wireshark displays `Dynamic addressing`); * [packet 476] SGSN sends GTPC `Create PDP Context Request` with the `End user address` IE being empty (expecting GGSN to assign it?); * [packet 478] GGSN (the testsuite) responds with `Create PDP Context Response`, but **not including** the `End user address` IE; * [packet 487] SGSN responds with GMM `Activate PDP Context Accept` with the `PDP Address` IE being empty (wireshark displays `Dynamic addressing`); * [packet 489] the testsuite sends `SGSN Context Request`; * [packet 492] SGSN responds with `SGSN Context Response` containing no `PDP Context` IE -- this makes the testcase fail. This is basically a), `apars.addr.addressInfo := omit` by default. This is also the case for the other (existing) testcases in SGSN_Tests.ttcn, see https://jenkins.osmocom.org/jenkins/view/TTCN3/job/ttcn3-sgsn-test/2318/art… for instance. Or any other `SGSN_Tests.TC_attach_pdp_act_...` testcase. The testsuite (virtual GGSN) assigns no IP address, and osmo-sgsn happily accepts that. I don't know if this is right or wrong, you're saying it's not right? -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/36601?usp=email 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: I965c0200ea380c55dc0cd8d3f8e5110883ce2646 Gerrit-Change-Number: 36601 Gerrit-PatchSet: 6 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-CC: daniel <dwillmann(a)sysmocom.de> Gerrit-Attention: laforge <laforge(a)osmocom.org> Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Attention: daniel <dwillmann(a)sysmocom.de> Gerrit-Comment-Date: Mon, 22 Apr 2024 13:04:38 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: laforge <laforge(a)osmocom.org> Comment-In-Reply-To: pespin <pespin(a)sysmocom.de> Comment-In-Reply-To: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-MessageType: comment

1 0

[M] Change in docker-playground[master]: ttcn3-sgsn-test: add config files for running SNS testcases
by fixeria 22 Apr '24

22 Apr '24

Attention is currently required from: laforge, osmith, pespin. fixeria has posted comments on this change. ( https://gerrit.osmocom.org/c/docker-playground/+/36623?usp=email ) Change subject: ttcn3-sgsn-test: add config files for running SNS testcases ...................................................................... Patch Set 2: (1 comment) File ttcn3-sgsn-test/sns/SGSN_Tests.cfg: https://gerrit.osmocom.org/c/docker-playground/+/36623/comment/ee0bf1b5_115… PS2, Line 46: SGSN_Tests_NS.control > See, you are assing it here, why adding it to the other already exiting SGSN_Tests. […] "you are assing" -- lol, what?!? 😄 See my comment in previous commit. -- To view, visit https://gerrit.osmocom.org/c/docker-playground/+/36623?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: docker-playground Gerrit-Branch: master Gerrit-Change-Id: I767ec223585002d6420eda27ed338722f764c902 Gerrit-Change-Number: 36623 Gerrit-PatchSet: 2 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)gnumonks.org> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-CC: pespin <pespin(a)sysmocom.de> Gerrit-Attention: osmith <osmith(a)sysmocom.de> Gerrit-Attention: laforge <laforge(a)gnumonks.org> Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Comment-Date: Mon, 22 Apr 2024 12:53:36 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: pespin <pespin(a)sysmocom.de> Gerrit-MessageType: comment

1 0

[S] Change in docker-playground[master]: ttcn3-sgsn-test: add missing SGSN_Tests_NS.control
by pespin 22 Apr '24

22 Apr '24

Attention is currently required from: fixeria. pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/docker-playground/+/36622?usp=email ) Change subject: ttcn3-sgsn-test: add missing SGSN_Tests_NS.control ...................................................................... Patch Set 2: Code-Review+1 (1 comment) Patchset: PS2: > ``` […] AH! NS vs SNS, got it 😄 -- To view, visit https://gerrit.osmocom.org/c/docker-playground/+/36622?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: docker-playground Gerrit-Branch: master Gerrit-Change-Id: I605526dff6d3c0ec4c5dc58dc00ca51b4ccd68a7 Gerrit-Change-Number: 36622 Gerrit-PatchSet: 2 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Comment-Date: Mon, 22 Apr 2024 12:52:41 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes Comment-In-Reply-To: fixeria <vyanitskiy(a)sysmocom.de> Comment-In-Reply-To: pespin <pespin(a)sysmocom.de> Gerrit-MessageType: comment

1 0

2025

2024

2023

2022

gerrit-log April 2024