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

Change in osmo-ci[master]: jobs/coverity.yml: add parameter for git branch
by osmith 12 Jul '22

12 Jul '22

osmith has submitted this change. ( https://gerrit.osmocom.org/c/osmo-ci/+/28580 ) Change subject: jobs/coverity.yml: add parameter for git branch ...................................................................... jobs/coverity.yml: add parameter for git branch Allow configuring a different git branch before starting the build. This is especially useful to reproduce a failure that only happens when using coverity tools during a build. Change-Id: I5f7c615ff5af4bf1f34ccef62767b5ca71cd2d8d --- M jobs/coverity.yml 1 file changed, 6 insertions(+), 1 deletion(-) Approvals: Jenkins Builder: Verified fixeria: Looks good to me, approved diff --git a/jobs/coverity.yml b/jobs/coverity.yml index 980c30c..cb39ea6 100644 --- a/jobs/coverity.yml +++ b/jobs/coverity.yml @@ -24,13 +24,18 @@ - build-discarder: days-to-keep: 30 num-to-keep: 30 + parameters: + - string: + name: BRANCH + description: osmo-ci.git branch + default: 'origin/master' scm: - git: url: git://git.osmocom.org/osmo-ci git-config-name: 'Jenkins Builder' git-config-email: 'jenkins(a)osmocom.org' branches: - - 'origin/master' + - '$BRANCH' triggers: - timed: "@daily" publishers: -- To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/28580 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-ci Gerrit-Branch: master Gerrit-Change-Id: I5f7c615ff5af4bf1f34ccef62767b5ca71cd2d8d Gerrit-Change-Number: 28580 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-MessageType: merged

1 0

Change in osmo-ci[master]: coverity: don't build tests for all
by osmith 12 Jul '22

12 Jul '22

osmith has submitted this change. ( https://gerrit.osmocom.org/c/osmo-ci/+/28576 ) Change subject: coverity: don't build tests for all ...................................................................... coverity: don't build tests for all The coverity jenkins job has been failing since we tried to build tests for all projects (and in an earlier attempt, build and run tests for all projects). Make the coverity job work again by skipping build of tests for asn1c. I looked into it earlier and the cause wasn't obvious to me, and it's currently not worth spending time on. Fix for: libtool: error: cannot find the library '../../skeletons/libasn1cskeletons.la' or unhandled argument '../../skeletons/libasn1cskeletons.la' Change-Id: Iba0bfe0e753812058436eaf515391eb1f4b09951 --- M coverity/prepare_source_Osmocom.sh 1 file changed, 41 insertions(+), 35 deletions(-) Approvals: Jenkins Builder: Verified fixeria: Looks good to me, but someone else must approve pespin: Looks good to me, but someone else must approve osmith: Looks good to me, approved diff --git a/coverity/prepare_source_Osmocom.sh b/coverity/prepare_source_Osmocom.sh index 11b7e5c..01e3d73 100755 --- a/coverity/prepare_source_Osmocom.sh +++ b/coverity/prepare_source_Osmocom.sh @@ -1,42 +1,46 @@ #!/bin/sh BASEDIR=source-Osmocom +PROJECTS=" + libasn1c + libosmo-abis + libosmocore + libosmo-dsp + libosmo-netif + libosmo-sccp + libsmpp34 + libusrp + osmo-bsc + osmo-msc + osmo-mgw + osmo-ggsn + osmo-gbproxy + osmo-sgsn + osmo-bts + osmo-gmr + osmo-iuh + osmo-pcu + osmo-sysmon + osmo-sip-connector + osmo-trx + osmo-hlr + osmocom-bb + osmo-smlc + osmo-cbc + simtrace2 + osmo-hnodeb + osmo-hnbgw + osmo-bsc-nat +" + +PROJECTS_DONT_BUILD_TEST=" + asn1c +" + [ -d $BASEDIR ] || mkdir -p $BASEDIR cd $BASEDIR -for proj in \ - asn1c \ - libasn1c \ - libosmo-abis \ - libosmocore \ - libosmo-dsp \ - libosmo-netif \ - libosmo-sccp \ - libsmpp34 \ - libusrp \ - osmo-bsc \ - osmo-msc \ - osmo-mgw \ - osmo-ggsn \ - osmo-gbproxy \ - osmo-sgsn \ - osmo-bts \ - osmo-gmr \ - osmo-iuh \ - osmo-pcu \ - osmo-sysmon \ - osmo-sip-connector \ - osmo-trx \ - osmo-hlr \ - osmocom-bb \ - osmo-smlc \ - osmo-cbc \ - simtrace2 \ - osmo-hnodeb \ - osmo-hnbgw \ - osmo-bsc-nat \ - ; do - +for proj in $PROJECTS $PROJECTS_DONT_BUILD_TEST; do if [ -d $proj ]; then if [ -z "$SRC_SKIP_FETCH" ]; then (cd $proj && git fetch && git checkout -f -B master origin/master) @@ -47,9 +51,11 @@ else git clone git://git.osmocom.org/$proj fi +done - # We want to compile tests, but not execute them. Using 'noinst_PROGRAMS' - # instead of 'check_PROGRAMS' allows building test binaries during 'make all'. +# We want to compile tests, but not execute them. Using 'noinst_PROGRAMS' +# instead of 'check_PROGRAMS' allows building test binaries during 'make all'. +for proj in $PROJECTS; do files="$(git -C $proj grep -l check_PROGRAMS)" if [ -n "$files" ]; then (cd $proj && sed -i "s/check_PROGRAMS/noinst_PROGRAMS/" $files) -- To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/28576 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-ci Gerrit-Branch: master Gerrit-Change-Id: Iba0bfe0e753812058436eaf515391eb1f4b09951 Gerrit-Change-Number: 28576 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

Change in osmo-ci[master]: Revert "WIP: coverity: Make sure curl is installed, as our coverity u...
by osmith 12 Jul '22

12 Jul '22

osmith has submitted this change. ( https://gerrit.osmocom.org/c/osmo-ci/+/28577 ) Change subject: Revert "WIP: coverity: Make sure curl is installed, as our coverity upload job needs it" ...................................................................... Revert "WIP: coverity: Make sure curl is installed, as our coverity upload job needs it" Revert 3038c19155a7a2fb4b075a84449fc09d06bef59a, which adds a second osmocom-coverity-deps job that apparently was never deployed (or was deployed and then deleted). The mentioned curl dependency shouldn't go here either. Change-Id: I5d4709c0019ef4bae6ccbd11c28eeb23d75692c1 --- M jobs/coverity.yml 1 file changed, 0 insertions(+), 10 deletions(-) Approvals: Jenkins Builder: Verified fixeria: Looks good to me, but someone else must approve pespin: Looks good to me, but someone else must approve osmith: Looks good to me, approved diff --git a/jobs/coverity.yml b/jobs/coverity.yml index 5614fc0..576fa29 100644 --- a/jobs/coverity.yml +++ b/jobs/coverity.yml @@ -2,19 +2,9 @@ - project: name: osmocom-coverity jobs: - - osmocom-coverity-deps - osmocom-coverity - job: - name: 'osmocom-coverity-deps' - apt: - name: "{{ item }}" - cache_valid_time: 3600 - update_cache: yes - with_items: - - curl - -- job: name: 'osmocom-coverity' project-type: freestyle defaults: global -- To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/28577 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-ci Gerrit-Branch: master Gerrit-Change-Id: I5d4709c0019ef4bae6ccbd11c28eeb23d75692c1 Gerrit-Change-Number: 28577 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

Change in osmo-ci[master]: jobs/coverity: modernize
by osmith 12 Jul '22

12 Jul '22

osmith has submitted this change. ( https://gerrit.osmocom.org/c/osmo-ci/+/28578 ) Change subject: jobs/coverity: modernize ...................................................................... jobs/coverity: modernize * no need to mention "(jenkins-job-builder)" in the display-name * set mail to jenkins-notifications ML like most other jobs Change-Id: I0284dd7354157e057903f53cde149ce652e76550 --- M jobs/coverity.yml 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Jenkins Builder: Verified fixeria: Looks good to me, approved diff --git a/jobs/coverity.yml b/jobs/coverity.yml index 576fa29..88b8a49 100644 --- a/jobs/coverity.yml +++ b/jobs/coverity.yml @@ -9,7 +9,7 @@ project-type: freestyle defaults: global description: 'Generated by job-builder' - display-name: 'Coverity-Upload (jenkins-job-builder)' + display-name: 'Coverity-Upload' node: coverity builders: - shell: | @@ -18,7 +18,7 @@ ./jenkins.sh publishers: - email: - recipients: openbsc(a)lists.osmocom.org holger(a)freyther.de + recipients: 'jenkins-notifications(a)lists.osmocom.org' notify-every-unstable-build: true properties: - build-discarder: -- To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/28578 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-ci Gerrit-Branch: master Gerrit-Change-Id: I0284dd7354157e057903f53cde149ce652e76550 Gerrit-Change-Number: 28578 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-MessageType: merged

1 0

Change in osmo-ci[master]: jobs/coverity: don't cd ~/osmo-ci/coverity
by osmith 12 Jul '22

12 Jul '22

osmith has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ci/+/28579 ) Change subject: jobs/coverity: don't cd ~/osmo-ci/coverity ...................................................................... Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/28579 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-ci Gerrit-Branch: master Gerrit-Change-Id: Ie992d71d2afec6982447b8fe3626c821a9e48d19 Gerrit-Change-Number: 28579 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-Comment-Date: Tue, 12 Jul 2022 14:03:29 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

1 0

Change in osmo-ci[master]: Revert "WIP: coverity: Make sure curl is installed, as our coverity u...
by osmith 12 Jul '22

12 Jul '22

osmith has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ci/+/28577 ) Change subject: Revert "WIP: coverity: Make sure curl is installed, as our coverity upload job needs it" ...................................................................... Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/28577 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-ci Gerrit-Branch: master Gerrit-Change-Id: I5d4709c0019ef4bae6ccbd11c28eeb23d75692c1 Gerrit-Change-Number: 28577 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-Comment-Date: Tue, 12 Jul 2022 14:03:25 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

1 0

Change in osmo-ci[master]: coverity: don't build tests for all
by osmith 12 Jul '22

12 Jul '22

osmith has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ci/+/28576 ) Change subject: coverity: don't build tests for all ...................................................................... Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/28576 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-ci Gerrit-Branch: master Gerrit-Change-Id: Iba0bfe0e753812058436eaf515391eb1f4b09951 Gerrit-Change-Number: 28576 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-Comment-Date: Tue, 12 Jul 2022 14:03:20 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

1 0

Change in osmo-ci[master]: coverity: don't build tests for all
by pespin 12 Jul '22

12 Jul '22

Attention is currently required from: osmith. pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ci/+/28576 ) Change subject: coverity: don't build tests for all ...................................................................... Patch Set 1: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/28576 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-ci Gerrit-Branch: master Gerrit-Change-Id: Iba0bfe0e753812058436eaf515391eb1f4b09951 Gerrit-Change-Number: 28576 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-Comment-Date: Tue, 12 Jul 2022 13:55:29 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

1 0

Change in osmo-ci[master]: jobs/coverity: don't cd ~/osmo-ci/coverity
by pespin 12 Jul '22

12 Jul '22

Attention is currently required from: osmith. pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ci/+/28579 ) Change subject: jobs/coverity: don't cd ~/osmo-ci/coverity ...................................................................... Patch Set 1: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/28579 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-ci Gerrit-Branch: master Gerrit-Change-Id: Ie992d71d2afec6982447b8fe3626c821a9e48d19 Gerrit-Change-Number: 28579 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-Comment-Date: Tue, 12 Jul 2022 13:54:36 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

1 0

Change in osmo-ci[master]: Revert "WIP: coverity: Make sure curl is installed, as our coverity u...
by pespin 12 Jul '22

12 Jul '22

Attention is currently required from: osmith. pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ci/+/28577 ) Change subject: Revert "WIP: coverity: Make sure curl is installed, as our coverity upload job needs it" ...................................................................... Patch Set 1: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/28577 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-ci Gerrit-Branch: master Gerrit-Change-Id: I5d4709c0019ef4bae6ccbd11c28eeb23d75692c1 Gerrit-Change-Number: 28577 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-Comment-Date: Tue, 12 Jul 2022 13:54:07 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

1 0

2025

2024

2023

2022

gerrit-log July 2022