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

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

by osmith

osmith has uploaded this change for review. ( https://gerrit.osmocom.org/c/docker-playground/+/29728 ) Change subject: debian-buster-build: order pkgs alphabetically ...................................................................... debian-buster-build: order pkgs alphabetically Change-Id: I16da41b9bcb3d1137218b0ed5c2f6adf2a723fce --- M debian-buster-build/Dockerfile 1 file changed, 3 insertions(+), 3 deletions(-) git pull ssh://gerrit.osmocom.org:29418/docker-playground refs/changes/28/29728/1 diff --git a/debian-buster-build/Dockerfile b/debian-buster-build/Dockerfile index 80e5678..0d2b5b7 100644 --- a/debian-buster-build/Dockerfile +++ b/debian-buster-build/Dockerfile @@ -44,15 +44,15 @@ libortp-dev \ libpcap-dev \ libpcsclite-dev \ - libtalloc-dev \ - libtool \ libreadline-dev \ libsctp-dev \ libsofia-sip-ua-glib-dev \ libsqlite3-dev \ libssl-dev \ - libusb-dev \ + libtalloc-dev \ + libtool \ libusb-1.0-0-dev \ + libusb-dev \ make \ pkg-config \ sqlite3 \ -- To view, visit https://gerrit.osmocom.org/c/docker-playground/+/29728 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: docker-playground Gerrit-Branch: master Gerrit-Change-Id: I16da41b9bcb3d1137218b0ed5c2f6adf2a723fce Gerrit-Change-Number: 29728 Gerrit-PatchSet: 1 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-MessageType: newchange

2 years, 8 months

1
0
0 0

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

by osmith

osmith has uploaded this change for review. ( https://gerrit.osmocom.org/c/docker-playground/+/29729 ) Change subject: debian-buster-build: add LLVMEmbeddedToolchainForArm ...................................................................... debian-buster-build: add LLVMEmbeddedToolchainForArm Related: OS#5260 Change-Id: Ib82a53fa7edc62d21e772efbb9b2c049d1b50c4d --- M debian-buster-build/Dockerfile 1 file changed, 12 insertions(+), 0 deletions(-) git pull ssh://gerrit.osmocom.org:29418/docker-playground refs/changes/29/29729/1 diff --git a/debian-buster-build/Dockerfile b/debian-buster-build/Dockerfile index 0d2b5b7..a22a3bf 100644 --- a/debian-buster-build/Dockerfile +++ b/debian-buster-build/Dockerfile @@ -6,6 +6,9 @@ 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 \ @@ -50,6 +53,7 @@ libsqlite3-dev \ libssl-dev \ libtalloc-dev \ + libtinfo5 \ libtool \ libusb-1.0-0-dev \ libusb-dev \ @@ -61,6 +65,14 @@ 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/+/29729 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: docker-playground Gerrit-Branch: master Gerrit-Change-Id: Ib82a53fa7edc62d21e772efbb9b2c049d1b50c4d Gerrit-Change-Number: 29729 Gerrit-PatchSet: 1 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-MessageType: newchange

2 years, 8 months

1
0
0 0

Change in osmo-ci[master]: jobs/gerrit-verifications: add approvals comment

by osmith

Attention is currently required from: msuraev. osmith has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ci/+/29687 ) Change subject: jobs/gerrit-verifications: add approvals comment ...................................................................... Patch Set 3: (1 comment) File jobs/gerrit-verifications.yml: https://gerrit.osmocom.org/c/osmo-ci/+/29687/comment/9b9ae7cb_4801d991 PS3, Line 21: # NOTE: after updating the job with Jenkins Job Builder as usually, check if a > "as usual" sounds better I think. Done -- To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/29687 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-ci Gerrit-Branch: master Gerrit-Change-Id: I4dfc0f84ffe584523bbff4c41295968a07ba097d Gerrit-Change-Number: 29687 Gerrit-PatchSet: 3 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: msuraev <msuraev(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: msuraev <msuraev(a)sysmocom.de> Gerrit-Comment-Date: Thu, 13 Oct 2022 08:52:08 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: msuraev <msuraev(a)sysmocom.de> Gerrit-MessageType: comment

2 years, 8 months

1
0
0 0

Change in osmo-ci[master]: jobs/gerrit-verifications: add approvals comment

by osmith

Attention is currently required from: osmith. Hello Jenkins Builder, pespin, msuraev, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/c/osmo-ci/+/29687 to look at the new patch set (#4). Change subject: jobs/gerrit-verifications: add approvals comment ...................................................................... jobs/gerrit-verifications: add approvals comment Related: OS#2385 Change-Id: I4dfc0f84ffe584523bbff4c41295968a07ba097d --- M jobs/gerrit-verifications.yml 1 file changed, 10 insertions(+), 0 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-ci refs/changes/87/29687/4 -- To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/29687 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-ci Gerrit-Branch: master Gerrit-Change-Id: I4dfc0f84ffe584523bbff4c41295968a07ba097d Gerrit-Change-Number: 29687 Gerrit-PatchSet: 4 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: msuraev <msuraev(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: osmith <osmith(a)sysmocom.de> Gerrit-MessageType: newpatchset

2 years, 8 months

1
0
0 0

Change in osmo-gsm-manuals[master]: osmux: Improve NAT related documentation

by osmith

Attention is currently required from: pespin. osmith has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-gsm-manuals/+/29723 ) Change subject: osmux: Improve NAT related documentation ...................................................................... Patch Set 1: Code-Review+1 (2 comments) File common/chapters/osmux/osmux.adoc: https://gerrit.osmocom.org/c/osmo-gsm-manuals/+/29723/comment/044db2c6_9e56… PS1, Line 57: charachtersitic characteristic https://gerrit.osmocom.org/c/osmo-gsm-manuals/+/29723/comment/371cb70a_3e82… PS1, Line 69: insance instance -- To view, visit https://gerrit.osmocom.org/c/osmo-gsm-manuals/+/29723 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-gsm-manuals Gerrit-Branch: master Gerrit-Change-Id: I5ed7aa834bfd0d5ef84ff69ece6d1d5e5b920d4e Gerrit-Change-Number: 29723 Gerrit-PatchSet: 1 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: Thu, 13 Oct 2022 08:49:05 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

2 years, 8 months

1
0
0 0

Change in osmo-mgw[master]: osmux: Clean up mgcp_config osmux fields

by osmith

Attention is currently required from: pespin. osmith has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-mgw/+/29725 ) Change subject: osmux: Clean up mgcp_config osmux fields ...................................................................... Patch Set 1: (2 comments) Patchset: PS1: I'd add indentation for the struct, otherwise LGTM File include/osmocom/mgcp/mgcp.h: https://gerrit.osmocom.org/c/osmo-mgw/+/29725/comment/c7e4eeb7_d25ad70b PS1, Line 162: enum osmux_usage usage; no indentation for contents of the struct? -- To view, visit https://gerrit.osmocom.org/c/osmo-mgw/+/29725 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-mgw Gerrit-Branch: master Gerrit-Change-Id: Icc1c1ce3ab3a8beb941cda7c112485a0db882a90 Gerrit-Change-Number: 29725 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-CC: osmith <osmith(a)sysmocom.de> Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Comment-Date: Thu, 13 Oct 2022 08:42:00 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment

2 years, 8 months

1
0
0 0

Change in osmo-mgw[master]: osmux: Introduce osmux peer-behind-nat (on|off) and rework conn activ...

by osmith

Attention is currently required from: pespin. osmith has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-mgw/+/29659 ) Change subject: osmux: Introduce osmux peer-behind-nat (on|off) and rework conn activation ...................................................................... Patch Set 3: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/c/osmo-mgw/+/29659 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-mgw Gerrit-Branch: master Gerrit-Change-Id: I7654ddf51d197a4107e55f4e406053b2e4a02f83 Gerrit-Change-Number: 29659 Gerrit-PatchSet: 3 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: Thu, 13 Oct 2022 08:35:32 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

2 years, 8 months

1
0
0 0

Change in osmo-ci[master]: jenkins-gerrit: tweak notify setting

by osmith

Attention is currently required from: pespin. osmith has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ci/+/29702 ) Change subject: jenkins-gerrit: tweak notify setting ...................................................................... Patch Set 1: (1 comment) Patchset: PS1: > That may be a bit problematic. […] I'm pretty sure that only having the owner of the patch notified is the behavior we already had before introducing the recent changes: whoever submits the patch is the owner of the patch and gets the notification if one of the CI checks fails. So I guess there was a misunderstanding here, owner meaning the person who submitted the patch and not an administrator or something. Does that resolve it? -- To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/29702 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-ci Gerrit-Branch: master Gerrit-Change-Id: I7c6c183b98624eb75d5dccd9766ee3ff5568b06e Gerrit-Change-Number: 29702 Gerrit-PatchSet: 1 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-CC: pespin <pespin(a)sysmocom.de> Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Comment-Date: Thu, 13 Oct 2022 08:23:10 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: pespin <pespin(a)sysmocom.de> Gerrit-MessageType: comment

2 years, 8 months

1
0
0 0

Change in osmo-ci[master]: jenkins-gerrit: meaningful re_start_build error

by osmith

osmith has submitted this change. ( https://gerrit.osmocom.org/c/osmo-ci/+/29700 ) Change subject: jenkins-gerrit: meaningful re_start_build error ...................................................................... jenkins-gerrit: meaningful re_start_build error Related: OS#2385 Change-Id: Ib29b7ce69a27453b460f946ad67bd12faffc2c89 --- M scripts/jenkins-gerrit/pipeline_summary.py 1 file changed, 4 insertions(+), 0 deletions(-) Approvals: Jenkins Builder: Verified laforge: 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/scripts/jenkins-gerrit/pipeline_summary.py b/scripts/jenkins-gerrit/pipeline_summary.py index c12ad64..85b2393 100755 --- a/scripts/jenkins-gerrit/pipeline_summary.py +++ b/scripts/jenkins-gerrit/pipeline_summary.py @@ -65,6 +65,10 @@ # Parse result lines if re_result.match(line): stage = line.split("_")[1].lower() + assert stage in ret, f"found result for stage {stage}, but" \ + " didn't find where it was started. The" \ + " re_start_build regex probably needs to be adjusted" \ + " to match the related gerrit-*-build job." passed = line.split("=")[1].rstrip() == "1" ret[stage]["passed"] = passed -- To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/29700 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-ci Gerrit-Branch: master Gerrit-Change-Id: Ib29b7ce69a27453b460f946ad67bd12faffc2c89 Gerrit-Change-Number: 29700 Gerrit-PatchSet: 1 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: 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 osmo-ci[master]: jenkins-gerrit: add 0-9 to re_start_build

by osmith

osmith has submitted this change. ( https://gerrit.osmocom.org/c/osmo-ci/+/29701 ) Change subject: jenkins-gerrit: add 0-9 to re_start_build ...................................................................... jenkins-gerrit: add 0-9 to re_start_build Let it match "Starting building: gerrit-osmo-ttcn3-hacks-build #5". It failed on the 3 in ttcn3 before. Related: OS#2385 Change-Id: I247af55e2c0e3a2dd088ab1c951d8535cfc93229 --- M scripts/jenkins-gerrit/pipeline_summary.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Jenkins Builder: Verified laforge: Looks good to me, approved diff --git a/scripts/jenkins-gerrit/pipeline_summary.py b/scripts/jenkins-gerrit/pipeline_summary.py index 85b2393..37d4028 100755 --- a/scripts/jenkins-gerrit/pipeline_summary.py +++ b/scripts/jenkins-gerrit/pipeline_summary.py @@ -8,7 +8,7 @@ import urllib.request jenkins_url = "https://jenkins.osmocom.org" -re_start_build = re.compile("Starting building: gerrit-[a-zA-Z-_]* #[0-9]*") +re_start_build = re.compile("Starting building: gerrit-[a-zA-Z-_0-9]* #[0-9]*") re_result = re.compile("^PIPELINE_[A-Z]*_PASSED=[01]$") def parse_args(): -- To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/29701 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-ci Gerrit-Branch: master Gerrit-Change-Id: I247af55e2c0e3a2dd088ab1c951d8535cfc93229 Gerrit-Change-Number: 29701 Gerrit-PatchSet: 1 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-MessageType: merged

2 years, 8 months

1
0
0 0

2025

2024

2023

2022

gerrit-log October 2022