January 2026 - gerrit-log - lists.osmocom.org

[S] Change in osmo-ttcn3-hacks[master]: testenv: less cluttered output for failed cmds
by osmith 20 Jan '26

20 Jan '26

osmith has posted comments on this change by osmith. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41868?usp=email ) Change subject: testenv: less cluttered output for failed cmds ...................................................................... Patch Set 2: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41868?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: I508322b017e5ed9c21396384ef759a4c49c8d3da Gerrit-Change-Number: 41868 Gerrit-PatchSet: 2 Gerrit-Owner: osmith <osmith(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-Comment-Date: Tue, 20 Jan 2026 10:17:58 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

[S] Change in osmo-ttcn3-hacks[master]: bts: testenv: check ulimit -r in prepare=
by osmith 20 Jan '26

20 Jan '26

osmith has submitted this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41862?usp=email ) Change subject: bts: testenv: check ulimit -r in prepare= ...................................................................... bts: testenv: check ulimit -r in prepare= Don't respawning over and over again if osmo-bts-trx fails to start up because the user isn't allowed to set the rtpriority. Show instructions for fixing it instead. Change-Id: I7bc1414f95e4a54a40eb7c6f7b3d6c83ce3500c6 --- M _testenv/README.md A _testenv/data/scripts/require_ulimit_r.sh M _testenv/testenv/podman.py M bts/testenv_generic.cfg M bts/testenv_hopping.cfg M bts/testenv_oml.cfg 6 files changed, 31 insertions(+), 1 deletion(-) Approvals: Jenkins Builder: Verified fixeria: Looks good to me, approved pespin: Looks good to me, but someone else must approve diff --git a/_testenv/README.md b/_testenv/README.md index b62295e..a973b25 100644 --- a/_testenv/README.md +++ b/_testenv/README.md @@ -272,6 +272,11 @@ * `TERM`: Is set to the same `TERM` passed to testenv with fallback to `dumb`. +* `HOST_USER`: + When running in podman (`--podman`), this variable is set to the user that is + running podman. This is used in `_testenv/data/scripts/require_ulimit_r.sh` + to display a hint for setting the rtprio limit for this user. + ## Eclipse-titan versions It is possible to change the eclipse-titan version that is used to build and diff --git a/_testenv/data/scripts/require_ulimit_r.sh b/_testenv/data/scripts/require_ulimit_r.sh new file mode 100755 index 0000000..eb4e0e2 --- /dev/null +++ b/_testenv/data/scripts/require_ulimit_r.sh @@ -0,0 +1,21 @@ +#!/bin/sh +MIN=$1 + +if [ -z "$MIN" ]; then + echo "usage: require_ulimit_r.sh MIN" + exit 1 +fi + +if [ -z "$USER" ]; then + USER="$HOST_USER" +fi + +if [ $(ulimit -r) -lt $MIN ]; then + echo + echo "====================================================" + echo "Allow your user to set rtprio, logout and try again:" + echo "$ echo '$USER - rtprio $MIN' | sudo tee '/etc/security/limits.d/$USER-allow-rtprio.conf'" + echo "====================================================" + echo + exit 1 +fi diff --git a/_testenv/testenv/podman.py b/_testenv/testenv/podman.py index 2013699..79fc5b9 100644 --- a/_testenv/testenv/podman.py +++ b/_testenv/testenv/podman.py @@ -244,6 +244,8 @@ "net.ipv4.conf.default.send_redirects=0", # OS#6575 "-e", "PODMAN=1", + "-e", + f"HOST_USER={os.getenv('USER')}", ] if testenv.args.binary_repo: diff --git a/bts/testenv_generic.cfg b/bts/testenv_generic.cfg index e04f89c..3b4a7f1 100644 --- a/bts/testenv_generic.cfg +++ b/bts/testenv_generic.cfg @@ -20,6 +20,7 @@ package=osmocom-bb-trxcon [bts] +prepare=require_ulimit_r.sh 30 program=respawn.sh osmo-bts-trx make=osmo-bts package=osmo-bts diff --git a/bts/testenv_hopping.cfg b/bts/testenv_hopping.cfg index 9426457..3e1e1c9 100644 --- a/bts/testenv_hopping.cfg +++ b/bts/testenv_hopping.cfg @@ -23,6 +23,7 @@ package=osmocom-bb-trxcon [bts] +prepare=require_ulimit_r.sh 30 program=respawn.sh osmo-bts-trx make=osmo-bts package=osmo-bts diff --git a/bts/testenv_oml.cfg b/bts/testenv_oml.cfg index b1268c6..1921536 100644 --- a/bts/testenv_oml.cfg +++ b/bts/testenv_oml.cfg @@ -14,7 +14,7 @@ package=osmocom-bb-trxcon [bts] -prepare=osmo-config-merge osmo-bts.cfg osmo-bts-oml.confmerge > osmo-bts-oml.cfg +prepare=osmo-config-merge osmo-bts.cfg osmo-bts-oml.confmerge > osmo-bts-oml.cfg && require_ulimit_r.sh 30 program=osmo-bts-trx -c osmo-bts-oml.cfg make=osmo-bts package=osmo-bts-trx libosmocore-utils -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41862?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: osmo-ttcn3-hacks Gerrit-Branch: master Gerrit-Change-Id: I7bc1414f95e4a54a40eb7c6f7b3d6c83ce3500c6 Gerrit-Change-Number: 41862 Gerrit-PatchSet: 4 Gerrit-Owner: osmith <osmith(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: laforge <laforge(a)osmocom.org>

1 0

[S] Change in osmo-ttcn3-hacks[master]: bts: testenv: check ulimit -r in prepare=
by fixeria 20 Jan '26

20 Jan '26

Attention is currently required from: laforge, osmith. fixeria has posted comments on this change by osmith. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41862?usp=email ) Change subject: bts: testenv: check ulimit -r in prepare= ...................................................................... Patch Set 3: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41862?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: I7bc1414f95e4a54a40eb7c6f7b3d6c83ce3500c6 Gerrit-Change-Number: 41862 Gerrit-PatchSet: 3 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-CC: laforge <laforge(a)osmocom.org> Gerrit-Attention: osmith <osmith(a)sysmocom.de> Gerrit-Attention: laforge <laforge(a)osmocom.org> Gerrit-Comment-Date: Tue, 20 Jan 2026 09:27:31 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

[S] Change in pysim[master]: pySim.apdu_source.stdin_hex
by laforge 20 Jan '26

20 Jan '26

Attention is currently required from: dexter, fixeria. laforge has posted comments on this change by laforge. ( https://gerrit.osmocom.org/c/pysim/+/37925?usp=email ) Change subject: pySim.apdu_source.stdin_hex ...................................................................... Patch Set 1: (1 comment) File pySim/apdu_source/stdin_hex.py: https://gerrit.osmocom.org/c/pysim/+/37925/comment/f14f8b40_762de7ce?usp=em… : PS1, Line 41: raise StopIteration > How do we get here? IIUC, only an exception can break out the loop, like a `KeyboardInterrupt`. […] yeah, likely a copy+paste result. I'll revisit it. -- To view, visit https://gerrit.osmocom.org/c/pysim/+/37925?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: I5aacf13b7c27cea9efd42f01dacca61068c3aa33 Gerrit-Change-Number: 37925 Gerrit-PatchSet: 1 Gerrit-Owner: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Attention: dexter <pmaier(a)sysmocom.de> Gerrit-Comment-Date: Tue, 20 Jan 2026 09:25:17 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: fixeria <vyanitskiy(a)sysmocom.de>

1 0

[S] Change in pysim[master]: Print SMSC in pySim-read.py
by laforge 20 Jan '26

20 Jan '26

Attention is currently required from: dexter, fixeria, phcoder. laforge has posted comments on this change by phcoder. ( https://gerrit.osmocom.org/c/pysim/+/41786?usp=email ) Change subject: Print SMSC in pySim-read.py ...................................................................... Patch Set 15: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/c/pysim/+/41786?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: I17067b68086316d51fd71ba77049874605594e3f Gerrit-Change-Number: 41786 Gerrit-PatchSet: 15 Gerrit-Owner: phcoder <phcoder(a)gmail.com> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-CC: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Attention: phcoder <phcoder(a)gmail.com> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Attention: dexter <pmaier(a)sysmocom.de> Gerrit-Comment-Date: Tue, 20 Jan 2026 09:23:44 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

[S] Change in pysim[master]: contrib: add utility to receive ES2+handleDownloadProgressInfo calls
by laforge 20 Jan '26

20 Jan '26

Attention is currently required from: dexter. laforge has posted comments on this change by dexter. ( https://gerrit.osmocom.org/c/pysim/+/41875?usp=email ) Change subject: contrib: add utility to receive ES2+handleDownloadProgressInfo calls ...................................................................... Patch Set 1: Code-Review-1 (1 comment) File contrib/es2p_server.py: https://gerrit.osmocom.org/c/pysim/+/41875/comment/a0a318b2_b4d65e45?usp=em… : PS1, Line 36: logging.info("ES2+:handleDownloadProgressInfo: %s" % str(data)) this is not yet including the simaResponse decode, which is the most important part, IMHO. -- To view, visit https://gerrit.osmocom.org/c/pysim/+/41875?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: I95af30cebae31f7dc682617b1866f4a2dc9b760c Gerrit-Change-Number: 41875 Gerrit-PatchSet: 1 Gerrit-Owner: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Attention: dexter <pmaier(a)sysmocom.de> Gerrit-Comment-Date: Tue, 20 Jan 2026 09:22:57 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes

1 0

[L] Change in pysim[master]: esim/http_json_api: extend JSON API with server functionality
by laforge 20 Jan '26

20 Jan '26

Attention is currently required from: dexter. laforge has posted comments on this change by dexter. ( https://gerrit.osmocom.org/c/pysim/+/41874?usp=email ) Change subject: esim/http_json_api: extend JSON API with server functionality ...................................................................... Patch Set 1: Code-Review+1 (3 comments) Commit Message: https://gerrit.osmocom.org/c/pysim/+/41874/comment/a2130702_24bbd46b?usp=em… : PS1, Line 17: tread treat https://gerrit.osmocom.org/c/pysim/+/41874/comment/4d19d320_0a8193a1?usp=em… : PS1, Line 27: the tread "to treat the header"? Not sure what you mean File pySim/esim/es2p.py: https://gerrit.osmocom.org/c/pysim/+/41874/comment/14a1601f_2704f52e?usp=em… : PS1, Line 252: The API user is expected to override wouldn't it be better to have an abc.ABC here? that way, the code woul fail to start if anyone ever did not override all the methods. But I guess here you actually want to permit users who do not override all the methods? -- To view, visit https://gerrit.osmocom.org/c/pysim/+/41874?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: I277aa90fddb5171c4bf6c3436259aa371d30d092 Gerrit-Change-Number: 41874 Gerrit-PatchSet: 1 Gerrit-Owner: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Attention: dexter <pmaier(a)sysmocom.de> Gerrit-Comment-Date: Tue, 20 Jan 2026 09:22:01 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes

1 0

[XS] Change in pysim[master]: esim/http_json_api: add missing check
by laforge 20 Jan '26

20 Jan '26

Attention is currently required from: dexter. laforge has posted comments on this change by dexter. ( https://gerrit.osmocom.org/c/pysim/+/41873?usp=email ) Change subject: esim/http_json_api: add missing check ...................................................................... Patch Set 1: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/c/pysim/+/41873?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: I8a1a3cd5e029dba4a3aec1a64702e19b0d694ae2 Gerrit-Change-Number: 41873 Gerrit-PatchSet: 1 Gerrit-Owner: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Attention: dexter <pmaier(a)sysmocom.de> Gerrit-Comment-Date: Tue, 20 Jan 2026 08:51:38 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

[S] Change in libosmo-pfcp[master]: Avoid marking rx PFCP Assoc Setup Req as duplicate
by osmith 20 Jan '26

20 Jan '26

Attention is currently required from: daniel, fixeria, laforge, neels, pespin. osmith has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/libosmo-pfcp/+/41876?usp=email ) Change subject: Avoid marking rx PFCP Assoc Setup Req as duplicate ...................................................................... Patch Set 1: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/c/libosmo-pfcp/+/41876?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: libosmo-pfcp Gerrit-Branch: master Gerrit-Change-Id: Ia461550e6791aaf00d18e0310bb4f17fdd2a3f65 Gerrit-Change-Number: 41876 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: daniel <dwillmann(a)sysmocom.de> Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: neels <nhofmeyr(a)sysmocom.de> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Attention: neels <nhofmeyr(a)sysmocom.de> Gerrit-Attention: laforge <laforge(a)osmocom.org> Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Attention: daniel <dwillmann(a)sysmocom.de> Gerrit-Comment-Date: Tue, 20 Jan 2026 08:29:59 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

[XS] Change in libosmocore[master]: contrib/jenkins_arm.sh: Remove explicit configure.sh disable args
by osmith 20 Jan '26

20 Jan '26

Attention is currently required from: fixeria, laforge, pespin. osmith has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/libosmocore/+/41871?usp=email ) Change subject: contrib/jenkins_arm.sh: Remove explicit configure.sh disable args ...................................................................... Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/libosmocore/+/41871?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: Ie1f99305c198a4e164d54ce7090c9fc67e99a891 Gerrit-Change-Number: 41871 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Attention: laforge <laforge(a)osmocom.org> Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Comment-Date: Tue, 20 Jan 2026 08:18:42 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

2026

2025

2024

2023

2022

gerrit-log January 2026