Attention is currently required from: osmith, pespin.
fixeria has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43378?usp=email )
Change subject: smf: Implement PFCP Sess Est and Sess Del procedures
......................................................................
Patch Set 9: Code-Review+2
(1 comment)
File smf/SMF_Session_CT.ttcn:
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43378/comment/0cbf30d5_755e… :
PS7, Line 680: f_pfcp_sess_del_resp
> No, I meant the function body is somehow tabulated twice...
Done
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43378?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: I661a1893a3f7c3659753c82b009398763a06a92d
Gerrit-Change-Number: 43378
Gerrit-PatchSet: 9
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>
Gerrit-Attention: osmith <osmith(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Fri, 28 Aug 2026 20:02:56 +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>
Attention is currently required from: osmith, pespin.
Hello Jenkins Builder, fixeria, osmith,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43378?usp=email
to look at the new patch set (#9).
The following approvals got outdated and were removed:
Verified+1 by Jenkins Builder
Change subject: smf: Implement PFCP Sess Est and Sess Del procedures
......................................................................
smf: Implement PFCP Sess Est and Sess Del procedures
With these in place, the following tests now pass:
TC_createSession_deleteSession
f_TC_deleteSession_unknown
TC_s2b_createSession_v4_noapco
TC_s2b_createSession_v4_apco
Change-Id: I661a1893a3f7c3659753c82b009398763a06a92d
---
M library/PFCP_Templates.ttcn
M smf/SMF_Session_CT.ttcn
M smf/SMF_Tests.ttcn
M smf/expected-results.xml
M smf/gen_links.sh
5 files changed, 395 insertions(+), 26 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/78/43378/9
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43378?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: newpatchset
Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: I661a1893a3f7c3659753c82b009398763a06a92d
Gerrit-Change-Number: 43378
Gerrit-PatchSet: 9
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>
Gerrit-Attention: osmith <osmith(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Hello Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43436?usp=email
to look at the new patch set (#7).
The following approvals got outdated and were removed:
Verified+1 by Jenkins Builder
Change subject: smf: Implement TC_gy_ccr_update_rejected
......................................................................
smf: Implement TC_gy_ccr_update_rejected
Add missing code to have a successful TC_gy_ccr_update_rejected.
Change-Id: I9ad9fb1520f8a2163ed90e8ca5c121edcb267006
---
M library/PFCP_Templates.ttcn
M smf/SMF_Session_CT.ttcn
M smf/SMF_Tests.ttcn
3 files changed, 489 insertions(+), 13 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/36/43436/7
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43436?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: newpatchset
Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: I9ad9fb1520f8a2163ed90e8ca5c121edcb267006
Gerrit-Change-Number: 43436
Gerrit-PatchSet: 7
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Attention is currently required from: osmith, pespin.
fixeria has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43378?usp=email )
Change subject: smf: Implement PFCP Sess Est and Sess Del procedures
......................................................................
Patch Set 8:
(1 comment)
File smf/SMF_Session_CT.ttcn:
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43378/comment/1da8d0f4_1839… :
PS7, Line 680: f_pfcp_sess_del_resp
> Done
No, I meant the function body is somehow tabulated twice...
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43378?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: I661a1893a3f7c3659753c82b009398763a06a92d
Gerrit-Change-Number: 43378
Gerrit-PatchSet: 8
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>
Gerrit-Attention: osmith <osmith(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Fri, 28 Aug 2026 16:18:58 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: pespin <pespin(a)sysmocom.de>
Comment-In-Reply-To: fixeria <vyanitskiy(a)sysmocom.de>
Attention is currently required from: fixeria, osmith.
Hello Jenkins Builder, fixeria, osmith,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43378?usp=email
to look at the new patch set (#8).
The following approvals got outdated and were removed:
Code-Review+1 by fixeria, Code-Review+1 by osmith, Verified+1 by Jenkins Builder
Change subject: smf: Implement PFCP Sess Est and Sess Del procedures
......................................................................
smf: Implement PFCP Sess Est and Sess Del procedures
With these in place, the following tests now pass:
TC_createSession_deleteSession
f_TC_deleteSession_unknown
TC_s2b_createSession_v4_noapco
TC_s2b_createSession_v4_apco
Change-Id: I661a1893a3f7c3659753c82b009398763a06a92d
---
M library/PFCP_Templates.ttcn
M smf/SMF_Session_CT.ttcn
M smf/SMF_Tests.ttcn
M smf/expected-results.xml
M smf/gen_links.sh
5 files changed, 395 insertions(+), 26 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/78/43378/8
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43378?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: newpatchset
Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: I661a1893a3f7c3659753c82b009398763a06a92d
Gerrit-Change-Number: 43378
Gerrit-PatchSet: 8
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>
Gerrit-Attention: osmith <osmith(a)sysmocom.de>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Hello Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43436?usp=email
to look at the new patch set (#5).
The following approvals got outdated and were removed:
Verified+1 by Jenkins Builder
Change subject: smf: Implement TC_gy_ccr_update_rejected
......................................................................
smf: Implement TC_gy_ccr_update_rejected
Add missing code to have a successful TC_gy_ccr_update_rejected.
Change-Id: I9ad9fb1520f8a2163ed90e8ca5c121edcb267006
---
M library/PFCP_Templates.ttcn
M smf/SMF_Session_CT.ttcn
M smf/SMF_Tests.ttcn
3 files changed, 489 insertions(+), 13 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/36/43436/5
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43436?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: newpatchset
Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: I9ad9fb1520f8a2163ed90e8ca5c121edcb267006
Gerrit-Change-Number: 43436
Gerrit-PatchSet: 5
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Attention is currently required from: lynxis lazus, osmith, pespin.
fixeria has posted comments on this change by fixeria. ( https://gerrit.osmocom.org/c/osmo-sgsn/+/42594?usp=email )
Change subject: gprs_gmm: gsm48_rx_gmm_att_req(): drop stale PDP contexts on re-Attach
......................................................................
Patch Set 5:
(1 comment)
Patchset:
PS4:
> Ack, makes sense. Marking as unresolved because it's not.
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43440 is a work in progress TTCN-3 testcase. It's currently failing due to Tguard timeout, because re-attach is somewhat broken. I am leaving it sitting WIP in Gerrit until this is resolved.
--
To view, visit https://gerrit.osmocom.org/c/osmo-sgsn/+/42594?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: osmo-sgsn
Gerrit-Branch: master
Gerrit-Change-Id: I20c1f5f741275115635188b2f4b1c5fe7c6e40f1
Gerrit-Change-Number: 42594
Gerrit-PatchSet: 5
Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-CC: lynxis lazus <lynxis(a)fe80.eu>
Gerrit-Attention: osmith <osmith(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: lynxis lazus <lynxis(a)fe80.eu>
Gerrit-Comment-Date: Fri, 28 Aug 2026 16:10:14 +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>
fixeria has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43440?usp=email )
Change subject: sgsn: add TC_attach_reattach_drops_stale_pdp
......................................................................
sgsn: add TC_attach_reattach_drops_stale_pdp
Verify that on a re-Attach with a known IMSI/P-TMSI, the SGSN drops
stale PDP contexts left over from before the MS restarted, but only
once the re-Attach has actually been authenticated.
Change-Id: I4a0b6cfbb46f673e3037c0fbf3e8c03a8b6316ad
Related: osmo-sgsn.git I20c1f5f741275115635188b2f4b1c5fe7c6e40f1
Related: OS#6922
---
M sgsn/SGSN_Tests.ttcn
M sgsn/expected-results.xml
2 files changed, 38 insertions(+), 0 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/40/43440/1
diff --git a/sgsn/SGSN_Tests.ttcn b/sgsn/SGSN_Tests.ttcn
index 1d58b46..78d2103 100644
--- a/sgsn/SGSN_Tests.ttcn
+++ b/sgsn/SGSN_Tests.ttcn
@@ -1267,6 +1267,42 @@
f_cleanup();
}
+/* ATTACH + PDP CTX ACT + re-ATTACH (with the P-TMSI allocated by the first
+ * Attach): the SGSN must recognize the known identity and release the stale
+ * PDP context left over from before towards the GGSN, but only once the
+ * re-Attach has actually been authenticated. */
+private function f_TC_attach_reattach_drops_stale_pdp(charstring id) runs on BSSGP_ConnHdlr {
+ var PdpActPars apars := valueof(t_PdpActPars(mp_ggsn_ip));
+ timer T := 5.0;
+
+ /* first perform regular attach + activate a PDP context */
+ f_TC_attach(id);
+ f_pdp_ctx_act(apars);
+
+ /* MS "restarts" and sends a re-Attach Request using the P-TMSI it was
+ * assigned before.
+ * XXX: the test is getting stuck here :/ */
+ f_gmm_attach(false, false, 0, omit, true);
+
+ /* the stale PDP context must have been released towards the GGSN */
+ T.start;
+ alt {
+ [] as_ggsn_gtp_ctx_del_req(apars) {
+ setverdict(pass);
+ }
+ [] T.timeout {
+ setverdict(fail, "SGSN did not release stale PDP context towards GGSN after re-Attach");
+ }
+ }
+}
+testcase TC_attach_reattach_drops_stale_pdp() runs on test_CT {
+ var BSSGP_ConnHdlr vc_conn;
+ f_init();
+ vc_conn := f_start_handler(refers(f_TC_attach_reattach_drops_stale_pdp), testcasename(), g_gb, 74);
+ vc_conn.done;
+ f_cleanup();
+}
+
private function f_TC_attach_echo_timeout(charstring id) runs on BSSGP_ConnHdlr {
var Gtp1cUnitdata g_ud;
var PdpActPars apars := valueof(t_PdpActPars(mp_ggsn_ip));
@@ -3423,6 +3459,7 @@
execute( TC_attach_pdp_act_user_deact_mt() );
execute( TC_attach_pdp_act_deact_dup() );
execute( TC_attach_second_attempt() );
+ execute( TC_attach_reattach_drops_stale_pdp() );
execute( TC_attach_echo_timeout() );
execute( TC_attach_restart_ctr_echo() );
execute( TC_attach_restart_ctr_create() );
diff --git a/sgsn/expected-results.xml b/sgsn/expected-results.xml
index e2e099b..06352cd 100644
--- a/sgsn/expected-results.xml
+++ b/sgsn/expected-results.xml
@@ -61,6 +61,7 @@
SGSN_Tests.ttcn:MASKED TC_attach_second_attempt testcase
</failure>
</testcase>
+ <testcase classname='SGSN_Tests' name='TC_attach_reattach_drops_stale_pdp' time='MASKED'/>
<testcase classname='SGSN_Tests' name='TC_attach_echo_timeout' time='MASKED'/>
<testcase classname='SGSN_Tests' name='TC_attach_restart_ctr_echo' time='MASKED'/>
<testcase classname='SGSN_Tests' name='TC_attach_restart_ctr_create' time='MASKED'/>
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43440?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: newchange
Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: I4a0b6cfbb46f673e3037c0fbf3e8c03a8b6316ad
Gerrit-Change-Number: 43440
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de>