Attention is currently required from: falconia, fixeria, jolly, laforge.
pespin has posted comments on this change by falconia. ( https://gerrit.osmocom.org/c/libosmo-netif/+/39280?usp=email )
Change subject: bring twjit into libosmo-netif
......................................................................
Patch Set 6:
(1 comment)
File include/osmocom/netif/twjit.h:
https://gerrit.osmocom.org/c/libosmo-netif/+/39280/comment/bc99dac1_8bcd1e0… :
PS6, Line 50: struct osmo_twjit_config {
> @pespin@sysmocom. […]
Ok I see indeed. I am fine then with having a osmo_twjit_config_alloc() and osmo_twjit_config_free() API and keep the struct private (eg. src/twjit_private.h so it can be accessed by both twjit.c and twrtp.c if needed).
Then during osmo_twjit_config_init() copy/apply the config internally; caller is responsible for freeing it when needed.
--
To view, visit https://gerrit.osmocom.org/c/libosmo-netif/+/39280?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: libosmo-netif
Gerrit-Branch: master
Gerrit-Change-Id: Ia3be5834571ca18b68939abbcf1ce3a879156658
Gerrit-Change-Number: 39280
Gerrit-PatchSet: 6
Gerrit-Owner: falconia <falcon(a)freecalypso.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: jolly <andreas(a)eversberg.eu>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: jolly <andreas(a)eversberg.eu>
Gerrit-Attention: falconia <falcon(a)freecalypso.org>
Gerrit-Attention: laforge <laforge(a)osmocom.org>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Comment-Date: Wed, 27 Aug 2025 11:23:23 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: falconia <falcon(a)freecalypso.org>
Comment-In-Reply-To: pespin <pespin(a)sysmocom.de>
osmith has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40942?usp=email )
Change subject: testenv: init podman: add --distro arg
......................................................................
testenv: init podman: add --distro arg
Allow building the podman image for another distribution. This was
mostly implemented already, but due to the missing --distro argument for
"./testenv.py init podman" it could not be used yet.
Change-Id: Iff59b5d403e88046a3f98438394e1e02a36e7c0a
---
M _testenv/testenv/__init__.py
1 file changed, 6 insertions(+), 0 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/42/40942/1
diff --git a/_testenv/testenv/__init__.py b/_testenv/testenv/__init__.py
index 3de7f0c..49f6368 100644
--- a/_testenv/testenv/__init__.py
+++ b/_testenv/testenv/__init__.py
@@ -69,6 +69,12 @@
action="store_true",
help="build image even if it is up-to-date",
)
+ sub_podman.add_argument(
+ "-d",
+ "--distro",
+ default=distro_default,
+ help=f"distribution for podman (default: {distro_default})",
+ )
sub_run = sub.add_parser("run", help="build components and run a testsuite")
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40942?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: Iff59b5d403e88046a3f98438394e1e02a36e7c0a
Gerrit-Change-Number: 40942
Gerrit-PatchSet: 1
Gerrit-Owner: osmith <osmith(a)sysmocom.de>
osmith has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40946?usp=email )
Change subject: testenv: check value of --distro early
......................................................................
testenv: check value of --distro early
Make sure that the value for --distro is supported early on, instead of
only checking it if --binary-repo is set while trying to add the binary
repository.
Change-Id: I37493f26d71e70a19f4a19b3bbda972c03c84905
---
M _testenv/testenv/__init__.py
M _testenv/testenv/podman.py
2 files changed, 10 insertions(+), 7 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/46/40946/1
diff --git a/_testenv/testenv/__init__.py b/_testenv/testenv/__init__.py
index 3a92ff0..f0d6629 100644
--- a/_testenv/testenv/__init__.py
+++ b/_testenv/testenv/__init__.py
@@ -21,6 +21,12 @@
log_prefix = "[testenv]"
+# Mapping of supported distros (docker/podman image names) and repository dirs
+# from the binary repository
+distros_repodirs = {
+ "debian:bookworm": "Debian_12",
+}
+
def resolve_testsuite_name_alias(name):
mapping = {
@@ -252,6 +258,9 @@
if args.distro and not args.podman:
raise NoTraceException("--distro requires --podman")
+ if args.distro and args.distro not in distros_repodirs:
+ raise NoTraceException(f"--distro must be one of {', '.join(distros_repodirs)}")
+
if args.kernel == "debian" and not args.podman:
raise NoTraceException("--kernel-debian requires --podman")
diff --git a/_testenv/testenv/podman.py b/_testenv/testenv/podman.py
index 8ce6766..bc45b3a 100644
--- a/_testenv/testenv/podman.py
+++ b/_testenv/testenv/podman.py
@@ -300,18 +300,12 @@
sys.exit(1)
-def distro_to_repo_dir(distro):
- if distro == "debian:bookworm":
- return "Debian_12"
- raise RuntimeError(f"Can't translate distro {distro} to repo_dir!")
-
-
def enable_binary_repo():
config = "deb [signed-by=/obs.key]"
config += " https://downloads.osmocom.org/packages/"
config += testenv.args.binary_repo.replace(":", ":/")
config += "/"
- config += distro_to_repo_dir(distro)
+ config += testenv.distros_repodirs[distro]
config += "/ ./"
path = "/etc/apt/sources.list.d/osmocom.list"
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40946?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: I37493f26d71e70a19f4a19b3bbda972c03c84905
Gerrit-Change-Number: 40946
Gerrit-PatchSet: 1
Gerrit-Owner: osmith <osmith(a)sysmocom.de>
osmith has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40945?usp=email )
Change subject: testenv: check for --distro without --podman
......................................................................
testenv: check for --distro without --podman
The --distro option only makes sense with --podman, complain if it is
used without --podman.
Change-Id: I1b7b26320c65d498a1e812f85f0b98eda95be783
---
M _testenv/testenv/__init__.py
1 file changed, 12 insertions(+), 1 deletion(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/45/40945/1
diff --git a/_testenv/testenv/__init__.py b/_testenv/testenv/__init__.py
index 49f6368..3a92ff0 100644
--- a/_testenv/testenv/__init__.py
+++ b/_testenv/testenv/__init__.py
@@ -187,7 +187,7 @@
group.add_argument(
"-d",
"--distro",
- default=distro_default,
+ default=None, # override in set_args_defaults
help=f"distribution for podman (default: {distro_default})",
)
group.add_argument(
@@ -249,6 +249,9 @@
if args.binary_repo and not args.podman:
raise NoTraceException("--binary-repo requires --podman")
+ if args.distro and not args.podman:
+ raise NoTraceException("--distro requires --podman")
+
if args.kernel == "debian" and not args.podman:
raise NoTraceException("--kernel-debian requires --podman")
@@ -263,9 +266,17 @@
raise NoTraceException(f"testsuite dir not found: {testsuite_dir}")
+def set_args_defaults():
+ """Some defaults are set later, e.g. after verifying that --distro is not
+ being used without --podman."""
+ if args.distro is None:
+ args.distro = distro_default
+
+
def init_args():
parse_args()
verify_args_run()
+ set_args_defaults()
class ColorFormatter(logging.Formatter):
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40945?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: I1b7b26320c65d498a1e812f85f0b98eda95be783
Gerrit-Change-Number: 40945
Gerrit-PatchSet: 1
Gerrit-Owner: osmith <osmith(a)sysmocom.de>