October 2022 - gerrit-log - lists.osmocom.org

Change in osmo-mgw[master]: mgcp-client: Introduce APIs to manually select mgcp_client from pool

by osmith

Attention is currently required from: pespin. osmith has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-mgw/+/29750 ) Change subject: mgcp-client: Introduce APIs to manually select mgcp_client from pool ...................................................................... Patch Set 2: (1 comment) File include/osmocom/mgcp_client/mgcp_client_pool.h: https://gerrit.osmocom.org/c/osmo-mgw/+/29750/comment/0be3a892_c52e4234 PS2, Line 17: mgcp_client_pool_member_get > This follows the classical get() and put() to obtain a pointer with a refcount. […] Ack -- To view, visit https://gerrit.osmocom.org/c/osmo-mgw/+/29750 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-mgw Gerrit-Branch: master Gerrit-Change-Id: I75ce3e04cd3f6d9cc20d7b4fc0f51083780786c8 Gerrit-Change-Number: 29750 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: Fri, 14 Oct 2022 11:14:24 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: osmith <osmith(a)sysmocom.de> Comment-In-Reply-To: pespin <pespin(a)sysmocom.de> Gerrit-MessageType: comment

2 years, 8 months

1
0
0 0

Change in osmo-mgw[master]: mgcp-client: Introduce APIs to manually select mgcp_client from pool

by pespin

Attention is currently required from: osmith. pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-mgw/+/29750 ) Change subject: mgcp-client: Introduce APIs to manually select mgcp_client from pool ...................................................................... Patch Set 2: (1 comment) File include/osmocom/mgcp_client/mgcp_client_pool.h: https://gerrit.osmocom.org/c/osmo-mgw/+/29750/comment/14882030_a5674d99 PS2, Line 17: mgcp_client_pool_member_get > Not so obvious from the function name what this does. […] This follows the classical get() and put() to obtain a pointer with a refcount. It can also be inferred by looking at mgcp_client_pool_get() or the documentation. -- To view, visit https://gerrit.osmocom.org/c/osmo-mgw/+/29750 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-mgw Gerrit-Branch: master Gerrit-Change-Id: I75ce3e04cd3f6d9cc20d7b4fc0f51083780786c8 Gerrit-Change-Number: 29750 Gerrit-PatchSet: 2 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Attention: osmith <osmith(a)sysmocom.de> Gerrit-Comment-Date: Fri, 14 Oct 2022 11:11:54 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: osmith <osmith(a)sysmocom.de> Gerrit-MessageType: comment

2 years, 8 months

1
0
0 0

Build failure of network:osmocom:nightly/osmo-hnbgw in Raspbian_10/armv7l

by OBS Notification

Visit https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/o… Package network:osmocom:nightly/osmo-hnbgw failed to build in Raspbian_10/armv7l Check out the package for editing: osc checkout network:osmocom:nightly osmo-hnbgw Last lines of build log: [ 416s] /var/lib/texmf/fmtutil.cnf-TEXLIVEDIST [ 416s] If you want to activate the changes in the above file(s), [ 416s] you should run fmtutil-sys or fmtutil. [ 416s] [473/487] installing texlive-extra-utils-2018.20190227-2 [ 430s] [474/487] installing texlive-2018.20190227-2 [ 430s] [475/487] installing texlive-pictures-2018.20190227-2 [ 437s] [476/487] installing texlive-science-2018.20190227-2 [ 443s] [477/487] installing libgtk2.0-0-2.24.32-3 [ 444s] [478/487] installing texlive-latex-extra-2018.20190227-2 [ 456s] [479/487] installing libgtkmm-2.4-1v5-1:2.24.5-4 [ 457s] [480/487] installing libgtkspell0-2.0.16-1.2 [ 458s] [481/487] installing texlive-xetex-2018.20190227-2 [ 470s] [482/487] installing dblatex-0.3.10-2 [ 473s] [483/487] installing inkscape-0.92.4-3 [ 502s] [484/487] installing texlive-formats-extra-2018.20190227-2 [ 502s] dpkg-deb (subprocess): decompressing archive member: lzma error: compressed data is corrupt [ 502s] dpkg-deb: error: <decompress> subprocess returned error exit status 2 [ 502s] dpkg: error processing archive .init_b_cache/texlive-formats-extra.deb (--install): [ 502s] cannot copy extracted data for './usr/share/doc/texlive-doc/eplain/eplain.pdf' to '/usr/share/doc/texlive-doc/eplain/eplain.pdf.dpkg-new': unexpected end of file or stream [ 502s] Errors were encountered while processing: [ 502s] .init_b_cache/texlive-formats-extra.deb [ 502s] exit ... [ 502s] ### VM INTERACTION START ### [ 503s] [ 473.631988] sysrq: SysRq : Power Off [ 503s] [ 473.633301] reboot: Power down [ 503s] ### VM INTERACTION END ### [ 503s] [ 503s] armbuild01 failed "build osmo-hnbgw_1.3.0.44.bbad.202210140849.dsc" at Fri Oct 14 11:10:14 UTC 2022. [ 503s] -- Configure notifications at https://build.opensuse.org/my/subscriptions openSUSE Build Service (https://build.opensuse.org/)

2 years, 8 months

1
0
0 0

Change in docker-playground[master]: debian-buster-jenkins: add LLVMEmbeddedToolchainForArm

by osmith

osmith has submitted this change. ( https://gerrit.osmocom.org/c/docker-playground/+/29739 ) Change subject: debian-buster-jenkins: add LLVMEmbeddedToolchainForArm ...................................................................... debian-buster-jenkins: add LLVMEmbeddedToolchainForArm This is needed for simtrace2. As the bootloader goes beyond partition size in modern gcc, we'll use clang instead. Related: OS#5260 Related: simtrace2 I2aa2e20e75e334560dbe1f6db9fd1491873ff91f Change-Id: I4f3fd7f7a40e5fa00bbfc713c1365f573febfe41 --- M debian-buster-jenkins/Dockerfile 1 file changed, 12 insertions(+), 0 deletions(-) Approvals: Jenkins Builder: Verified pespin: Looks good to me, but someone else must approve osmith: Looks good to me, approved diff --git a/debian-buster-jenkins/Dockerfile b/debian-buster-jenkins/Dockerfile index 269aa83..1439065 100644 --- a/debian-buster-jenkins/Dockerfile +++ b/debian-buster-jenkins/Dockerfile @@ -12,6 +12,9 @@ # https://docs.docker.com/engine/reference/builder/#understand-how-arg-and-fr… ARG DEBIAN_VERSION +# LLVM-embedded-toolchain-for-Arm version +ARG LLVM_VERSION="14.0.0" + # Install apt dependencies (keep in alphabetic order) RUN \ [ "$(arch)" = "x86_64" ] && dpkg --add-architecture i386; \ @@ -83,6 +86,7 @@ libsqlite3-dev \ libssl-dev \ libtalloc-dev \ + libtinfo5 \ libtool \ libusb-1.0-0-dev \ libusb-dev \ @@ -123,6 +127,14 @@ wget \ xsltproc +RUN set -x && \ + wget -q https://github.com/ARM-software/LLVM-embedded-toolchain-for-Arm/releases/do… && \ + tar -xf LLVMEmbeddedToolchainForArm-${LLVM_VERSION}-linux.tar.gz && \ + rm LLVMEmbeddedToolchainForArm-${LLVM_VERSION}-linux.tar.gz && \ + mv LLVMEmbeddedToolchainForArm-${LLVM_VERSION} /opt/llvm-arm && \ + /opt/llvm-arm/bin/clang --version && \ + /opt/llvm-arm/bin/clang --print-targets + # Install pip dependencies (keep in alphabetic order) RUN pip3 install \ git+https://github.com/podshumok/python-smpplib.git \ -- To view, visit https://gerrit.osmocom.org/c/docker-playground/+/29739 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: docker-playground Gerrit-Branch: master Gerrit-Change-Id: I4f3fd7f7a40e5fa00bbfc713c1365f573febfe41 Gerrit-Change-Number: 29739 Gerrit-PatchSet: 1 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: msuraev <msuraev(a)sysmocom.de> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-MessageType: merged

2 years, 8 months

1
0
0 0

Change in docker-playground[master]: Revert "debian-buster-build: add LLVMEmbeddedToolchainForArm"

by osmith

osmith has submitted this change. ( https://gerrit.osmocom.org/c/docker-playground/+/29737 ) Change subject: Revert "debian-buster-build: add LLVMEmbeddedToolchainForArm" ...................................................................... Revert "debian-buster-build: add LLVMEmbeddedToolchainForArm" Wrong docker container, debian-buster-jenkins is used for the build, not debian-buster-build. The next patches will adjust the right one. This reverts commit 1ce91ecfb027134c035d81756e47f27ba86e2be6. Change-Id: Idff1d5a263181070133743c781615b1102311efb --- M debian-buster-build/Dockerfile 1 file changed, 0 insertions(+), 12 deletions(-) Approvals: Jenkins Builder: Verified pespin: Looks good to me, but someone else must approve msuraev: Looks good to me, but someone else must approve osmith: Looks good to me, approved diff --git a/debian-buster-build/Dockerfile b/debian-buster-build/Dockerfile index a22a3bf..0d2b5b7 100644 --- a/debian-buster-build/Dockerfile +++ b/debian-buster-build/Dockerfile @@ -6,9 +6,6 @@ ARG OSMOCOM_REPO_PATH="packages/osmocom:" ARG OSMOCOM_REPO="$OSMOCOM_REPO_MIRROR/${OSMOCOM_REPO_PATH}/nightly/Debian_10/" -# LLVM-embedded-toolchain-for-Arm version -ARG LLVM_VERSION="14.0.0" - RUN apt-get update && \ apt-get upgrade -y && \ apt-get install -y --no-install-recommends \ @@ -53,7 +50,6 @@ libsqlite3-dev \ libssl-dev \ libtalloc-dev \ - libtinfo5 \ libtool \ libusb-1.0-0-dev \ libusb-dev \ @@ -65,14 +61,6 @@ wget && \ apt-get clean -RUN set -x && \ - wget -q https://github.com/ARM-software/LLVM-embedded-toolchain-for-Arm/releases/do… && \ - tar -xf LLVMEmbeddedToolchainForArm-${LLVM_VERSION}-linux.tar.gz && \ - rm LLVMEmbeddedToolchainForArm-${LLVM_VERSION}-linux.tar.gz && \ - mv LLVMEmbeddedToolchainForArm-${LLVM_VERSION} /opt/llvm-arm && \ - /opt/llvm-arm/bin/clang --version && \ - /opt/llvm-arm/bin/clang --print-targets - # Make respawn.sh part of this image, so it can be used by other images based on it COPY .common/respawn.sh /usr/local/bin/respawn.sh -- To view, visit https://gerrit.osmocom.org/c/docker-playground/+/29737 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: docker-playground Gerrit-Branch: master Gerrit-Change-Id: Idff1d5a263181070133743c781615b1102311efb Gerrit-Change-Number: 29737 Gerrit-PatchSet: 1 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: msuraev <msuraev(a)sysmocom.de> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-MessageType: merged

2 years, 8 months

1
0
0 0

Change in docker-playground[master]: debian-buster-jenkins: order pkgs alphabetically

by osmith

osmith has submitted this change. ( https://gerrit.osmocom.org/c/docker-playground/+/29738 ) Change subject: debian-buster-jenkins: order pkgs alphabetically ...................................................................... debian-buster-jenkins: order pkgs alphabetically Change-Id: I32dffc65239f1ad650b9e8d6ed6e4fe48878b916 --- M debian-buster-jenkins/Dockerfile 1 file changed, 5 insertions(+), 5 deletions(-) Approvals: Jenkins Builder: Verified pespin: Looks good to me, but someone else must approve osmith: Looks good to me, approved diff --git a/debian-buster-jenkins/Dockerfile b/debian-buster-jenkins/Dockerfile index 5229373..269aa83 100644 --- a/debian-buster-jenkins/Dockerfile +++ b/debian-buster-jenkins/Dockerfile @@ -97,6 +97,10 @@ osc \ patchelf \ pkg-config \ + python-pip \ + python-setuptools \ + python-yaml \ + python2.7 \ python3 \ python3-gi \ python3-mako \ @@ -106,14 +110,10 @@ python3-setuptools \ python3-usb \ python3-yaml \ - python2.7 \ - python-pip \ - python-setuptools \ - python-yaml \ rsync \ sdcc \ - sqlite3 \ source-highlight \ + sqlite3 \ stow \ sudo \ systemd \ -- To view, visit https://gerrit.osmocom.org/c/docker-playground/+/29738 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: docker-playground Gerrit-Branch: master Gerrit-Change-Id: I32dffc65239f1ad650b9e8d6ed6e4fe48878b916 Gerrit-Change-Number: 29738 Gerrit-PatchSet: 1 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: msuraev <msuraev(a)sysmocom.de> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-MessageType: merged

2 years, 8 months

1
0
0 0

Change in docker-playground[master]: Revert "debian-buster-build: add LLVMEmbeddedToolchainForArm"

by osmith

osmith has posted comments on this change. ( https://gerrit.osmocom.org/c/docker-playground/+/29737 ) Change subject: Revert "debian-buster-build: add LLVMEmbeddedToolchainForArm" ...................................................................... Patch Set 1: Code-Review+2 (1 comment) Patchset: PS1: 1+1=2 -- To view, visit https://gerrit.osmocom.org/c/docker-playground/+/29737 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: docker-playground Gerrit-Branch: master Gerrit-Change-Id: Idff1d5a263181070133743c781615b1102311efb Gerrit-Change-Number: 29737 Gerrit-PatchSet: 1 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: msuraev <msuraev(a)sysmocom.de> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Comment-Date: Fri, 14 Oct 2022 11:07:41 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

2 years, 8 months

1
0
0 0

Change in docker-playground[master]: debian-buster-jenkins: add LLVMEmbeddedToolchainForArm

by osmith

Attention is currently required from: msuraev. osmith has posted comments on this change. ( https://gerrit.osmocom.org/c/docker-playground/+/29739 ) Change subject: debian-buster-jenkins: add LLVMEmbeddedToolchainForArm ...................................................................... Patch Set 1: Code-Review+2 (1 comment) Patchset: PS1: this is basically the same as previously submitted and already approved patch https://gerrit.osmocom.org/c/docker-playground/+/29729, therefore setting +2 -- To view, visit https://gerrit.osmocom.org/c/docker-playground/+/29739 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: docker-playground Gerrit-Branch: master Gerrit-Change-Id: I4f3fd7f7a40e5fa00bbfc713c1365f573febfe41 Gerrit-Change-Number: 29739 Gerrit-PatchSet: 1 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: msuraev <msuraev(a)sysmocom.de> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: msuraev <msuraev(a)sysmocom.de> Gerrit-Comment-Date: Fri, 14 Oct 2022 11:07:23 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

2 years, 8 months

1
0
0 0

Change in docker-playground[master]: debian-buster-jenkins: order pkgs alphabetically

by osmith

Attention is currently required from: msuraev. osmith has posted comments on this change. ( https://gerrit.osmocom.org/c/docker-playground/+/29738 ) Change subject: debian-buster-jenkins: order pkgs alphabetically ...................................................................... Patch Set 1: Code-Review+2 (1 comment) Patchset: PS1: this is trivial, setting +2 -- To view, visit https://gerrit.osmocom.org/c/docker-playground/+/29738 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: docker-playground Gerrit-Branch: master Gerrit-Change-Id: I32dffc65239f1ad650b9e8d6ed6e4fe48878b916 Gerrit-Change-Number: 29738 Gerrit-PatchSet: 1 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: msuraev <msuraev(a)sysmocom.de> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: msuraev <msuraev(a)sysmocom.de> Gerrit-Comment-Date: Fri, 14 Oct 2022 11:06:44 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

2 years, 8 months

1
0
0 0

Change in osmo-ci[master]: jobs/gerrit-verifications: pipeline: agent none

by laforge

laforge has submitted this change. ( https://gerrit.osmocom.org/c/osmo-ci/+/29753 ) Change subject: jobs/gerrit-verifications: pipeline: agent none ...................................................................... jobs/gerrit-verifications: pipeline: agent none Don't allocate a slot on one of the jenkins nodes for running the pipeline. The pipeline job is lightweight, it just starts other jobs and waits for their completion. Without this patch we got into a situation where a lot of patches were submitted to gerrit at the same time, pipelines were started for all of them, taking up slots, and then all of the pipelines waiting until free slots become available before starting the actual build jobs. Thus blocking each other until I manually stopped them. Related: OS#2385 Change-Id: I289fcf1431a91a38fc37270e6a2f43d744c79e4f --- M jobs/gerrit-verifications.yml 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Jenkins Builder: Verified laforge: Looks good to me, approved diff --git a/jobs/gerrit-verifications.yml b/jobs/gerrit-verifications.yml index 36646f8..a9616ff 100644 --- a/jobs/gerrit-verifications.yml +++ b/jobs/gerrit-verifications.yml @@ -384,7 +384,7 @@ # https://jenkins.osmocom.org/jenkins/scriptApproval/ dsl: | pipeline {{ - agent {{ label "osmocom-gerrit" }} + agent none stages {{ stage("Verification") {{ parallel {{ -- To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/29753 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-ci Gerrit-Branch: master Gerrit-Change-Id: I289fcf1431a91a38fc37270e6a2f43d744c79e4f Gerrit-Change-Number: 29753 Gerrit-PatchSet: 2 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: msuraev <msuraev(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-MessageType: merged

2 years, 8 months

1
0
0 0

2025

2024

2023

2022

gerrit-log October 2022