Attention is currently required from: fixeria, jolly, laforge, pespin.
Hello Jenkins Builder, fixeria, laforge, pespin,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/osmo-msc/+/41504?usp=email
to look at the new patch set (#2).
The following approvals got outdated and were removed:
Code-Review+1 by fixeria, Code-Review+1 by pespin, Verified+1 by Jenkins Builder
Change subject: ASCI: Fix establishment of calls when ciphering is enabled
......................................................................
ASCI: Fix establishment of calls when ciphering is enabled
We do not support any VGCS/VBS ciphering. If the CM Service Request
indicates VGCS or VBS service, encryption is not enabled for the channel
to be set up. There will be no Ciphering Command sent to the ME.
Authentication is still performed with the initiator of the call, if
authentication is enabled. Only authenticated subscribers will then be
able to initiate ASCI calls.
Change-Id: Ic59221e6b0f64c9f19508ae31cc3672548d0666e
---
M src/libvlr/vlr_access_req_fsm.c
1 file changed, 6 insertions(+), 2 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-msc refs/changes/04/41504/2
--
To view, visit https://gerrit.osmocom.org/c/osmo-msc/+/41504?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: newpatchset
Gerrit-Project: osmo-msc
Gerrit-Branch: master
Gerrit-Change-Id: Ic59221e6b0f64c9f19508ae31cc3672548d0666e
Gerrit-Change-Number: 41504
Gerrit-PatchSet: 2
Gerrit-Owner: jolly <andreas(a)eversberg.eu>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: jolly <andreas(a)eversberg.eu>
Gerrit-Attention: laforge <laforge(a)osmocom.org>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Attention is currently required from: fixeria.
osmith has posted comments on this change by fixeria. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41517?usp=email )
Change subject: s1gw: f_ConnHdlr_ue_ctx_release_req(): add a quirk for -latest
......................................................................
Patch Set 1: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41517?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: I90c44955ad3aabfe7ca9198b3ec9b6b97a46c0ae
Gerrit-Change-Number: 41517
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Comment-Date: Thu, 27 Nov 2025 07:04:44 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Attention is currently required from: fixeria.
pespin has posted comments on this change by fixeria. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41517?usp=email )
Change subject: s1gw: f_ConnHdlr_ue_ctx_release_req(): add a quirk for -latest
......................................................................
Patch Set 1: Code-Review+1
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41517?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: I90c44955ad3aabfe7ca9198b3ec9b6b97a46c0ae
Gerrit-Change-Number: 41517
Gerrit-PatchSet: 1
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: Wed, 26 Nov 2025 18:51:46 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
fixeria has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41517?usp=email )
Change subject: s1gw: f_ConnHdlr_ue_ctx_release_req(): add a quirk for -latest
......................................................................
s1gw: f_ConnHdlr_ue_ctx_release_req(): add a quirk for -latest
We're hitting a crash in Eclipse TITAN when running
S1GW_Tests.TC_uemux_ue_ctx_release_req against the -latest version
of osmo-s1gw, which is sending PFCP Session Deletion Request that
we no longer expect to receive. This results in an empty junit-xml
file due to early termination of the runtime.
Re-introduce the f_ConnHdlr_session_delete(), only for the -latest.
Change-Id: I90c44955ad3aabfe7ca9198b3ec9b6b97a46c0ae
Related: 8e315ed6 ("s1gw: correct handling of UE CONTEXT RELEASE REQUEST")
Related: osmo-s1gw.git If467162e871b7989f0578be30c04d1c32c263c99
Related: OS#6888, SYS#7738, SYS#7599
---
M s1gw/S1GW_ConnHdlr.ttcn
1 file changed, 4 insertions(+), 0 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/17/41517/1
diff --git a/s1gw/S1GW_ConnHdlr.ttcn b/s1gw/S1GW_ConnHdlr.ttcn
index b850f55..ec6d08a 100644
--- a/s1gw/S1GW_ConnHdlr.ttcn
+++ b/s1gw/S1GW_ConnHdlr.ttcn
@@ -1662,6 +1662,10 @@
log("[eNB -> S1GW] -> MME: UE CONTEXT RELEASE REQUEST");
f_ConnHdlr_tx_s1ap_from_enb(ts_S1AP_UeContextReleaseReq(mme_ue_id, enb_ue_id, cause));
+ /* XXX: remove this quirk when osmo-s1gw > 0.3.0 is released */
+ if (Misc_Helpers.f_osmo_repo_is("latest")) {
+ f_ConnHdlr_session_delete(erabs);
+ }
log("eNB -> [S1GW -> MME]: UE CONTEXT RELEASE REQUEST");
f_ConnHdlr_rx_s1ap_from_enb(pdu, tr_S1AP_UeContextReleaseReq(mme_ue_id, enb_ue_id, cause));
}
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41517?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: I90c44955ad3aabfe7ca9198b3ec9b6b97a46c0ae
Gerrit-Change-Number: 41517
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de>
Attention is currently required from: dexter.
fixeria has posted comments on this change by dexter. ( https://gerrit.osmocom.org/c/pysim/+/41508?usp=email )
Change subject: card_key_provider: add PostgreSQL support
......................................................................
Patch Set 4:
(5 comments)
File contrib/csv-to-pgsql.py:
https://gerrit.osmocom.org/c/pysim/+/41508/comment/bca5d7b1_fe5f280e?usp=em… :
PS4, Line 16: # cmd2 >= 2.3.0 has deprecated the bg/fg in favor of Bg/Fg :(
Other than `YELLOW`, `cmd2` API is not used in this script. I would avoid importing it just because you want a color in logging. I believe you can just use an ASCII color code?
https://gerrit.osmocom.org/c/pysim/+/41508/comment/8cd99406_fee94e2b?usp=em… :
PS4, Line 225: log.info("CSV file: %s" % opts.csv)
BTW, for logging API it's recommended to use lazy format-sting evaluation:
```
log.info("CSV file: %s", opts.csv)
```
https://gerrit.osmocom.org/c/pysim/+/41508/comment/ee7844fb_d607dda4?usp=em… :
PS4, Line 227: if not csv_file:
Does `open()` return `None` at all? It raises an exception on error.
File pySim/card_key_provider.py:
https://gerrit.osmocom.org/c/pysim/+/41508/comment/3cf45c38_64837df4?usp=em… :
PS4, Line 242: break;
cosmetic: semicolon is not needed here
File setup.py:
https://gerrit.osmocom.org/c/pysim/+/41508/comment/1548a453_b40f2048?usp=em… :
PS4, Line 37: psycopg2-binary
Not blocking here, but do we really want this as a required dependency? This list is already quite long, giving headaches to those trying to package pySim...
--
To view, visit https://gerrit.osmocom.org/c/pysim/+/41508?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: Icba625c02a60d7e1f519b506a46bda5ded0537d3
Gerrit-Change-Number: 41508
Gerrit-PatchSet: 4
Gerrit-Owner: dexter <pmaier(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: daniel <dwillmann(a)sysmocom.de>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-CC: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Attention: dexter <pmaier(a)sysmocom.de>
Gerrit-Comment-Date: Wed, 26 Nov 2025 17:49:41 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Attention is currently required from: pespin.
daniel has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41516?usp=email )
Change subject: stp: tcap: Fix s/utds/udts/g typo
......................................................................
Patch Set 1: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41516?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: Ie30f665ecff2cefc1acbfd1aa65d9bebdd9311ec
Gerrit-Change-Number: 41516
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: daniel <dwillmann(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Wed, 26 Nov 2025 13:02:24 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes