Attention is currently required from: keith.
pespin has posted comments on this change by keith. ( https://gerrit.osmocom.org/c/osmo-msc/+/28342?usp=email )
Change subject: SMS UX optimisation: Deliver rx'd SMS right away
......................................................................
Patch Set 4:
(1 comment)
Patchset:
PS4:
I'm not really involved into the big picture/roadmap regarding SMS, but I'd argue it makes more sense to send SMS in order in the queue instead of reordering SMS (what would happen if we send the SMS directly upon receive).
Think for example 2 SMS sent by Alice to Bob where the 2nd SMS provides contradictory information wrt 1st SMS (eg. "I'm arriving on time" vs "I'm late", or eg. 2-factor authentication codes).
Without looking in detail, may it be that the delay @keith@rhizomatica.org is mentioning comes from the fact that when we run the queue we try to lookup for SMS for all subscribers and retrigger them? AFAIU the sane approach would be to lookup SMS for that specific subscriber, not sure if we are doing that already.
--
To view, visit https://gerrit.osmocom.org/c/osmo-msc/+/28342?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: osmo-msc
Gerrit-Branch: master
Gerrit-Change-Id: I9af51ef0d9c2e6c5acc5128efd6195df881b680c
Gerrit-Change-Number: 28342
Gerrit-PatchSet: 4
Gerrit-Owner: keith <keith(a)rhizomatica.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: laforge <laforge(a)osmocom.org>
Gerrit-CC: neels <nhofmeyr(a)sysmocom.de>
Gerrit-CC: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: keith <keith(a)rhizomatica.org>
Gerrit-Comment-Date: Tue, 04 Nov 2025 11:10:30 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
osmith has uploaded a new patch set (#2). ( https://gerrit.osmocom.org/c/libosmocore/+/41296?usp=email )
Change subject: osmo-release: debian_patch_apply: handle empty dir
......................................................................
osmo-release: debian_patch_apply: handle empty dir
Don't fail with the following bogus error message if the debian/patches
directory exists but is empty:
error: can't open patch '…/git/libosmocore/debian/patches/*.patch': No such file or directory
Fixes: OS#6873
Change-Id: I6656602c58fd221c8bb933890712d1392a1dfdab
---
M osmo-release.sh
1 file changed, 3 insertions(+), 3 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/libosmocore refs/changes/96/41296/2
--
To view, visit https://gerrit.osmocom.org/c/libosmocore/+/41296?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: newpatchset
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I6656602c58fd221c8bb933890712d1392a1dfdab
Gerrit-Change-Number: 41296
Gerrit-PatchSet: 2
Gerrit-Owner: osmith <osmith(a)sysmocom.de>
Gerrit-CC: Jenkins Builder
osmith has uploaded this change for review. ( https://gerrit.osmocom.org/c/libosmocore/+/41296?usp=email )
Change subject: osmo-release: debian_patch_apply: handle empty dir
......................................................................
osmo-release: debian_patch_apply: handle empty dir
Don't fail with the following bogus error message if the debian/patches
directory exists but is empty:
error: can't open patch '/crypt/space/home/laforge/projects/git/libosmocore/debian/patches/*.patch': No such file or directory
Fixes: OS#6873
Change-Id: I6656602c58fd221c8bb933890712d1392a1dfdab
---
M osmo-release.sh
1 file changed, 3 insertions(+), 3 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/libosmocore refs/changes/96/41296/1
diff --git a/osmo-release.sh b/osmo-release.sh
index 14790da..5a9584c 100755
--- a/osmo-release.sh
+++ b/osmo-release.sh
@@ -151,10 +151,10 @@
# Make sure that patches under debian/patches/ apply:
check_debian_patch_apply() {
- if [ ! -d "${GIT_TOPDIR}/debian/patches" ]; then
- return
- fi
for patch in ${GIT_TOPDIR}/debian/patches/*.patch; do
+ if ! [ -e "$patch" ]; then
+ continue
+ fi
git apply --check $patch
if [ $? -ne 0 ]; then
error "patch no longer applies! $patch"
--
To view, visit https://gerrit.osmocom.org/c/libosmocore/+/41296?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: newchange
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I6656602c58fd221c8bb933890712d1392a1dfdab
Gerrit-Change-Number: 41296
Gerrit-PatchSet: 1
Gerrit-Owner: osmith <osmith(a)sysmocom.de>
Attention is currently required from: fixeria, pespin.
lynxis lazus has uploaded a new patch set (#8) to the change originally created by pespin. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41136?usp=email )
The following approvals got outdated and were removed:
Verified+1 by Jenkins Builder
Change subject: stp: Introduce STP_Tests_TCAP
......................................................................
stp: Introduce STP_Tests_TCAP
The network topology and test infrastructure already allows testing TCAP forwarding
at STP over IPA<->IPA, M3UA<->IPA and M3UA<->M3UA.
Co-authored-by: Alexander Couzens <lynxis(a)fe80.eu>
Related: SYS#5423
Change-Id: I4340bdf126118c66af861f5a447e90883783920e
---
M stp/STP_Tests_Common.ttcn
A stp/STP_Tests_TCAP.cfg
A stp/STP_Tests_TCAP.ttcn
M stp/expected-results.xml
M stp/gen_links.sh
A stp/osmo-stp-tcap.confmerge
M stp/regen_makefile.sh
A stp/testenv_tcap.cfg
8 files changed, 1,087 insertions(+), 2 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/36/41136/8
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41136?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: I4340bdf126118c66af861f5a447e90883783920e
Gerrit-Change-Number: 41136
Gerrit-PatchSet: 8
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-CC: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-CC: lynxis lazus <lynxis(a)fe80.eu>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Attention is currently required from: fixeria, lynxis lazus.
lynxis lazus has uploaded a new patch set (#7) to the change originally created by pespin. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41136?usp=email )
The following approvals got outdated and were removed:
Verified+1 by Jenkins Builder
Change subject: stp: Introduce STP_Tests_TCAP
......................................................................
stp: Introduce STP_Tests_TCAP
The network topology and test infrastructure already allows testing TCAP forwarding
at STP over IPA<->IPA, M3UA<->IPA and M3UA<->M3UA.
Co-authored: Alexander Couzens <lynxis(a)fe80.eu>
Related: SYS#5423
Change-Id: I4340bdf126118c66af861f5a447e90883783920e
---
M stp/STP_Tests_Common.ttcn
A stp/STP_Tests_TCAP.cfg
A stp/STP_Tests_TCAP.ttcn
M stp/expected-results.xml
M stp/gen_links.sh
A stp/osmo-stp-tcap.confmerge
M stp/regen_makefile.sh
A stp/testenv_tcap.cfg
8 files changed, 1,081 insertions(+), 2 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/36/41136/7
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41136?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: I4340bdf126118c66af861f5a447e90883783920e
Gerrit-Change-Number: 41136
Gerrit-PatchSet: 7
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-CC: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-CC: lynxis lazus <lynxis(a)fe80.eu>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Attention: lynxis lazus <lynxis(a)fe80.eu>
Attention is currently required from: jolly.
laforge has posted comments on this change by laforge. ( https://gerrit.osmocom.org/c/libosmocore/+/41295?usp=email )
Change subject: gsm/gsm_utils: Support the ER-GSM band (ARFCN 940..954)
......................................................................
Patch Set 3: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/c/libosmocore/+/41295?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I2f17bca405403ee12cfe7c5c804c135ab021b369
Gerrit-Change-Number: 41295
Gerrit-PatchSet: 3
Gerrit-Owner: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: jolly <andreas(a)eversberg.eu>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>
Gerrit-Attention: jolly <andreas(a)eversberg.eu>
Gerrit-Comment-Date: Mon, 03 Nov 2025 13:16:16 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes