November 2024 - gerrit-log - lists.osmocom.org

[S] Change in osmo-bts[master]: csd_v110: clarify field names in csd_v110_lchan_desc[]

by pespin

Attention is currently required from: fixeria. pespin has posted comments on this change by fixeria. ( https://gerrit.osmocom.org/c/osmo-bts/+/38816?usp=email ) Change subject: csd_v110: clarify field names in csd_v110_lchan_desc[] ...................................................................... Patch Set 1: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/c/osmo-bts/+/38816?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-bts Gerrit-Branch: master Gerrit-Change-Id: If97787a64e30ff9b39c26749ba32aed09d3d7983 Gerrit-Change-Number: 38816 Gerrit-PatchSet: 1 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: falconia <falcon(a)freecalypso.org> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Comment-Date: Tue, 19 Nov 2024 09:17:51 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

7 months, 1 week

1
0
0 0

[S] Change in osmo-bts[master]: csd_v110: add CSD_V110_NUM_BITS macro

by pespin

Attention is currently required from: fixeria. pespin has posted comments on this change by fixeria. ( https://gerrit.osmocom.org/c/osmo-bts/+/38815?usp=email ) Change subject: csd_v110: add CSD_V110_NUM_BITS macro ...................................................................... Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/osmo-bts/+/38815?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-bts Gerrit-Branch: master Gerrit-Change-Id: Iabd2ee5ea9d580e0a6f5c93d681f6be67d1949c5 Gerrit-Change-Number: 38815 Gerrit-PatchSet: 1 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: falconia <falcon(a)freecalypso.org> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Comment-Date: Tue, 19 Nov 2024 09:16:01 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

7 months, 1 week

1
0
0 0

[XS] Change in osmo-ttcn3-hacks[master]: testenv: make it pass 'ruff check' linter

by osmith

osmith has submitted this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/38778?usp=email ) ( 2 is the latest approved patch-set. No files were changed between the latest approved patch-set and the submitted one. )Change subject: testenv: make it pass 'ruff check' linter ...................................................................... testenv: make it pass 'ruff check' linter I've added 'ruff check' to my pre-commit script. Make it pass initially, so it can detect future bugs. The missing f-string is a bug that causes ggsn testsuites with a custom kernel path to not work. Change-Id: I3e5cd05ce0aa241509d77391c5ed64e618f385ba --- M _testenv/testenv/podman.py M _testenv/testenv/testsuite.py 2 files changed, 2 insertions(+), 2 deletions(-) Approvals: pespin: Looks good to me, but someone else must approve fixeria: Looks good to me, approved laforge: Looks good to me, but someone else must approve Jenkins Builder: Verified diff --git a/_testenv/testenv/podman.py b/_testenv/testenv/podman.py index 16db400..1281b3b 100644 --- a/_testenv/testenv/podman.py +++ b/_testenv/testenv/podman.py @@ -254,7 +254,7 @@ cmd += ["--volume", "/dev/kvm:/dev/kvm"] if os.path.islink(testenv.custom_kernel_path): dest = os.readlink(testenv.custom_kernel_path) - cmd += ["--volume", "{dest}:{dest}:ro"] + cmd += ["--volume", f"{dest}:{dest}:ro"] cmd += [ "--volume", diff --git a/_testenv/testenv/testsuite.py b/_testenv/testenv/testsuite.py index 0fa42ab..9b28668 100644 --- a/_testenv/testenv/testsuite.py +++ b/_testenv/testenv/testsuite.py @@ -225,7 +225,7 @@ try: with open(path, "r") as h: return h.readline().rstrip() - except: + except: # noqa # File may not exist, e.g. if test was stopped return None -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/38778?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: I3e5cd05ce0aa241509d77391c5ed64e618f385ba Gerrit-Change-Number: 38778 Gerrit-PatchSet: 3 Gerrit-Owner: osmith <osmith(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-Reviewer: pespin <pespin(a)sysmocom.de>

7 months, 1 week

1
0
0 0

[M] Change in osmo-ttcn3-hacks[master]: testenv: add run --until-nok

by osmith

osmith has submitted this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/38776?usp=email ) ( 1 is the latest approved patch-set. No files were changed between the latest approved patch-set and the submitted one. )Change subject: testenv: add run --until-nok ...................................................................... testenv: add run --until-nok Add an argument to run a specific test (if using --test) or a whole testsuite until it fails with "failure" or "error". This helped me in reproducing a race condition in the mgw testsuite (related issue). Related: OS#3849 Change-Id: I17e1ebcc5d6ff1b6a087c4d4c9405a02798212f1 --- M _testenv/testenv.py M _testenv/testenv/__init__.py M _testenv/testenv/testdir.py M _testenv/testenv/testenv_cfg.py M _testenv/testenv/testsuite.py 5 files changed, 76 insertions(+), 23 deletions(-) Approvals: pespin: Looks good to me, but someone else must approve Jenkins Builder: Verified fixeria: Looks good to me, approved laforge: Looks good to me, but someone else must approve diff --git a/_testenv/testenv.py b/_testenv/testenv.py index 81b3aac..b9de4e8 100755 --- a/_testenv/testenv.py +++ b/_testenv/testenv.py @@ -16,6 +16,21 @@ import testenv.testsuite +def loop_continue_cond(loop_count): + if loop_count == 0: + return True + + if testenv.args.until_nok: + logging.info("Checking testsuite logs for failures and errors") + for match_str in ["failures='0'", "errors='0'"]: + if not testenv.testsuite.check_junit_logs_have(loop_count - 1, match_str): + logging.critical("Stopping the loop") + return False + return True + else: + return False + + def run(): testenv.testenv_cfg.init() @@ -45,26 +60,34 @@ testenv.osmo_dev.make(cfg) # Run the components + testsuite - cfg_count = 0 - for cfg_name, cfg in testenv.testenv_cfg.cfgs.items(): - # Restart podman container before running with another config - if testenv.args.podman and cfg_count: + loop_count = 0 + while loop_continue_cond(loop_count): + # Restart podman container before running again + if testenv.args.podman and loop_count: testenv.podman.stop(True) - testenv.testenv_cfg.set_current(cfg_name) + cfg_count = 0 + for cfg_name, cfg in testenv.testenv_cfg.cfgs.items(): + # Restart podman container before running with another config + if testenv.args.podman and cfg_count: + testenv.podman.stop(True) - if testenv.args.binary_repo: - testenv.podman.enable_binary_repo() - testenv.podman_install.packages(cfg, cfg_name) + testenv.testenv_cfg.set_current(cfg_name, loop_count) - testenv.testdir.prepare(cfg_name, cfg) - testenv.daemons.start(cfg) - testenv.testsuite.run(cfg) - testenv.daemons.stop() - testenv.testdir.clean_run_scripts("finished") + if testenv.args.binary_repo: + testenv.podman.enable_binary_repo() + testenv.podman_install.packages(cfg, cfg_name) - cfg_count += 1 - testenv.set_log_prefix("[testenv]") + testenv.testdir.prepare(cfg_name, cfg, loop_count) + testenv.daemons.start(cfg) + testenv.testsuite.run(cfg) + testenv.daemons.stop() + testenv.testdir.clean_run_scripts("finished") + + cfg_count += 1 + testenv.set_log_prefix("[testenv]") + + loop_count += 1 # Show test results testenv.testsuite.cat_junit_logs() diff --git a/_testenv/testenv/__init__.py b/_testenv/testenv/__init__.py index eb687f3..c1991ca 100644 --- a/_testenv/testenv/__init__.py +++ b/_testenv/testenv/__init__.py @@ -93,6 +93,14 @@ help="use binary packages from this Osmocom OBS project instead (e.g. osmocom:nightly)", ) + group = sub_run.add_argument_group("loop options", "Run the testsuite / a single test multiple times.") + group.add_argument( + "-u", + "--until-nok", + action="store_true", + help="run until there was either a failure or error", + ) + group = sub_run.add_argument_group( "QEMU options", "For some tests, the SUT can or must run in QEMU, typically to use kernel GTP-U.", diff --git a/_testenv/testenv/testdir.py b/_testenv/testenv/testdir.py index 88a2dca..b18efd1 100644 --- a/_testenv/testenv/testdir.py +++ b/_testenv/testenv/testdir.py @@ -51,14 +51,19 @@ testenv.cmd.run(["ln", "-sf", testdir_topdir, "/tmp/logs"], no_podman=True) -def prepare(cfg_name, cfg): +def prepare(cfg_name, cfg, loop_count): global testdir global clean_scripts + topdir = testdir_topdir + + if testenv.args.until_nok: + topdir = os.path.join(topdir, f"loop-{loop_count}") + if len(testenv.testenv_cfg.cfgs) == 1: - testdir = testdir_topdir + testdir = topdir else: - testdir = os.path.join(testdir_topdir, cfg_name.replace("testenv_", "").replace(".cfg", "")) + testdir = os.path.join(topdir, cfg_name.replace("testenv_", "").replace(".cfg", "")) logging.info(f"Preparing testdir: {testdir}") testsuite_dir = os.path.join(testenv.testsuite.ttcn3_hacks_dir, testenv.args.testsuite) diff --git a/_testenv/testenv/testenv_cfg.py b/_testenv/testenv/testenv_cfg.py index fefaf5c..4044225 100644 --- a/_testenv/testenv/testenv_cfg.py +++ b/_testenv/testenv/testenv_cfg.py @@ -13,16 +13,20 @@ current = None -def set_current(cfg_name): +def set_current(cfg_name, loop_count=0): global current current = cfg_name + loop_str = "" + + if testenv.args.until_nok: + loop_str = f"[loop-{loop_count}]" if cfg_name == "testenv.cfg": - testenv.set_log_prefix("[testenv]") + testenv.set_log_prefix(f"[testenv]{loop_str}") else: cfg_name = cfg_name.replace("testenv_", "") cfg_name = cfg_name.replace(".cfg", "") - testenv.set_log_prefix(f"[testenv][{cfg_name}]") + testenv.set_log_prefix(f"[testenv]{loop_str}[{cfg_name}]") def exit_error_readme(): diff --git a/_testenv/testenv/testsuite.py b/_testenv/testenv/testsuite.py index 1878b5c..0fa42ab 100644 --- a/_testenv/testenv/testsuite.py +++ b/_testenv/testenv/testsuite.py @@ -138,6 +138,11 @@ testenv.cmd.run(cmd, cwd=cwd) +def get_junit_logs(topdir): + pattern = os.path.join(topdir, "**", "junit-*.log") + return glob.glob(pattern, recursive=True) + + def cat_junit_logs(): tool = "cat" @@ -145,13 +150,21 @@ colors = os.environ.get("TESTENV_SOURCE_HIGHLIGHT_COLORS", "esc256") tool = f"source-highlight -f {shlex.quote(colors)} -s xml -i" - pattern = os.path.join(testenv.testdir.testdir_topdir, "**", "junit-*.log") - for path in glob.glob(pattern, recursive=True): + for path in get_junit_logs(testenv.testdir.testdir_topdir): cmd = f"echo && {tool} {shlex.quote(path)} && echo" logging.info(f"Showing {os.path.relpath(path, testenv.testdir.testdir_topdir)}") testenv.cmd.run(cmd) +def check_junit_logs_have(loop_count, match_str): + topdir = os.path.join(testenv.testdir.testdir_topdir, f"loop-{loop_count}") + for path in get_junit_logs(topdir): + cmd = ["grep", "-q", match_str, path] + if testenv.cmd.run(cmd, check=False).returncode: + return False + return True + + def run(cfg): global testsuite_proc -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/38776?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: I17e1ebcc5d6ff1b6a087c4d4c9405a02798212f1 Gerrit-Change-Number: 38776 Gerrit-PatchSet: 2 Gerrit-Owner: osmith <osmith(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: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>

7 months, 1 week

1
0
0 0

[S] Change in osmo-ttcn3-hacks[master]: mgw: fix RTP packets received while RX disabled

by osmith

osmith has submitted this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/38777?usp=email ) Change subject: mgw: fix RTP packets received while RX disabled ...................................................................... mgw: fix RTP packets received while RX disabled Fix the race condition causing tests to abort with: 1 RTP packets received while RX was disabled Fixes: OS#3849 Change-Id: Ieaf5440a693448cdeeaac7f47e6110cb480bff5f --- M mgw/MGCP_Test.ttcn 1 file changed, 27 insertions(+), 14 deletions(-) Approvals: osmith: Looks good to me, approved Jenkins Builder: Verified diff --git a/mgw/MGCP_Test.ttcn b/mgw/MGCP_Test.ttcn index d341022..84e5943 100644 --- a/mgw/MGCP_Test.ttcn +++ b/mgw/MGCP_Test.ttcn @@ -656,13 +656,14 @@ } /* Delete an existing RTP flow */ - function f_flow_delete(RTPEM_CTRL_PT pt, template MgcpEndpoint ep := omit, template MgcpCallId call_id := omit) + function f_flow_delete(RTPEM_CTRL_PT pt, template MgcpEndpoint ep := omit, template MgcpCallId call_id := omit, + RtpemMode mode := RTPEM_MODE_NONE) runs on dummy_CT { var template MgcpCommand cmd; var MgcpResponse resp; /* Switch off RTP flow */ - f_rtpem_mode(pt, RTPEM_MODE_NONE); + f_rtpem_mode(pt, mode); /* Delete connection on MGW (if needed) */ if (isvalue(call_id) and isvalue(ep)) { @@ -1672,8 +1673,9 @@ f_osmuxem_stats_err_check(stats_osmux); /* Tear down */ - f_flow_delete(RTPEM[0]); + f_flow_delete(RTPEM[0], mode := RTPEM_MODE_RXONLY); f_flow_delete_osmux(OsmuxEM, ep, call_id); + f_rtpem_mode(RTPEM[0], RTPEM_MODE_NONE); /* OS#3849 */ setverdict(pass); } @@ -2009,8 +2011,9 @@ f_sleep(1.0); - f_flow_delete(RTPEM[1]); + f_flow_delete(RTPEM[1], mode := RTPEM_MODE_RXONLY); f_flow_delete(RTPEM[0], ep, call_id); + f_rtpem_mode(RTPEM[1], RTPEM_MODE_NONE); /* OS#3849 */ stats[0] := f_rtpem_stats_get(RTPEM[0]); stats[1] := f_rtpem_stats_get(RTPEM[1]); @@ -2146,8 +2149,9 @@ f_rtpem_stats_err_check(stats[1]); /* Tear down */ - f_flow_delete(RTPEM[0]); + f_flow_delete(RTPEM[0], mode := RTPEM_MODE_RXONLY); f_flow_delete(RTPEM[1], ep, call_id); + f_rtpem_mode(RTPEM[0], RTPEM_MODE_NONE); /* OS#3849 */ setverdict(pass); } @@ -2210,8 +2214,9 @@ /* Stop transmitting packets and tear down the flows */ f_rtpem_mode(RTPEM[2], RTPEM_MODE_NONE); - f_flow_delete(RTPEM[0]); + f_flow_delete(RTPEM[0], mode := RTPEM_MODE_RXONLY); f_flow_delete(RTPEM[1], ep, call_id); + f_rtpem_mode(RTPEM[0], RTPEM_MODE_NONE); /* OS#3849 */ stats[0] := f_rtpem_stats_get(RTPEM[0]); stats[1] := f_rtpem_stats_get(RTPEM[1]); @@ -2273,8 +2278,9 @@ f_sleep(1.0); /* Terminate call */ - f_flow_delete(RTPEM[0]); + f_flow_delete(RTPEM[0], mode := RTPEM_MODE_RXONLY); f_flow_delete(RTPEM[1], ep, call_id); + f_rtpem_mode(RTPEM[0], RTPEM_MODE_NONE); /* OS#3849 */ stats[0] := f_rtpem_stats_get(RTPEM[0]); stats[1] := f_rtpem_stats_get(RTPEM[1]); @@ -2317,8 +2323,9 @@ f_rtpem_mode(RTPEM[0], RTPEM_MODE_BIDIR); f_rtpem_mode(RTPEM[1], RTPEM_MODE_BIDIR); f_sleep(1.0); - f_flow_delete(RTPEM[0]); + f_flow_delete(RTPEM[0], mode := RTPEM_MODE_RXONLY); f_flow_delete(RTPEM[1], ep, call_id); + f_rtpem_mode(RTPEM[0], RTPEM_MODE_NONE); /* OS#3849 */ stats[0] := f_rtpem_stats_get(RTPEM[0]); stats[1] := f_rtpem_stats_get(RTPEM[1]); @@ -2404,8 +2411,9 @@ f_rtpem_mode(RTPEM[0], RTPEM_MODE_NONE); /* Remove RTP flows and check statistics */ - f_flow_delete(RTPEM[0]); + f_flow_delete(RTPEM[0], mode := RTPEM_MODE_RXONLY); f_flow_delete(RTPEM[1], ep, call_id); + f_rtpem_mode(RTPEM[0], RTPEM_MODE_NONE); /* OS#3849 */ /* Check for errors */ stats[0] := f_rtpem_stats_get(RTPEM[0]); @@ -2496,8 +2504,9 @@ f_rtpem_mode(RTPEM[0], RTPEM_MODE_NONE); /* Remove RTP flows and check statistics */ - f_flow_delete(RTPEM[0]); + f_flow_delete(RTPEM[0], mode := RTPEM_MODE_RXONLY); f_flow_delete(RTPEM[1], ep, call_id); + f_rtpem_mode(RTPEM[0], RTPEM_MODE_NONE); /* OS#3849 */ /* Check for errors */ stats[0] := f_rtpem_stats_get(RTPEM[0]); @@ -2958,8 +2967,9 @@ /* Tear down */ if (tear_down_rtp) { - f_flow_delete(RTPEM[0]); + f_flow_delete(RTPEM[0], mode := RTPEM_MODE_RXONLY); f_flow_delete(RTPEM[1], ep, call_id); + f_rtpem_mode(RTPEM[0], RTPEM_MODE_NONE); /* OS#3849 */ } setverdict(pass); } @@ -3077,8 +3087,9 @@ } /* Tear down */ - f_flow_delete(RTPEM[0]); + f_flow_delete(RTPEM[0], mode := RTPEM_MODE_RXONLY); f_flow_delete(RTPEM[1], ep, call_id); + f_rtpem_mode(RTPEM[0], RTPEM_MODE_NONE); /* OS#3849 */ setverdict(pass); } testcase TC_two_crcx_mdcx_and_iuup_mdcx_recvonly() runs on dummy_CT { @@ -3215,8 +3226,9 @@ } /* Tear down */ - f_flow_delete(RTPEM[0]); + f_flow_delete(RTPEM[0], mode := RTPEM_MODE_RXONLY); f_flow_delete(RTPEM[1], ep, call_id); + f_rtpem_mode(RTPEM[0], RTPEM_MODE_NONE); /* OS#3849 */ setverdict(pass); } testcase TC_two_crcx_mdcx_and_iuup_rtp_mdcx_recvonly() runs on dummy_CT { @@ -3317,8 +3329,9 @@ } /* Tear down */ - f_flow_delete(RTPEM[0]); + f_flow_delete(RTPEM[0], mode := RTPEM_MODE_RXONLY); f_flow_delete(RTPEM[1], ep, call_id); + f_rtpem_mode(RTPEM[0], RTPEM_MODE_NONE); /* OS#3849 */ setverdict(pass); } testcase TC_two_crcx_mdcx_and_iuup_rtp_mdcx_to_iuup() runs on dummy_CT { -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/38777?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: Ieaf5440a693448cdeeaac7f47e6110cb480bff5f Gerrit-Change-Number: 38777 Gerrit-PatchSet: 2 Gerrit-Owner: osmith <osmith(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-Reviewer: pespin <pespin(a)sysmocom.de>

7 months, 1 week

1
0
0 0

[M] Change in osmo-ttcn3-hacks[master]: testenv: install dbg pkgs with --binary-repo too

by osmith

osmith has submitted this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/38803?usp=email ) ( 2 is the latest approved patch-set. No files were changed between the latest approved patch-set and the submitted one. )Change subject: testenv: install dbg pkgs with --binary-repo too ...................................................................... testenv: install dbg pkgs with --binary-repo too When using --binary-repo, figure out the -dbg and -dbgsym packages for all dependencies of packages to be installed, and install them as well. This will make debug symbols available in jenkins, useful for the related issue. Before this patch debug symbols were only available when building locally without --binary-repo. Related: OS#6630 Change-Id: I4dc57257b944126fed2c7b031c92c77a3d4286e0 --- M _testenv/data/podman/Dockerfile M _testenv/testenv/podman_install.py 2 files changed, 58 insertions(+), 0 deletions(-) Approvals: fixeria: Looks good to me, approved Jenkins Builder: Verified pespin: Looks good to me, but someone else must approve laforge: Looks good to me, but someone else must approve diff --git a/_testenv/data/podman/Dockerfile b/_testenv/data/podman/Dockerfile index 6f6c739..5321bfe 100644 --- a/_testenv/data/podman/Dockerfile +++ b/_testenv/data/podman/Dockerfile @@ -19,6 +19,7 @@ -y \ --no-install-recommends \ -o Dpkg::Options::="--force-confold" \ + apt-rdepends \ autoconf \ automake \ bc \ diff --git a/_testenv/testenv/podman_install.py b/_testenv/testenv/podman_install.py index 9c680b9..3c38285 100644 --- a/_testenv/testenv/podman_install.py +++ b/_testenv/testenv/podman_install.py @@ -3,6 +3,9 @@ import logging import multiprocessing import os +import shlex +import string +import subprocess import sys import testenv.cmd import testenv.podman @@ -30,6 +33,58 @@ os.makedirs(git_dir, exist_ok=True) +def get_dbg_pkgs(dep): + ret = [f"{dep}-dbg", f"{dep}-dbgsym"] + + # Get from e.g. libosmocore22 to libosmocore-dbg + dep_nodigits = dep.rstrip(string.digits) + if dep_nodigits != dep: + ret += [f"{dep_nodigits}-dbg", f"{dep_nodigits}-dbgsym"] + + return ret + + +def apt_get_dbg_pkgs(pkgs): + dbg_pkgs_all = os.path.join(testenv.args.cache, "podman", "dbg_pkgs_all") + dbg_pkgs = {} + + testenv.cmd.run(f"apt-cache pkgnames | grep -- -dbg > {shlex.quote(dbg_pkgs_all)}") + + for pkg in pkgs: + # Iterate over apt-rdepends, example output: + # osmo-mgw + # Depends: libc6 (>= 2.34) + # Depends: libosmoabis13 + rdeps = testenv.cmd.run(["apt-rdepends", pkg], stdout=subprocess.PIPE) + for line in rdeps.stdout.decode("utf-8").split("\n"): + if line.startswith(" "): + continue + dep = line.rstrip().split(" ", 1)[0] + + if dep not in dbg_pkgs: + for dbg_pkg in get_dbg_pkgs(dep): + # Use subprocess.run so we don't get lots of log messages. + # Also we don't need to run grep through podman. + grep = subprocess.run(["grep", "-q", f"^{dbg_pkg}$", dbg_pkgs_all]) + + if grep.returncode == 0: + dbg_pkgs[dep] = dbg_pkg + break + + if dep not in dbg_pkgs: + dbg_pkgs[dep] = None + + if dbg_pkgs[dep]: + logging.debug(f"{pkg} -> {dep}: installing {dbg_pkgs[dep]}") + + ret = [] + for dep, dbg in dbg_pkgs.items(): + if dbg: + ret += [dbg] + + return ret + + def apt_install(pkgs): if not pkgs: return @@ -37,6 +92,8 @@ # Remove duplicates pkgs = list(set(pkgs)) + pkgs += apt_get_dbg_pkgs(pkgs) + logging.info(f"Installing packages: {', '.join(pkgs)}") testenv.cmd.run(["apt-get", "-q", "install", "-y", "--no-install-recommends"] + pkgs) -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/38803?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: I4dc57257b944126fed2c7b031c92c77a3d4286e0 Gerrit-Change-Number: 38803 Gerrit-PatchSet: 3 Gerrit-Owner: osmith <osmith(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-Reviewer: pespin <pespin(a)sysmocom.de>

7 months, 1 week

1
0
0 0

[S] Change in osmo-ttcn3-hacks[master]: mgw: fix RTP packets received while RX disabled

by osmith

Attention is currently required from: fixeria, laforge, pespin. osmith has posted comments on this change by osmith. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/38777?usp=email ) Change subject: mgw: fix RTP packets received while RX disabled ...................................................................... Patch Set 2: Code-Review+2 (2 comments) Patchset: PS2: addressed the cosmetic issue, re-applying +2 File mgw/MGCP_Test.ttcn: https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/38777/comment/e515b922_540f… : PS1, Line 660: RtpemMode mode := RTPEM_MODE_NONE > Please align it with the opening brace. Done -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/38777?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: Ieaf5440a693448cdeeaac7f47e6110cb480bff5f Gerrit-Change-Number: 38777 Gerrit-PatchSet: 2 Gerrit-Owner: osmith <osmith(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-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: laforge <laforge(a)osmocom.org> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Comment-Date: Tue, 19 Nov 2024 07:33:48 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes Comment-In-Reply-To: fixeria <vyanitskiy(a)sysmocom.de>

7 months, 1 week

1
0
0 0

[S] Change in osmo-ttcn3-hacks[master]: mgw: fix RTP packets received while RX disabled

by osmith

Attention is currently required from: fixeria, laforge, osmith, pespin. Hello Jenkins Builder, fixeria, laforge, pespin, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/38777?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 laforge, Code-Review+1 by pespin, Verified+1 by Jenkins Builder Change subject: mgw: fix RTP packets received while RX disabled ...................................................................... mgw: fix RTP packets received while RX disabled Fix the race condition causing tests to abort with: 1 RTP packets received while RX was disabled Fixes: OS#3849 Change-Id: Ieaf5440a693448cdeeaac7f47e6110cb480bff5f --- M mgw/MGCP_Test.ttcn 1 file changed, 27 insertions(+), 14 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/77/38777/2 -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/38777?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: Ieaf5440a693448cdeeaac7f47e6110cb480bff5f Gerrit-Change-Number: 38777 Gerrit-PatchSet: 2 Gerrit-Owner: osmith <osmith(a)sysmocom.de> 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: osmith <osmith(a)sysmocom.de> Gerrit-Attention: laforge <laforge(a)osmocom.org> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Attention: pespin <pespin(a)sysmocom.de>

7 months, 1 week

1
0
0 0

[XS] Change in osmo-ttcn3-hacks[master]: testenv: make it pass 'ruff check' linter

by osmith

Attention is currently required from: osmith. Hello Jenkins Builder, fixeria, laforge, pespin, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/38778?usp=email to look at the new patch set (#3). The following approvals got outdated and were removed: Verified+1 by Jenkins Builder The change is no longer submittable: Verified is unsatisfied now. Change subject: testenv: make it pass 'ruff check' linter ...................................................................... testenv: make it pass 'ruff check' linter I've added 'ruff check' to my pre-commit script. Make it pass initially, so it can detect future bugs. The missing f-string is a bug that causes ggsn testsuites with a custom kernel path to not work. Change-Id: I3e5cd05ce0aa241509d77391c5ed64e618f385ba --- M _testenv/testenv/podman.py M _testenv/testenv/testsuite.py 2 files changed, 2 insertions(+), 2 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/78/38778/3 -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/38778?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: I3e5cd05ce0aa241509d77391c5ed64e618f385ba Gerrit-Change-Number: 38778 Gerrit-PatchSet: 3 Gerrit-Owner: osmith <osmith(a)sysmocom.de> 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: osmith <osmith(a)sysmocom.de>

7 months, 1 week

1
0
0 0

[M] Change in osmo-ttcn3-hacks[master]: testenv: add run --until-nok

by osmith

Attention is currently required from: daniel, osmith. Hello Jenkins Builder, daniel, fixeria, laforge, pespin, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/38776?usp=email to look at the new patch set (#2). The following approvals got outdated and were removed: Verified+1 by Jenkins Builder The change is no longer submittable: Verified is unsatisfied now. Change subject: testenv: add run --until-nok ...................................................................... testenv: add run --until-nok Add an argument to run a specific test (if using --test) or a whole testsuite until it fails with "failure" or "error". This helped me in reproducing a race condition in the mgw testsuite (related issue). Related: OS#3849 Change-Id: I17e1ebcc5d6ff1b6a087c4d4c9405a02798212f1 --- M _testenv/testenv.py M _testenv/testenv/__init__.py M _testenv/testenv/testdir.py M _testenv/testenv/testenv_cfg.py M _testenv/testenv/testsuite.py 5 files changed, 76 insertions(+), 23 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/76/38776/2 -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/38776?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: I17e1ebcc5d6ff1b6a087c4d4c9405a02798212f1 Gerrit-Change-Number: 38776 Gerrit-PatchSet: 2 Gerrit-Owner: osmith <osmith(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: pespin <pespin(a)sysmocom.de> Gerrit-Attention: osmith <osmith(a)sysmocom.de> Gerrit-Attention: daniel <dwillmann(a)sysmocom.de>

7 months, 1 week

1
0
0 0

2025

2024

2023

2022

gerrit-log November 2024