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

[M] Change in osmo-mgw[master]: vty and log: also show local port for RTP conns
by keith 02 Nov '23

02 Nov '23

Attention is currently required from: dexter, neels, osmith. keith has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-mgw/+/34150?usp=email ) Change subject: vty and log: also show local port for RTP conns ...................................................................... Patch Set 3: Code-Review+1 (1 comment) Patchset: PS3: Thanks for dealing with this! Dropping the RTCP port is good, as is the C vs 0x part. -- To view, visit https://gerrit.osmocom.org/c/osmo-mgw/+/34150?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-mgw Gerrit-Branch: master Gerrit-Change-Id: Ib89a6779e1d68c6600f00699d4303f6c0ee07132 Gerrit-Change-Number: 34150 Gerrit-PatchSet: 3 Gerrit-Owner: keith <keith(a)rhizomatica.org> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: keith <keith(a)rhizomatica.org> Gerrit-Reviewer: neels <nhofmeyr(a)sysmocom.de> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: osmith <osmith(a)sysmocom.de> Gerrit-Attention: neels <nhofmeyr(a)sysmocom.de> Gerrit-Attention: dexter <pmaier(a)sysmocom.de> Gerrit-Comment-Date: Thu, 02 Nov 2023 18:42:53 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

1 0

[S] Change in osmo-ci[master]: repo-install-test: ignore limesuite-images
by pespin 02 Nov '23

02 Nov '23

Attention is currently required from: osmith. pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ci/+/34955?usp=email ) Change subject: repo-install-test: ignore limesuite-images ...................................................................... Patch Set 1: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/34955?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-ci Gerrit-Branch: master Gerrit-Change-Id: I237408c805977c831f352a57a301ea45753d1ac1 Gerrit-Change-Number: 34955 Gerrit-PatchSet: 1 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: osmith <osmith(a)sysmocom.de> Gerrit-Comment-Date: Thu, 02 Nov 2023 16:01:17 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

1 0

[S] Change in osmo-ci[master]: repo-install-test: fix ca-certificates-java error
by pespin 02 Nov '23

02 Nov '23

Attention is currently required from: osmith. pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ci/+/34954?usp=email ) Change subject: repo-install-test: fix ca-certificates-java error ...................................................................... Patch Set 1: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/34954?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-ci Gerrit-Branch: master Gerrit-Change-Id: I00b9c97d9d85fb37ba33a48caa732cd50de99683 Gerrit-Change-Number: 34954 Gerrit-PatchSet: 1 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: osmith <osmith(a)sysmocom.de> Gerrit-Comment-Date: Thu, 02 Nov 2023 16:00:54 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

1 0

[S] Change in osmo-ci[master]: Update git tag patterns for rtl-sdr, osmo-fl2k
by osmith 02 Nov '23

02 Nov '23

osmith has submitted this change. ( https://gerrit.osmocom.org/c/osmo-ci/+/34953?usp=email ) Change subject: Update git tag patterns for rtl-sdr, osmo-fl2k ...................................................................... Update git tag patterns for rtl-sdr, osmo-fl2k With the v2.0.0 release of rtl-sdr, it was decided to include the v in the tag pattern again, so adjust it in the related scripts. Adjust osmo-fl2k too in the OBS scripts because there the tag pattern was not set. Change-Id: If1bb64722bd24b0079d2427ea3b0f5b5c87ef7ac --- M scripts/common.sh M scripts/obs/lib/config.py 2 files changed, 17 insertions(+), 1 deletion(-) Approvals: osmith: Looks good to me, approved Jenkins Builder: Verified pespin: Looks good to me, but someone else must approve diff --git a/scripts/common.sh b/scripts/common.sh index 2bca384..ea106e2 100644 --- a/scripts/common.sh +++ b/scripts/common.sh @@ -80,7 +80,7 @@ local url ret pattern case "$project" in - gapk|osmo-fl2k) + gapk|osmo-fl2k|rtl-sdr) pattern='refs/tags/v[0-9.]*$' ;; *) diff --git a/scripts/obs/lib/config.py b/scripts/obs/lib/config.py index 33aced8..4779d77 100644 --- a/scripts/obs/lib/config.py +++ b/scripts/obs/lib/config.py @@ -111,6 +111,8 @@ git_latest_tag_pattern_other = { "limesuite": "^v[0-9]*\\.[0-9]*\\.[0-9]*$", "open5gs": "^v[0-9]*\\.[0-9]*\\.[0-9]*$", + "osmo-fl2k": "^v[0-9]*\\.[0-9]*\\.[0-9]*$", + "rtl-sdr": "^v[0-9]*\\.[0-9]*\\.[0-9]*$", "wireshark": "^v[0-9]*\\.[0-9]*\\.[0-9a-z]*$", } -- To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/34953?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-ci Gerrit-Branch: master Gerrit-Change-Id: If1bb64722bd24b0079d2427ea3b0f5b5c87ef7ac Gerrit-Change-Number: 34953 Gerrit-PatchSet: 1 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-MessageType: merged

1 0

[S] Change in osmo-ci[master]: Update git tag patterns for rtl-sdr, osmo-fl2k
by osmith 02 Nov '23

02 Nov '23

Attention is currently required from: fixeria. osmith has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ci/+/34953?usp=email ) Change subject: Update git tag patterns for rtl-sdr, osmo-fl2k ...................................................................... Patch Set 1: Code-Review+2 (1 comment) Patchset: PS1: merging this now, so the release script doesn't re-create releases for the tags that don't start with v -- To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/34953?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-ci Gerrit-Branch: master Gerrit-Change-Id: If1bb64722bd24b0079d2427ea3b0f5b5c87ef7ac Gerrit-Change-Number: 34953 Gerrit-PatchSet: 1 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-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Comment-Date: Thu, 02 Nov 2023 15:59:22 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

1 0

[S] Change in osmo-ci[master]: repo-install-test: fix ca-certificates-java error
by osmith 02 Nov '23

02 Nov '23

osmith has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-ci/+/34954?usp=email ) Change subject: repo-install-test: fix ca-certificates-java error ...................................................................... repo-install-test: fix ca-certificates-java error The repo-install-test started to fail on debian 10 with the following error. Apparently this happens when installing ca-certificates-java after installing java. Add a workaround. Setting up ca-certificates-java (20190405) ... head: cannot open '/etc/ssl/certs/java/cacerts' for reading: No such file or directory Exception in thread "main" java.lang.InternalError: Error loading java.security file Change-Id: I00b9c97d9d85fb37ba33a48caa732cd50de99683 --- M scripts/repo-install-test/run-inside.sh 1 file changed, 20 insertions(+), 1 deletion(-) git pull ssh://gerrit.osmocom.org:29418/osmo-ci refs/changes/54/34954/1 diff --git a/scripts/repo-install-test/run-inside.sh b/scripts/repo-install-test/run-inside.sh index b45d76d..e08338f 100755 --- a/scripts/repo-install-test/run-inside.sh +++ b/scripts/repo-install-test/run-inside.sh @@ -185,8 +185,10 @@ case "$DISTRO" in debian10) - # Can't access https://osmocom.org otherwise + # libgnutls30: can't access https://osmocom.org otherwise + # ca-certificates-java: fails if installed after java apt-get install -y --no-install-recommends \ + ca-certificates-java \ libgnutls30 ;; esac -- To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/34954?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-ci Gerrit-Branch: master Gerrit-Change-Id: I00b9c97d9d85fb37ba33a48caa732cd50de99683 Gerrit-Change-Number: 34954 Gerrit-PatchSet: 1 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-MessageType: newchange

1 0

[S] Change in osmo-ci[master]: repo-install-test: ignore limesuite-images
by osmith 02 Nov '23

02 Nov '23

osmith has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-ci/+/34955?usp=email ) Change subject: repo-install-test: ignore limesuite-images ...................................................................... repo-install-test: ignore limesuite-images Don't attempt to install limesuite-images. It runs a post-install script that downloads files from an external server and fails currently, as there are no images for 23.10. While we have limesuite-images packages in our OBS repository, this is just a side-effect of building limesuite. What we are really interested in is liblimesuite for osmo-trx, as I understand. Add --no-install-recommends to the apt-get install line in install_repo_packages_debian, because the main limesuite package has limesuite-images in recommends and would pull it in otherwise. Change-Id: I237408c805977c831f352a57a301ea45753d1ac1 --- M scripts/repo-install-test/blacklist_debian10.txt M scripts/repo-install-test/blacklist_debian11.txt M scripts/repo-install-test/blacklist_debian12.txt M scripts/repo-install-test/run-inside.sh 4 files changed, 29 insertions(+), 1 deletion(-) git pull ssh://gerrit.osmocom.org:29418/osmo-ci refs/changes/55/34955/1 diff --git a/scripts/repo-install-test/blacklist_debian10.txt b/scripts/repo-install-test/blacklist_debian10.txt index 6570de6..93dac75 100644 --- a/scripts/repo-install-test/blacklist_debian10.txt +++ b/scripts/repo-install-test/blacklist_debian10.txt @@ -19,6 +19,10 @@ osmocom-nitb osmocom-nitb-dbg +# Not relevant for the test +limesuite-images +limesuite-images23.10 + # SoapySDR is not used anymore (see OS#3542) soapysdr-module-lms7 soapysdr0.6-module-lms7 diff --git a/scripts/repo-install-test/blacklist_debian11.txt b/scripts/repo-install-test/blacklist_debian11.txt index 53187dc..c962ffc 100644 --- a/scripts/repo-install-test/blacklist_debian11.txt +++ b/scripts/repo-install-test/blacklist_debian11.txt @@ -31,6 +31,8 @@ libulfius2.7-dbgsym libyder2.0-dbgsym limesuite-dbgsym +limesuite-images +limesuite-images23.10 soapysdr0.7-module-lms7-dbgsym # Depends on specific verions 0.5.4.38.0847 of rtl-sdr, which we won't install diff --git a/scripts/repo-install-test/blacklist_debian12.txt b/scripts/repo-install-test/blacklist_debian12.txt index b470d44..9b5f5e7 100644 --- a/scripts/repo-install-test/blacklist_debian12.txt +++ b/scripts/repo-install-test/blacklist_debian12.txt @@ -32,6 +32,8 @@ libulfius2.7-dbgsym libyder2.0-dbgsym limesuite-dbgsym +limesuite-images +limesuite-images23.10 soapysdr0.7-module-lms7 soapysdr0.7-module-lms7-dbgsym soapysdr-module-lms7 diff --git a/scripts/repo-install-test/run-inside.sh b/scripts/repo-install-test/run-inside.sh index e08338f..d7730ff 100755 --- a/scripts/repo-install-test/run-inside.sh +++ b/scripts/repo-install-test/run-inside.sh @@ -329,7 +329,7 @@ cat osmocom_packages_all.txt filter_packages_txt - apt install -y $(cat osmocom_packages.txt) + apt-get install -y --no-install-recommends $(cat osmocom_packages.txt) } install_repo_packages_centos() { -- To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/34955?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-ci Gerrit-Branch: master Gerrit-Change-Id: I237408c805977c831f352a57a301ea45753d1ac1 Gerrit-Change-Number: 34955 Gerrit-PatchSet: 1 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-MessageType: newchange

1 0

[S] Change in osmo-ci[master]: Update git tag patterns for rtl-sdr, osmo-fl2k
by pespin 02 Nov '23

02 Nov '23

Attention is currently required from: fixeria, osmith. pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ci/+/34953?usp=email ) Change subject: Update git tag patterns for rtl-sdr, osmo-fl2k ...................................................................... Patch Set 1: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/34953?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-ci Gerrit-Branch: master Gerrit-Change-Id: If1bb64722bd24b0079d2427ea3b0f5b5c87ef7ac Gerrit-Change-Number: 34953 Gerrit-PatchSet: 1 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-Attention: osmith <osmith(a)sysmocom.de> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Comment-Date: Thu, 02 Nov 2023 14:14:07 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

1 0

[S] Change in osmo-ci[master]: Update git tag patterns for rtl-sdr, osmo-fl2k
by osmith 02 Nov '23

02 Nov '23

osmith has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-ci/+/34953?usp=email ) Change subject: Update git tag patterns for rtl-sdr, osmo-fl2k ...................................................................... Update git tag patterns for rtl-sdr, osmo-fl2k With the v2.0.0 release of rtl-sdr, it was decided to include the v in the tag pattern again, so adjust it in the related scripts. Adjust osmo-fl2k too in the OBS scripts because there the tag pattern was not set. Change-Id: If1bb64722bd24b0079d2427ea3b0f5b5c87ef7ac --- M scripts/common.sh M scripts/obs/lib/config.py 2 files changed, 17 insertions(+), 1 deletion(-) git pull ssh://gerrit.osmocom.org:29418/osmo-ci refs/changes/53/34953/1 diff --git a/scripts/common.sh b/scripts/common.sh index 2bca384..ea106e2 100644 --- a/scripts/common.sh +++ b/scripts/common.sh @@ -80,7 +80,7 @@ local url ret pattern case "$project" in - gapk|osmo-fl2k) + gapk|osmo-fl2k|rtl-sdr) pattern='refs/tags/v[0-9.]*$' ;; *) diff --git a/scripts/obs/lib/config.py b/scripts/obs/lib/config.py index 33aced8..4779d77 100644 --- a/scripts/obs/lib/config.py +++ b/scripts/obs/lib/config.py @@ -111,6 +111,8 @@ git_latest_tag_pattern_other = { "limesuite": "^v[0-9]*\\.[0-9]*\\.[0-9]*$", "open5gs": "^v[0-9]*\\.[0-9]*\\.[0-9]*$", + "osmo-fl2k": "^v[0-9]*\\.[0-9]*\\.[0-9]*$", + "rtl-sdr": "^v[0-9]*\\.[0-9]*\\.[0-9]*$", "wireshark": "^v[0-9]*\\.[0-9]*\\.[0-9a-z]*$", } -- To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/34953?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-ci Gerrit-Branch: master Gerrit-Change-Id: If1bb64722bd24b0079d2427ea3b0f5b5c87ef7ac Gerrit-Change-Number: 34953 Gerrit-PatchSet: 1 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-MessageType: newchange

1 0

[S] Change in osmo-ttcn3-hacks[master]: hnbgw: Introduce mp_hnbgw_timer_x31
by osmith 02 Nov '23

02 Nov '23

Attention is currently required from: laforge, neels, pespin. osmith has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/34951?usp=email ) Change subject: hnbgw: Introduce mp_hnbgw_timer_x31 ...................................................................... Patch Set 1: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/34951?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-ttcn3-hacks Gerrit-Branch: master Gerrit-Change-Id: I176ef96e193f2ca39077bcee3a2187768ddb45ce Gerrit-Change-Number: 34951 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder 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-Comment-Date: Thu, 02 Nov 2023 11:57:12 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

1 0