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

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

by pespin

Attention is currently required from: osmith. pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/docker-playground/+/29729 ) Change subject: debian-buster-build: add LLVMEmbeddedToolchainForArm ...................................................................... Patch Set 2: Code-Review+1 -- 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: 2 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: Thu, 13 Oct 2022 11:18:06 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

2 years, 8 months

1
0
0 0

Change in osmo-ci[master]: ansible/ttcn3-slave: fix adding apt key

by pespin

Attention is currently required from: osmith. pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ci/+/29708 ) Change subject: ansible/ttcn3-slave: fix adding apt key ...................................................................... Patch Set 3: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/29708 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-ci Gerrit-Branch: master Gerrit-Change-Id: Idb23231d629ba8a75510deace0d1f130e889d535 Gerrit-Change-Number: 29708 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: osmith <osmith(a)sysmocom.de> Gerrit-Comment-Date: Thu, 13 Oct 2022 11:17:17 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

2 years, 8 months

1
0
0 0

Change in osmo-ci[master]: ansible/hosts: add build2-deb11build-ansible

by pespin

Attention is currently required from: osmith. pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ci/+/29707 ) Change subject: ansible/hosts: add build2-deb11build-ansible ...................................................................... Patch Set 3: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/29707 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-ci Gerrit-Branch: master Gerrit-Change-Id: Ibe8e2a638fcbc7c0a69c211385a870b1a82b6fef Gerrit-Change-Number: 29707 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: osmith <osmith(a)sysmocom.de> Gerrit-Comment-Date: Thu, 13 Oct 2022 11:16:18 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

2 years, 8 months

1
0
0 0

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

by osmith

Attention is currently required from: pespin. osmith has posted comments on this change. ( https://gerrit.osmocom.org/c/docker-playground/+/29729 ) Change subject: debian-buster-build: add LLVMEmbeddedToolchainForArm ...................................................................... Patch Set 2: (1 comment) Commit Message: https://gerrit.osmocom.org/c/docker-playground/+/29729/comment/f4507485_62f… PS1, Line 7: debian-buster-build: add LLVMEmbeddedToolchainForArm > WhyDoWeNeedThat? Done -- 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: 2 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-CC: pespin <pespin(a)sysmocom.de> Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Comment-Date: Thu, 13 Oct 2022 11:12:04 +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 docker-playground[master]: debian-buster-build: add LLVMEmbeddedToolchainForArm

by osmith

Attention is currently required from: osmith. Hello Jenkins Builder, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/c/docker-playground/+/29729 to look at the new patch set (#2). Change subject: debian-buster-build: add LLVMEmbeddedToolchainForArm ...................................................................... debian-buster-build: 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: 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/2 -- 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: 2 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-CC: 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-ci[master]: jobs/gerrit-verifications: link to new wiki page

by osmith

osmith has submitted this change. ( https://gerrit.osmocom.org/c/osmo-ci/+/29690 ) Change subject: jobs/gerrit-verifications: link to new wiki page ...................................................................... jobs/gerrit-verifications: link to new wiki page Related: OS#2385 Change-Id: Ia4eac0c39fd156db1fc30e86ef55c48c6105912f --- M jobs/gerrit-verifications.yml 1 file changed, 3 insertions(+), 0 deletions(-) Approvals: Jenkins Builder: Verified msuraev: Looks good to me, approved pespin: Looks good to me, but someone else must approve diff --git a/jobs/gerrit-verifications.yml b/jobs/gerrit-verifications.yml index 86ed663..9e65c40 100644 --- a/jobs/gerrit-verifications.yml +++ b/jobs/gerrit-verifications.yml @@ -1,6 +1,9 @@ # This file holds all gerrit build verifications https://jenkins.osmocom.org/jenkins/view/Jenkins-Gerrit/. # One can simply add a gerrit job by adding project's repository to repos list. # +# Overview of jobs involved in the pipeline: +# https://osmocom.org/projects/osmocom-servers/wiki/Jenkins_gerrit-verificati… +# # How it works: # * Two jobs get added for each repository: # gerrit-{repos} 3 is the latest approved patch-set. No files were changed between the latest approved patch-set and the submitted one. -- To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/29690 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-ci Gerrit-Branch: master Gerrit-Change-Id: Ia4eac0c39fd156db1fc30e86ef55c48c6105912f Gerrit-Change-Number: 29690 Gerrit-PatchSet: 4 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 osmo-ci[master]: jobs/gerrit-verifications: set pipeline concurrent

by osmith

osmith has submitted this change. ( https://gerrit.osmocom.org/c/osmo-ci/+/29706 ) Change subject: jobs/gerrit-verifications: set pipeline concurrent ...................................................................... jobs/gerrit-verifications: set pipeline concurrent Allow running multiple gerrit-verification pipelines of the same project at the same time to use the jenkins nodes more efficiently. Note that the concurrent setting of the build jobs are not changed. Some projects have this set to false, and for those, there will be no more than one build job running at the same time. The build job gets started from the pipeline, along with other jobs like lint and deb/rpm package building. Related: OS#2385 Change-Id: Ie2a19013e73d96a937e8f83c815c6a82cfe0768d --- M jobs/gerrit-verifications.yml 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Jenkins Builder: Verified pespin: Looks good to me, but someone else must approve osmith: Looks good to me, approved msuraev: Looks good to me, but someone else must approve diff --git a/jobs/gerrit-verifications.yml b/jobs/gerrit-verifications.yml index 9e65c40..36646f8 100644 --- a/jobs/gerrit-verifications.yml +++ b/jobs/gerrit-verifications.yml @@ -359,6 +359,7 @@ - job-template: name: 'gerrit-{repos}' project-type: pipeline + concurrent: true parameters: - bool: name: PIPELINE_BUILD -- To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/29706 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-ci Gerrit-Branch: master Gerrit-Change-Id: Ie2a19013e73d96a937e8f83c815c6a82cfe0768d Gerrit-Change-Number: 29706 Gerrit-PatchSet: 4 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 osmo-ci[master]: jobs/gerrit-verifications: add osmo-ci

by osmith

osmith has submitted this change. ( https://gerrit.osmocom.org/c/osmo-ci/+/29689 ) Change subject: jobs/gerrit-verifications: add osmo-ci ...................................................................... jobs/gerrit-verifications: add osmo-ci Run the linter again on osmo-ci. It used to run on osmo-ci.git with gerrit-lint.yml before recent refactoring in I5f1340248cd116c5fa5134eee3fae50baf7ef851. Adding it to gerrit-verifications like this makes it run again, pipeline_lint: true is the default. Related: OS#2385 Change-Id: Ie6d8a99647846fb4008953b444bba642cbcf3b1e --- M jobs/gerrit-verifications.yml 1 file changed, 4 insertions(+), 0 deletions(-) Approvals: Jenkins Builder: Verified osmith: Looks good to me, approved pespin: Looks good to me, but someone else must approve msuraev: Looks good to me, but someone else must approve diff --git a/jobs/gerrit-verifications.yml b/jobs/gerrit-verifications.yml index f472752..86ed663 100644 --- a/jobs/gerrit-verifications.yml +++ b/jobs/gerrit-verifications.yml @@ -166,6 +166,10 @@ cmd: 'docker pull {docker_img} && {docker_run} {docker_img} {timeout_cmd} /build/contrib/jenkins.sh' pipeline_binpkgs: false + - osmo-ci: + pipeline_build: false + pipeline_binpkgs: false + - osmo-ggsn: a1_name: GTP a1: !!python/tuple [--enable-gtp-linux,--disable-gtp-linux] -- To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/29689 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-ci Gerrit-Branch: master Gerrit-Change-Id: Ie6d8a99647846fb4008953b444bba642cbcf3b1e Gerrit-Change-Number: 29689 Gerrit-PatchSet: 4 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 osmo-ci[master]: jobs/gerrit-verifications*: drop unsupported args

by osmith

osmith has submitted this change. ( https://gerrit.osmocom.org/c/osmo-ci/+/29688 ) Change subject: jobs/gerrit-verifications*: drop unsupported args ...................................................................... jobs/gerrit-verifications*: drop unsupported args Fix warnings when updating the job with Jenkins Job Builder: WARNING:jenkins_jobs.modules.triggers:Gerrit Trigger property 'no-name-and-email' is deprecated WARNING:jenkins_jobs.modules.triggers:Gerrit Trigger property 'trigger-for-unreviewed-patches' is not supported in this plugin version Related: OS#2385 Change-Id: I6a0057738955bc2507cb5d8c4d48edf652c3529f --- M jobs/gerrit-verifications-dahdi.yml M jobs/gerrit-verifications.yml 2 files changed, 0 insertions(+), 4 deletions(-) Approvals: Jenkins Builder: Verified osmith: Looks good to me, approved pespin: Looks good to me, but someone else must approve msuraev: Looks good to me, but someone else must approve diff --git a/jobs/gerrit-verifications-dahdi.yml b/jobs/gerrit-verifications-dahdi.yml index 51bfb04..a117ff3 100644 --- a/jobs/gerrit-verifications-dahdi.yml +++ b/jobs/gerrit-verifications-dahdi.yml @@ -76,8 +76,6 @@ notbuilt: false silent: false escape-quotes: false - no-name-and-email: false - trigger-for-unreviewed-patches: true server-name: gerrit.osmocom.org builders: diff --git a/jobs/gerrit-verifications.yml b/jobs/gerrit-verifications.yml index 8331096..f472752 100644 --- a/jobs/gerrit-verifications.yml +++ b/jobs/gerrit-verifications.yml @@ -534,8 +534,6 @@ notbuilt: false silent: true # comment + vote is done in gerrit-pipeline-result.yml escape-quotes: false - no-name-and-email: false - trigger-for-unreviewed-patches: true server-name: gerrit.osmocom.org - job-template: -- To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/29688 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-ci Gerrit-Branch: master Gerrit-Change-Id: I6a0057738955bc2507cb5d8c4d48edf652c3529f Gerrit-Change-Number: 29688 Gerrit-PatchSet: 4 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 osmo-ci[master]: jenkins-gerrit: tweak notify setting

by osmith

osmith has submitted this change. ( https://gerrit.osmocom.org/c/osmo-ci/+/29702 ) Change subject: jenkins-gerrit: tweak notify setting ...................................................................... jenkins-gerrit: tweak notify setting Instead of notifying everybody listed as reviewers and in CC: * notify the owner if the build failed * notify nobody if the build was successful Related: OS#2385 Change-Id: I7c6c183b98624eb75d5dccd9766ee3ff5568b06e --- M scripts/jenkins-gerrit/pipeline_summary.py 1 file changed, 7 insertions(+), 2 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 37d4028..b5310c6 100755 --- a/scripts/jenkins-gerrit/pipeline_summary.py +++ b/scripts/jenkins-gerrit/pipeline_summary.py @@ -137,7 +137,8 @@ :returns: a dict that is expected by gerrit's set-review api, e.g. {"tag": "jenkins", "message": "...", - "labels": {"Code-Review": -1}} """ + "labels": {"Code-Review": -1}, + "notify": "OWNER"} """ summary = "" pipeline = parse_pipeline(build_url) @@ -164,15 +165,19 @@ if jobs["failed"]: summary += "Build Failed\n" vote = -1 + notify = "OWNER" else: summary += "Build Successful\n" vote = 1 + notify = "NONE" # Reference: # https://gerrit-review.googlesource.com/Documentation/rest-api-changes.html#… + # https://gerrit-review.googlesource.com/Documentation/rest-api-changes.html#… return {"tag": "jenkins", "message": summary, - "labels": {"Verified": vote}} + "labels": {"Verified": vote}, + "notify": notify} def main(): -- 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-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-CC: pespin <pespin(a)sysmocom.de> Gerrit-MessageType: merged

2 years, 8 months

1
0
0 0

2025

2024

2023

2022

gerrit-log October 2022