From 92e57bfbd6e426c733e4687868b7637327c34958 Mon Sep 17 00:00:00 2001 From: agardner Date: Fri, 26 Jan 2018 10:32:04 -0500 Subject: Linting should be done on a build server, not on master Linting was broken by this patch https://gerrit.opnfv.org/gerrit/#/c/50737/ Also added sudo to pip installs, as they are not in a venv Patch forgot to put linting on build server so it defaulted to master. Change-Id: Iba8c1f633fad6d5f3d097f4911f45100c3b43e30 Signed-off-by: agardner --- jjb/global/releng-macros.yml | 6 +++--- jjb/releng/opnfv-lint.yml | 14 ++++++++++++++ 2 files changed, 17 insertions(+), 3 deletions(-) (limited to 'jjb') diff --git a/jjb/global/releng-macros.yml b/jjb/global/releng-macros.yml index 25e4e4a94..1c74732d5 100644 --- a/jjb/global/releng-macros.yml +++ b/jjb/global/releng-macros.yml @@ -345,7 +345,7 @@ builders: - shell: | # Install python packages - pip install "flake8==2.6.2" + sudo pip install "flake8==2.6.2" echo "Checking python code..." for f in $(egrep '\.py$' modified_files) @@ -361,8 +361,8 @@ name: lint-yaml-code builders: - shell: | - # Install python packages - pip install "yamllint==1.8.2" + # sudo Install python packages + sudo pip install "yamllint==1.8.2" echo "Checking yaml file..." for f in $(egrep '\.ya?ml$' modified_files) diff --git a/jjb/releng/opnfv-lint.yml b/jjb/releng/opnfv-lint.yml index c0da8ae51..8cf647439 100644 --- a/jjb/releng/opnfv-lint.yml +++ b/jjb/releng/opnfv-lint.yml @@ -35,6 +35,13 @@ - project-parameter: project: $GERRIT_PROJECT branch: '{branch}' + - node: + name: SLAVE_NAME + description: Slaves to execute yamllint + default-slaves: + - lf-build1 + allowed-multiselect: true + ignore-offline-nodes: true scm: - git-scm-gerrit @@ -80,6 +87,13 @@ - project-parameter: project: $GERRIT_PROJECT branch: '{branch}' + - node: + name: SLAVE_NAME + description: Slaves to execute yamllint + default-slaves: + - lf-build1 + allowed-multiselect: true + ignore-offline-nodes: true scm: - git-scm-gerrit -- cgit 1.2.3-korg