aboutsummaryrefslogtreecommitdiffstats
path: root/jobs
diff options
context:
space:
mode:
authorOliver Smith <osmith@sysmocom.de>2023-12-04 08:46:02 +0100
committerOliver Smith <osmith@sysmocom.de>2023-12-04 08:55:28 +0100
commitc4ee740d57d283ae76dd1f3ae9497c9e926642a0 (patch)
tree00b80ea109fd85c64f8f7dc04f0c4eac5ccdf25d /jobs
parent2031b3d747d8d1bb952a6d565aea104a6d1ae315 (diff)
jobs: fix undefined parameter log warnings
Adjust all jobs in the gerrit verifications pipeline to use the same parameters from a new include file, to prevent undefined parameter warnings in the log. Fixes: OS#6261 Change-Id: Iadc5cd8996eb4ed86634ceb35829a3e9239e598d
Diffstat (limited to 'jobs')
-rw-r--r--jobs/gerrit-binpkgs.yml21
-rw-r--r--jobs/gerrit-lint.yml33
-rw-r--r--jobs/gerrit-verifications-comment.yml39
-rw-r--r--jobs/gerrit-verifications-parameters.yaml.inc53
-rw-r--r--jobs/gerrit-verifications.yml10
5 files changed, 60 insertions, 96 deletions
diff --git a/jobs/gerrit-binpkgs.yml b/jobs/gerrit-binpkgs.yml
index ecfbd52..7cf2230 100644
--- a/jobs/gerrit-binpkgs.yml
+++ b/jobs/gerrit-binpkgs.yml
@@ -24,26 +24,7 @@
Related issue: <a href="https://osmocom.org/issues/2385">OS#2385</a>
parameters:
- - string:
- name: BRANCH_CI
- description: |
- osmo-ci.git branch
- default: 'master'
- - string:
- name: GERRIT_REPO_URL
- description: set by gerrit verification pipeline job
- - string:
- name: GERRIT_BRANCH
- description: set by gerrit verification pipeline job
- - string:
- name: GERRIT_REFSPEC
- description: set by gerrit verification pipeline job
- - string:
- name: PROJECT_NAME
- description: project to test, e.g. 'libosmocore' (set by pipeline)
- - string:
- name: DISTRO
- description: distribution to build for, e.g. 'debian:12' (set by pipeline)
+ !include: gerrit-verifications-parameters.yaml.inc
scm:
- git:
diff --git a/jobs/gerrit-lint.yml b/jobs/gerrit-lint.yml
index 16377cd..3039829 100644
--- a/jobs/gerrit-lint.yml
+++ b/jobs/gerrit-lint.yml
@@ -23,38 +23,7 @@
Related issue: <a href="https://osmocom.org/issues/5087">OS#5087</a>
parameters:
- - string:
- name: BRANCH_CI
- description: |
- osmo-ci.git branch
- default: 'master'
- - string:
- name: GERRIT_PROJECT
- description: set by gerrit verification pipeline job
- - string:
- name: GERRIT_CHANGE_NUMBER
- description: set by gerrit verification pipeline job
- - string:
- name: GERRIT_PATCHSET_NUMBER
- description: set by gerrit verification pipeline job
- - string:
- name: GERRIT_BRANCH
- description: set by gerrit verification pipeline job
- - string:
- name: GERRIT_HOST
- description: set by gerrit verification pipeline job
- - string:
- name: GERRIT_PATCHSET_REVISION
- description: set by gerrit verification pipeline job
- - string:
- name: GERRIT_PORT
- description: set by gerrit verification pipeline job
- - string:
- name: GERRIT_REFSPEC
- description: set by gerrit verification pipeline job
- - string:
- name: GERRIT_REPO_URL
- description: set by gerrit verification pipeline job
+ !include: gerrit-verifications-parameters.yaml.inc
scm:
- git:
diff --git a/jobs/gerrit-verifications-comment.yml b/jobs/gerrit-verifications-comment.yml
index 6c8d90d..3ed54ad 100644
--- a/jobs/gerrit-verifications-comment.yml
+++ b/jobs/gerrit-verifications-comment.yml
@@ -22,44 +22,7 @@
Related issue: <a href="https://osmocom.org/issues/2385">OS#2385</a>
parameters:
- - string:
- name: BRANCH_CI
- description: |
- osmo-ci.git branch
- default: 'master'
- - string:
- name: COMMENT_TYPE
- description: set by gerrit verification pipeline job
- - string:
- name: GERRIT_PROJECT
- description: set by gerrit verification pipeline job
- - string:
- name: GERRIT_CHANGE_NUMBER
- description: set by gerrit verification pipeline job
- - string:
- name: GERRIT_PATCHSET_NUMBER
- description: set by gerrit verification pipeline job
- - string:
- name: GERRIT_BRANCH
- description: set by gerrit verification pipeline job
- - string:
- name: GERRIT_HOST
- description: set by gerrit verification pipeline job
- - string:
- name: GERRIT_PATCHSET_REVISION
- description: set by gerrit verification pipeline job
- - string:
- name: GERRIT_PATCHSET_UPLOADER_NAME
- description: set by gerrit verification pipeline job
- - string:
- name: GERRIT_PORT
- description: set by gerrit verification pipeline job
- - string:
- name: GERRIT_REFSPEC
- description: set by gerrit verification pipeline job
- - string:
- name: PIPELINE_BUILD_URL
- description: set by gerrit verification pipeline job
+ !include: gerrit-verifications-parameters.yaml.inc
scm:
- git:
diff --git a/jobs/gerrit-verifications-parameters.yaml.inc b/jobs/gerrit-verifications-parameters.yaml.inc
new file mode 100644
index 0000000..32df1b6
--- /dev/null
+++ b/jobs/gerrit-verifications-parameters.yaml.inc
@@ -0,0 +1,53 @@
+# This file gets included as parameters element of various jobs in the
+# gerrit-verifications pipeline. All of these jobs have the same parameters
+# and can be called from one run_job() function, without creating log entries
+# about undefined parameters (OS#6261). See gerrit-verifications.yml for more
+# information.
+
+- string:
+ name: BRANCH_CI
+ description: |
+ osmo-ci.git branch
+ default: 'master'
+- string:
+ name: COMMENT_TYPE
+ description: set by gerrit verification pipeline job
+- string:
+ name: DISTRO
+ description: set by gerrit verification pipeline job
+- string:
+ name: GERRIT_BRANCH
+ description: set by gerrit verification pipeline job
+- string:
+ name: GERRIT_CHANGE_NUMBER
+ description: set by gerrit verification pipeline job
+- string:
+ name: GERRIT_HOST
+ description: set by gerrit verification pipeline job
+- string:
+ name: GERRIT_PATCHSET_NUMBER
+ description: set by gerrit verification pipeline job
+- string:
+ name: GERRIT_PATCHSET_REVISION
+ description: set by gerrit verification pipeline job
+- string:
+ name: GERRIT_PATCHSET_UPLOADER_NAME
+ description: set by gerrit verification pipeline job
+- string:
+ name: GERRIT_PORT
+ description: set by gerrit verification pipeline job
+- string:
+ name: GERRIT_PROJECT
+ description: set by gerrit verification pipeline job
+- string:
+ name: GERRIT_REFSPEC
+ description: set by gerrit verification pipeline job
+- string:
+ name: GERRIT_REPO_URL
+ description: set by gerrit verification pipeline job
+- string:
+ name: PIPELINE_BUILD_URL
+ description: set by gerrit verification pipeline job
+- string:
+ name: PROJECT_NAME
+ description: set by gerrit verification pipeline job
diff --git a/jobs/gerrit-verifications.yml b/jobs/gerrit-verifications.yml
index 07f07f4..90c5fb5 100644
--- a/jobs/gerrit-verifications.yml
+++ b/jobs/gerrit-verifications.yml
@@ -473,6 +473,9 @@
try {{
echo "pipeline_${{display_name}}: running job ${{job_name}}"
+ // IMPORTANT: Keep gerrit-verifications-parameters.yaml.inc in sync
+ // with the list of parameters here, and make sure all jobs passed
+ // to run_job() include the .yaml.inc file. (OS#6261)
build job: job_name, parameters: [
string(name: "COMMENT_TYPE", value: comment_type),
string(name: "DISTRO", value: distro),
@@ -691,12 +694,7 @@
recursive: true
parameters:
- - string:
- name: GERRIT_BRANCH
- description: set by gerrit-{repos} job
- - string:
- name: GERRIT_REFSPEC
- description: set by gerrit-{repos} job
+ !include: gerrit-verifications-parameters.yaml.inc
builders:
- shell: '{obj:cmd}'