- gerrit-log - lists.osmocom.org

[S] Change in osmo-ttcn3-hacks[master]: mgw: fix RTP packets received while RX disabled
by osmith 19 Nov '24

19 Nov '24

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>

1 0

[S] Change in osmo-ttcn3-hacks[master]: mgw: fix RTP packets received while RX disabled
by osmith 19 Nov '24

19 Nov '24

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>

1 0

[XS] Change in osmo-ttcn3-hacks[master]: testenv: make it pass 'ruff check' linter
by osmith 19 Nov '24

19 Nov '24

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>

1 0

[M] Change in osmo-ttcn3-hacks[master]: testenv: add run --until-nok
by osmith 19 Nov '24

19 Nov '24

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>

1 0

[M] Change in osmo-ttcn3-hacks[master]: testenv: install dbg pkgs with --binary-repo too
by osmith 19 Nov '24

19 Nov '24

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/+/38803?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: 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(-) git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/03/38803/3 -- 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: newpatchset 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: pespin <pespin(a)sysmocom.de> Gerrit-Attention: osmith <osmith(a)sysmocom.de>

1 0

[S] Change in osmo-ci[master]: jobs/registry-rebuild-upload-android: new job
by osmith 19 Nov '24

19 Nov '24

osmith has submitted this change. ( https://gerrit.osmocom.org/c/osmo-ci/+/38800?usp=email ) Change subject: jobs/registry-rebuild-upload-android: new job ...................................................................... jobs/registry-rebuild-upload-android: new job Add a job to build and upload the debian-bookworm-android image, to be used for building the android-apdu-proxy app and possibly more in the future. Related: OS#6367 Depends: docker-playground Ic7a454be23072c3f0ed609d370fc88f49205b04c Change-Id: Ia71bf8df638b9c4359d8b1a723795029541e023a --- A jobs/registry-rebuild-upload.yml 1 file changed, 39 insertions(+), 0 deletions(-) Approvals: pespin: Looks good to me, but someone else must approve osmith: Looks good to me, approved Jenkins Builder: Verified laforge: Looks good to me, but someone else must approve diff --git a/jobs/registry-rebuild-upload.yml b/jobs/registry-rebuild-upload.yml new file mode 100644 index 0000000..c5a094a --- /dev/null +++ b/jobs/registry-rebuild-upload.yml @@ -0,0 +1,39 @@ +- project: + name: registry-rebuild-upload + jobs: + - "{job-name}" + job-name: + - registry-rebuild-upload-android: + image: "debian-bookworm-android" + +- job-template: + name: '{job-name}' + description: | + Build the <code>{image}</code> image from + <a href="https://gitea.osmocom.org/osmocom/docker-playground/"> + docker-playground + </a> and push it to the Osmocom registry. + concurrent: false + parameters: + - string: + name: BRANCH + description: | + Branch of <code>docker-playground.git</code> + default: 'master' + scm: + - git: + url: https://gerrit.osmocom.org/docker-playground + skip-tag: true + branches: + - '$BRANCH' + builders: + - shell: | + REGISTRY="registry.osmocom.org" + make -C "{image}" REGISTRY_HOST=$REGISTRY + docker push "$REGISTRY/$USER/{image}" + properties: + - build-discarder: + num-to-keep: 120 + wrappers: + - ansicolor: + colormap: xterm -- To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/38800?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: Ia71bf8df638b9c4359d8b1a723795029541e023a Gerrit-Change-Number: 38800 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-Reviewer: pespin <pespin(a)sysmocom.de>

1 0

[S] Change in osmo-ci[master]: jobs/master-builds: add android-apdu-proxy
by osmith 19 Nov '24

19 Nov '24

osmith has submitted this change. ( https://gerrit.osmocom.org/c/osmo-ci/+/38801?usp=email ) Change subject: jobs/master-builds: add android-apdu-proxy ...................................................................... jobs/master-builds: add android-apdu-proxy Gradle downloads a lot of files during the build, mount the cache dirs so this is only done once for master builds. Related: OS#6367 Change-Id: Ie7c3948e8c4a1a75a1efaac3e7c238601c5cdccb --- M jobs/master-builds.yml 1 file changed, 14 insertions(+), 0 deletions(-) Approvals: Jenkins Builder: Verified pespin: Looks good to me, but someone else must approve osmith: Looks good to me, approved laforge: Looks good to me, but someone else must approve diff --git a/jobs/master-builds.yml b/jobs/master-builds.yml index f711d4d..3896e67 100644 --- a/jobs/master-builds.yml +++ b/jobs/master-builds.yml @@ -67,6 +67,7 @@ -v "$CCACHE_DIR:/ccache" \ docker_img: '$USER/debian-bookworm-build' docker_img_erlang: '$USER/debian-bookworm-erlang' + docker_img_android: 'registry.osmocom.org/osmocom-build/debian-bookworm-android' timeout_cmd: '/usr/bin/timeout 30m' cmd: '{docker_run_ccache} {docker_img} {timeout_cmd} /build/contrib/jenkins.sh' trigger: @@ -80,6 +81,19 @@ # in alphabetical order repos: + - android-apdu-proxy: + repos_url: '{gitea_url}/sim-card/android-apdu-proxy' + branch: master + cmd: | + ANDROID_CACHE="$HOME/android-cache/master-builds/android" + GRADLE_CACHE="$HOME/android-cache/master-builds/gradle" + mkdir -p "$ANDROID_CACHE" "$GRADLE_CACHE" + docker pull {docker_img_android} + {docker_run} \ + -v "$ANDROID_CACHE":/build/.android \ + -v "$GRADLE_CACHE":/build/.gradle \ + {docker_img_android} {timeout_cmd} contrib/jenkins.sh + - asn1c: branch: osmo-iuh/master -- To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/38801?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: Ie7c3948e8c4a1a75a1efaac3e7c238601c5cdccb Gerrit-Change-Number: 38801 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-Reviewer: pespin <pespin(a)sysmocom.de>

1 0

[S] Change in osmo-ci[master]: jobs/master-builds: archive android-apdu-proxy apk
by osmith 19 Nov '24

19 Nov '24

osmith has submitted this change. ( https://gerrit.osmocom.org/c/osmo-ci/+/38802?usp=email ) Change subject: jobs/master-builds: archive android-apdu-proxy apk ...................................................................... jobs/master-builds: archive android-apdu-proxy apk Store the apk after successful run of the android-apdu-proxy job. It can be found by clicking on a job at [1] and then clicking on "default" (since the job is behind a matrix, as all master jobs). [1]: https://jenkins.osmocom.org/jenkins/job/master-android-apdu-proxy/ Related: OS#6367 Change-Id: If3224da6c4e3fa90fa84abfec4e75efd634a8bfa --- M jobs/master-builds.yml 1 file changed, 7 insertions(+), 3 deletions(-) Approvals: pespin: Looks good to me, but someone else must approve Jenkins Builder: Verified laforge: Looks good to me, but someone else must approve osmith: Looks good to me, approved diff --git a/jobs/master-builds.yml b/jobs/master-builds.yml index 3896e67..e37d1d2 100644 --- a/jobs/master-builds.yml +++ b/jobs/master-builds.yml @@ -78,6 +78,7 @@ branch: master ssh: - docs_ftp.osmocom.org + archive_artifacts_on_success: false # in alphabetical order repos: @@ -93,6 +94,7 @@ -v "$ANDROID_CACHE":/build/.android \ -v "$GRADLE_CACHE":/build/.gradle \ {docker_img_android} {timeout_cmd} contrib/jenkins.sh + archive_artifacts_on_success: true - asn1c: branch: osmo-iuh/master @@ -677,13 +679,15 @@ - shell: | # Remove workspace on success to free up space. Create an empty dir # so the job does not abort when attempting to collect artifacts. - rm -rf "$WORKSPACE" - mkdir -p "$WORKSPACE" + if [ "{archive_artifacts_on_success}" != "True" ]; then + rm -rf "$WORKSPACE" + mkdir -p "$WORKSPACE" + fi publishers: - archive: allow-empty: true - artifacts: '**/core, **/testsuite.log, **/workspace.tar.xz' + artifacts: '**/core, **/testsuite.log, **/workspace.tar.xz, **/app-release.apk' - warnings: console-log-parsers: - 'GNU C Compiler 4 (gcc)' -- To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/38802?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: If3224da6c4e3fa90fa84abfec4e75efd634a8bfa Gerrit-Change-Number: 38802 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-Reviewer: pespin <pespin(a)sysmocom.de>

1 0

[S] Change in osmo-ci[master]: jobs/master-builds: archive android-apdu-proxy apk
by osmith 19 Nov '24

19 Nov '24

osmith has posted comments on this change by osmith. ( https://gerrit.osmocom.org/c/osmo-ci/+/38802?usp=email ) Change subject: jobs/master-builds: archive android-apdu-proxy apk ...................................................................... Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/38802?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: If3224da6c4e3fa90fa84abfec4e75efd634a8bfa Gerrit-Change-Number: 38802 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-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Comment-Date: Tue, 19 Nov 2024 07:12:16 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

[S] Change in osmo-ci[master]: jobs/master-builds: add android-apdu-proxy
by osmith 19 Nov '24

19 Nov '24

osmith has posted comments on this change by osmith. ( https://gerrit.osmocom.org/c/osmo-ci/+/38801?usp=email ) Change subject: jobs/master-builds: add android-apdu-proxy ...................................................................... Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/38801?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: Ie7c3948e8c4a1a75a1efaac3e7c238601c5cdccb Gerrit-Change-Number: 38801 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-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Comment-Date: Tue, 19 Nov 2024 07:12:08 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

2026

2025

2024

2023

2022

gerrit-log