osmith has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43795?usp=email )
Change subject: testenv: fix running configs with omitted package=
......................................................................
testenv: fix running configs with omitted package=
The ns and sccp testsuites have test components in their testenv.cfg
without a package= line (osmo-ns-dummy and sccp_demo_user). These are
built from source against the -dev packages of the currently used binary
repository with testenv.podman_install.from_source().
Fix regressions from recent patches that broke this in two places:
1. testenv.daemons.start() always tried to access "packages" without
checking if it exists in the section first.
2. testenv.osmo_dev.get_targets() was treating a missing packages= line
the same as packages=no (which means building with osmo-dev instead
of testenv.podman_install.from_source()). Add the logic to not add
the make target to ret if there is no package= line. While at it,
rewrite the long if conditions to make the function easier to
understand.
Fixes: cf009ae7 ("testenv: run osmo-dev logic for --binary-repo too")
Fixes: 6cbec87b ("testenv: build package=no components with osmo-dev")
Change-Id: Id5920aacba832a1f43d87d09a9e133d95384e635
---
M _testenv/testenv/daemons.py
M _testenv/testenv/osmo_dev.py
2 files changed, 18 insertions(+), 4 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/95/43795/1
diff --git a/_testenv/testenv/daemons.py b/_testenv/testenv/daemons.py
index 0c402e9..e6085b6 100644
--- a/_testenv/testenv/daemons.py
+++ b/_testenv/testenv/daemons.py
@@ -60,7 +60,7 @@
env["LIBOSMO_IO_BACKEND"] = "IO_URING"
envtype = "osmo_dev"
- if testenv.args.binary_repo and section_data["package"] != "no":
+ if testenv.args.binary_repo and section_data.get("package") != "no":
envtype = "binary_repo"
if testenv.args.podman:
diff --git a/_testenv/testenv/osmo_dev.py b/_testenv/testenv/osmo_dev.py
index edbe1b2..d2f9229 100644
--- a/_testenv/testenv/osmo_dev.py
+++ b/_testenv/testenv/osmo_dev.py
@@ -58,10 +58,24 @@
# Gets built with testenv.testsuite.build()
continue
- if "make" in section_data and section_data["make"] != "no" and section_data["make"] not in ret:
- if testenv.args.binary_repo and section_data.get("package", "no") != "no":
+ if "make" not in section_data or section_data["make"] == "no":
+ # No osmo-dev make target listed in testenv.cfg
+ continue
+
+ if section_data["make"] in ret:
+ # Already decided that this target will be built while parsing a
+ # previous section
+ continue
+
+ if testenv.args.binary_repo:
+ if "package" not in section_data:
+ # Gets built with testenv.podman_install.from_source()
continue
- ret += [section_data["make"]]
+ if section_data["package"] != "no":
+ # Gets installed from a debian package
+ continue
+
+ ret += [section_data["make"]]
return ret
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43795?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: Id5920aacba832a1f43d87d09a9e133d95384e635
Gerrit-Change-Number: 43795
Gerrit-PatchSet: 1
Gerrit-Owner: osmith <osmith(a)sysmocom.de>
Attention is currently required from: pespin.
osmith has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43792?usp=email )
Change subject: 5gc: Introduce test TC_handover_inter_ngran_n2_dataforwardingpossible_change
......................................................................
Patch Set 2: Code-Review+1
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43792?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: I51731e9fb771666d39d7ee53055b44bdec636cf6
Gerrit-Change-Number: 43792
Gerrit-PatchSet: 2
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Wed, 07 Oct 2026 06:28:28 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
fixeria has submitted this change. ( https://gerrit.osmocom.org/c/osmo-trx/+/43784?usp=email )
Change subject: libosmo-trx: osmo_trxc_setformat_cb(): document all params
......................................................................
libosmo-trx: osmo_trxc_setformat_cb(): document all params
Change-Id: Ic6efb19e3f4543f43b1b8612bbbfb73632f2b572
---
M libosmo-trx/include/osmocom/trx/trxc_client.h
1 file changed, 3 insertions(+), 1 deletion(-)
Approvals:
pespin: Looks good to me, but someone else must approve
fixeria: Looks good to me, approved
Jenkins Builder: Verified
diff --git a/libosmo-trx/include/osmocom/trx/trxc_client.h b/libosmo-trx/include/osmocom/trx/trxc_client.h
index 9e5de24..09373c5 100644
--- a/libosmo-trx/include/osmocom/trx/trxc_client.h
+++ b/libosmo-trx/include/osmocom/trx/trxc_client.h
@@ -79,7 +79,9 @@
int osmo_trxc_client_rx(struct osmo_trxc_client *client, const char *buf, size_t len);
/*! TRXD PDU version negotiation result call-back.
- * \param[in] ver_use the negotiated version to be used */
+ * \param[in] client TRXC client instance
+ * \param[in] ver_use the negotiated version to be used
+ * \param[in] cb_data data that was passed to osmo_trxc_client_negotiate_format() */
typedef void osmo_trxc_setformat_cb(struct osmo_trxc_client *client,
uint8_t ver_use, void *cb_data);
int osmo_trxc_client_negotiate_format(struct osmo_trxc_client *client,
--
To view, visit https://gerrit.osmocom.org/c/osmo-trx/+/43784?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: merged
Gerrit-Project: osmo-trx
Gerrit-Branch: master
Gerrit-Change-Id: Ic6efb19e3f4543f43b1b8612bbbfb73632f2b572
Gerrit-Change-Number: 43784
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Attention is currently required from: jolly.
lynxis lazus has posted comments on this change by jolly. ( https://gerrit.osmocom.org/c/osmo-remsim/+/43746?usp=email )
Change subject: Add optional timeout to maintenance mapping
......................................................................
Patch Set 2:
(1 comment)
Patchset:
PS2:
Couldn't this be a generic timeout for both mappings (normal and maintenance)?
--
To view, visit https://gerrit.osmocom.org/c/osmo-remsim/+/43746?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: osmo-remsim
Gerrit-Branch: master
Gerrit-Change-Id: I2f7ee24e6c926b65876af2b14da3534555398769
Gerrit-Change-Number: 43746
Gerrit-PatchSet: 2
Gerrit-Owner: jolly <andreas(a)eversberg.eu>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: lynxis lazus <lynxis(a)fe80.eu>
Gerrit-Attention: jolly <andreas(a)eversberg.eu>
Gerrit-Comment-Date: Tue, 06 Oct 2026 15:03:34 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No