- gerrit-log - lists.osmocom.org

[XS] Change in osmo-ci[master]: ansible: build5-deb12build is also a coverity builder
by osmith 15 Jan '25

15 Jan '25

osmith has submitted this change. ( https://gerrit.osmocom.org/c/osmo-ci/+/39333?usp=email ) Change subject: ansible: build5-deb12build is also a coverity builder ...................................................................... ansible: build5-deb12build is also a coverity builder Change-Id: Id9d35658197409bcaba3e267cab36d7aa2a28972 --- M ansible/hosts 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: pespin: Looks good to me, but someone else must approve osmith: Looks good to me, approved Jenkins Builder: Verified diff --git a/ansible/hosts b/ansible/hosts index d3c59d6..57966d3 100644 --- a/ansible/hosts +++ b/ansible/hosts @@ -4,6 +4,7 @@ [coverity_slaves] build4-deb12build-ansible ansible_host=2a01:4f8:121:200a::1:3 +build5-deb12build-ansible ansible_host=2a01:4f8:140:926a::1:3 [jenkins_slaves] build4-deb12build-ansible ansible_host=2a01:4f8:121:200a::1:3 osmocom_jenkins_slave_fstrim=True docker_max_image_space="50 GB" -- To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/39333?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: osmo-ci Gerrit-Branch: master Gerrit-Change-Id: Id9d35658197409bcaba3e267cab36d7aa2a28972 Gerrit-Change-Number: 39333 Gerrit-PatchSet: 2 Gerrit-Owner: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>

1 0

[XS] Change in osmo-ci[master]: update coverity version to 2024.6.1 (current one)
by osmith 15 Jan '25

15 Jan '25

osmith has submitted this change. ( https://gerrit.osmocom.org/c/osmo-ci/+/39332?usp=email ) Change subject: update coverity version to 2024.6.1 (current one) ...................................................................... update coverity version to 2024.6.1 (current one) coverity started to reject builds with older versions Change-Id: Ifa5621279bbf14839096e10ced74ed855908897b --- M ansible/roles/install-coverity/defaults/main.yml 1 file changed, 1 insertion(+), 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/ansible/roles/install-coverity/defaults/main.yml b/ansible/roles/install-coverity/defaults/main.yml index 7be5be6..1176fab 100644 --- a/ansible/roles/install-coverity/defaults/main.yml +++ b/ansible/roles/install-coverity/defaults/main.yml @@ -1,4 +1,4 @@ --- -coverity_version: 2023.6.2 +coverity_version: 2024.6.1 coverity_installer_file: "cov-analysis-linux64-{{ coverity_version }}.tar.gz" -- To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/39332?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: osmo-ci Gerrit-Branch: master Gerrit-Change-Id: Ifa5621279bbf14839096e10ced74ed855908897b Gerrit-Change-Number: 39332 Gerrit-PatchSet: 2 Gerrit-Owner: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>

1 0

[XS] Change in osmo-ci[master]: ansible: build5-deb12build is also a coverity builder
by osmith 15 Jan '25

15 Jan '25

Attention is currently required from: laforge. osmith has posted comments on this change by laforge. ( https://gerrit.osmocom.org/c/osmo-ci/+/39333?usp=email ) Change subject: ansible: build5-deb12build is also a coverity builder ...................................................................... Patch Set 1: (1 comment) Patchset: PS1: I'll merge these for fixing https://osmocom.org/issues/6702 on top -- To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/39333?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-ci Gerrit-Branch: master Gerrit-Change-Id: Id9d35658197409bcaba3e267cab36d7aa2a28972 Gerrit-Change-Number: 39333 Gerrit-PatchSet: 1 Gerrit-Owner: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: laforge <laforge(a)osmocom.org> Gerrit-Comment-Date: Wed, 15 Jan 2025 15:20:55 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No

1 0

[M] Change in osmo-bsc[master]: nokia_site: add object_type, object_state and reset_type
by pespin 15 Jan '25

15 Jan '25

Attention is currently required from: csaba.sipos. pespin has posted comments on this change by csaba.sipos. ( https://gerrit.osmocom.org/c/osmo-bsc/+/39315?usp=email ) Change subject: nokia_site: add object_type, object_state and reset_type ...................................................................... Patch Set 1: (1 comment) Patchset: PS1: > Hi Pau, […] Put it together in the same patch with the code using it. -- To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/39315?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-bsc Gerrit-Branch: master Gerrit-Change-Id: I0f31cd711768c963e5c14c0f4969ff76cb05c12b Gerrit-Change-Number: 39315 Gerrit-PatchSet: 1 Gerrit-Owner: csaba.sipos <metro4(a)freemail.hu> Gerrit-Reviewer: Jenkins Builder Gerrit-CC: pespin <pespin(a)sysmocom.de> Gerrit-Attention: csaba.sipos <metro4(a)freemail.hu> Gerrit-Comment-Date: Wed, 15 Jan 2025 15:20:31 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: csaba.sipos <metro4(a)freemail.hu>

1 0

[M] Change in osmo-bsc[master]: nokia_site: add object_type, object_state and reset_type
by csaba.sipos 15 Jan '25

15 Jan '25

csaba.sipos has posted comments on this change by csaba.sipos. ( https://gerrit.osmocom.org/c/osmo-bsc/+/39315?usp=email ) Change subject: nokia_site: add object_type, object_state and reset_type ...................................................................... Patch Set 1: (1 comment) Patchset: PS1: Hi Pau, The only warning Jenkins gave is that the newly added functions are not used. Which is correct as this commit only adds these elements, related functions are coming later. What should I do? Keep it like this and then this can be reviewed and merged, or I can also revoke this commit and add everything together later. Although I think it is better to keep these patches small and separate. -- To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/39315?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-bsc Gerrit-Branch: master Gerrit-Change-Id: I0f31cd711768c963e5c14c0f4969ff76cb05c12b Gerrit-Change-Number: 39315 Gerrit-PatchSet: 1 Gerrit-Owner: csaba.sipos <metro4(a)freemail.hu> Gerrit-Reviewer: Jenkins Builder Gerrit-CC: pespin <pespin(a)sysmocom.de> Gerrit-Comment-Date: Wed, 15 Jan 2025 15:13:23 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No

1 0

[M] Change in osmo-pcap[master]: server: Split buffer read from full msg handling
by pespin 15 Jan '25

15 Jan '25

Attention is currently required from: laforge, pespin. Hello Jenkins Builder, laforge, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/c/osmo-pcap/+/39323?usp=email to look at the new patch set (#3). The following approvals got outdated and were removed: Verified+1 by Jenkins Builder Change subject: server: Split buffer read from full msg handling ...................................................................... server: Split buffer read from full msg handling This also simplifies follow-up patches modifying the way we read packets through osmo_stream. Change-Id: Ie7b8af0edbccc83812a44c6bfd61c98fbc3df324 --- M src/osmo_server_network.c 1 file changed, 37 insertions(+), 28 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-pcap refs/changes/23/39323/3 -- To view, visit https://gerrit.osmocom.org/c/osmo-pcap/+/39323?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newpatchset Gerrit-Project: osmo-pcap Gerrit-Branch: master Gerrit-Change-Id: Ie7b8af0edbccc83812a44c6bfd61c98fbc3df324 Gerrit-Change-Number: 39323 Gerrit-PatchSet: 3 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Attention: laforge <laforge(a)osmocom.org> Gerrit-Attention: pespin <pespin(a)sysmocom.de>

1 0

[M] Change in libosmo-netif[master]: Revert "contrib: remove rpm spec file"
by osmith 15 Jan '25

15 Jan '25

osmith has submitted this change. ( https://gerrit.osmocom.org/c/libosmo-netif/+/39338?usp=email ) Change subject: Revert "contrib: remove rpm spec file" ...................................................................... Revert "contrib: remove rpm spec file" Bring back the rpm spec file, as osmo-pcap will soon depend on libosmo-netif and we build osmo-pcap for centos7. This reverts commit 6930cceab0d54778e1e93d7a6ea9a891092469c4. Related: SYS#6760 Related: osmo-pcap I3562185f98685ea5a412273212e6dfbe55b360e2 Change-Id: I287b11059b0d45a691e7a8cde1d507081e57ea04 --- M Makefile.am M configure.ac A contrib/libosmo-netif.spec.in 3 files changed, 87 insertions(+), 1 deletion(-) Approvals: Jenkins Builder: Verified pespin: Looks good to me, approved diff --git a/Makefile.am b/Makefile.am index fd03b74..f50cd0f 100644 --- a/Makefile.am +++ b/Makefile.am @@ -9,6 +9,7 @@ EXTRA_DIST = \ .version \ README.md \ + contrib/libosmo-netif.spec.in \ debian \ git-version-gen \ $(NULL) diff --git a/configure.ac b/configure.ac index 9513270..ea8031b 100644 --- a/configure.ac +++ b/configure.ac @@ -135,4 +135,5 @@ utils/Makefile tests/Makefile Doxyfile - Makefile) + Makefile + contrib/libosmo-netif.spec) diff --git a/contrib/libosmo-netif.spec.in b/contrib/libosmo-netif.spec.in new file mode 100644 index 0000000..4ac1302 --- /dev/null +++ b/contrib/libosmo-netif.spec.in @@ -0,0 +1,84 @@ +# +# spec file for package libosmo-netif +# +# Copyright (c) 2018 SUSE LINUX GmbH, Nuernberg, Germany. +# +# All modifications and additions to the file contributed by third parties +# remain the property of their copyright owners, unless otherwise agreed +# upon. The license for this file, and modifications and additions to the +# file, is the same license as for the pristine package itself (unless the +# license for the pristine package is not an Open Source License, in which +# case the license is the MIT License). An "Open Source License" is a +# license that conforms to the Open Source Definition (Version 1.9) +# published by the Open Source Initiative. + +Name: libosmo-netif +Version: @VERSION@ +Release: 0 +Summary: Osmocom library for muxed audio +License: GPL-2.0-or-later +Group: Productivity/Telephony/Utilities +URL: https://osmocom.org/projects/libosmo-netif +Source: %{name}-%{version}.tar.xz +BuildRequires: automake +BuildRequires: libtool >= 2 +BuildRequires: lksctp-tools-devel +BuildRequires: pkgconfig >= 0.20 +BuildRequires: pkgconfig(libosmocore) >= 1.9.0 +BuildRequires: pkgconfig(libosmogsm) >= 1.9.0 +BuildRequires: pkgconfig(libosmocodec) >= 1.9.0 + +%description +Network interface demuxer library for OsmoCom projects. + +%package -n libosmonetif11 +Summary: Osmocom library for muxed audio +License: AGPL-3.0-or-later +Group: System/Libraries + +%description -n libosmonetif11 +Network interface demuxer library for OsmoCom projects. + +%package -n libosmonetif-devel +Summary: Development files for the Osmocom muxed audio library +License: AGPL-3.0-or-later +Group: Development/Libraries/C and C++ +Requires: libosmonetif11 = %{version} + +%description -n libosmonetif-devel +Network interface demuxer library for OsmoCom projects. + +This subpackage contains libraries and header files for developing +applications that want to make use of libosmo-netif. + +%prep +%setup -q + +%build +echo "%{version}" >.tarball-version +autoreconf -fiv +%configure --enable-shared --disable-static --includedir="%{_includedir}/%{name}" +make %{?_smp_mflags} + +%install +%make_install +find %{buildroot} -type f -name "*.la" -delete -print + +%check +make %{?_smp_mflags} check || (find . -name testsuite.log -exec cat {} +) + +%post -n libosmonetif11 -p /sbin/ldconfig +%postun -n libosmonetif11 -p /sbin/ldconfig + +%files -n libosmonetif11 +%{_libdir}/libosmonetif.so.11* + +%files -n libosmonetif-devel +%license COPYING +%dir %{_includedir}/%{name} +%dir %{_includedir}/%{name}/osmocom +%{_includedir}/%{name}/osmocom/netif/ +%{_libdir}/libosmonetif.so +%{_libdir}/pkgconfig/libosmo-netif.pc + +%changelog -- To view, visit https://gerrit.osmocom.org/c/libosmo-netif/+/39338?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: libosmo-netif Gerrit-Branch: master Gerrit-Change-Id: I287b11059b0d45a691e7a8cde1d507081e57ea04 Gerrit-Change-Number: 39338 Gerrit-PatchSet: 1 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>

1 0

[XS] Change in osmo-ci[master]: jobs/gerrit: build libosmo-netif for centos7
by osmith 15 Jan '25

15 Jan '25

osmith has submitted this change. ( https://gerrit.osmocom.org/c/osmo-ci/+/39339?usp=email ) Change subject: jobs/gerrit: build libosmo-netif for centos7 ...................................................................... jobs/gerrit: build libosmo-netif for centos7 osmo-pcap will soon depend on libosmo-netif, and we build osmo-pcap for centos7. Related: SYS#6760 Related: osmo-pcap I3562185f98685ea5a412273212e6dfbe55b360e2 Change-Id: Ie39f375152e8000c4e34bddd591c2c1ea91f9765 --- M jobs/gerrit-verifications.yml 1 file changed, 4 insertions(+), 1 deletion(-) Approvals: Jenkins Builder: Verified pespin: Looks good to me, approved diff --git a/jobs/gerrit-verifications.yml b/jobs/gerrit-verifications.yml index fdec2e8..440f032 100644 --- a/jobs/gerrit-verifications.yml +++ b/jobs/gerrit-verifications.yml @@ -125,7 +125,10 @@ - libgtpnl - libosmo-abis - libosmo-gprs - - libosmo-netif + + - libosmo-netif: + pipeline_binpkgs: "centos:7 debian:10 debian:12" # centos7: SYS#6760 (osmo-pcap dep) + - libosmo-pfcp - libosmo-sigtran -- To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/39339?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: osmo-ci Gerrit-Branch: master Gerrit-Change-Id: Ie39f375152e8000c4e34bddd591c2c1ea91f9765 Gerrit-Change-Number: 39339 Gerrit-PatchSet: 1 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>

1 0

[XS] Change in osmo-ci[master]: jobs/gerrit: build libosmo-netif for centos7
by pespin 15 Jan '25

15 Jan '25

Attention is currently required from: osmith. pespin has posted comments on this change by osmith. ( https://gerrit.osmocom.org/c/osmo-ci/+/39339?usp=email ) Change subject: jobs/gerrit: build libosmo-netif for centos7 ...................................................................... Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/39339?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-ci Gerrit-Branch: master Gerrit-Change-Id: Ie39f375152e8000c4e34bddd591c2c1ea91f9765 Gerrit-Change-Number: 39339 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: Wed, 15 Jan 2025 15:08:06 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

[M] Change in libosmo-netif[master]: Revert "contrib: remove rpm spec file"
by pespin 15 Jan '25

15 Jan '25

Attention is currently required from: osmith. pespin has posted comments on this change by osmith. ( https://gerrit.osmocom.org/c/libosmo-netif/+/39338?usp=email ) Change subject: Revert "contrib: remove rpm spec file" ...................................................................... Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/libosmo-netif/+/39338?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: I287b11059b0d45a691e7a8cde1d507081e57ea04 Gerrit-Change-Number: 39338 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: Wed, 15 Jan 2025 15:06:36 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

2026

2025

2024

2023

2022

gerrit-log