osmith has submitted this change. ( https://gerrit.osmocom.org/c/osmo-ci/+/29296 )
Change subject: jobs/gerrit-lint: add osmo-ci branch parameter ......................................................................
jobs/gerrit-lint: add osmo-ci branch parameter
Extend the jobs to clone the trusted osmo-ci.git repository from which we execute the linter next to the untrusted code from gerrit, and add a parameter for the osmo-ci branch.
Change-Id: Ic4134e6755014b3e5fbf6d3225e49c0ca5b27f9b --- M jobs/gerrit-lint.yml 1 file changed, 16 insertions(+), 1 deletion(-)
Approvals: Jenkins Builder: Verified fixeria: Looks good to me, but someone else must approve laforge: Looks good to me, approved
diff --git a/jobs/gerrit-lint.yml b/jobs/gerrit-lint.yml index f1fa389..6883f1e 100644 --- a/jobs/gerrit-lint.yml +++ b/jobs/gerrit-lint.yml @@ -107,8 +107,16 @@ </br></br> Related issue: <a href="https://osmocom.org/issues/5087">OS#5087</a>
+ parameters: + - string: + name: BRANCH_CI + description: | + osmo-ci.git branch + default: 'master' + scm: - git: + basedir: 'code-from-gerrit' url: '{obj:repos_url}' credentials-id: d5eda5e9-b59d-44ba-88d2-43473cb6e42d branches: @@ -120,6 +128,13 @@ skip-tag: true submodule: recursive: false + - git: + basedir: 'osmo-ci' + url: '{gerrit_url}/osmo-ci' + credentials-id: d5eda5e9-b59d-44ba-88d2-43473cb6e42d + branches: + - '$BRANCH_CI' + wipe-workspace: true
triggers: - gerrit: @@ -145,7 +160,7 @@ server-name: gerrit.osmocom.org
builders: - - shell: '~/osmo-ci/lint/lint_diff.sh HEAD~1' + - shell: 'cd code-from-gerrit && ../osmo-ci/lint/lint_diff.sh HEAD~1'
wrappers: - ansicolor: