diff options
| author |   agardner <agardner@linuxfoundation.org> | 2018-01-26 10:32:04 -0500 |
|---|---|---|
| committer | agardner <agardner@linuxfoundation.org> | 2018-01-26 10:44:22 -0500 |
| commit | 92e57bfbd6e426c733e4687868b7637327c34958 (patch) | |
| tree | 4434757c083f8021053a746d24203ed56dc7f174 | |
| parent | 85add8399995e3d29eb5c65f08dd1ffd46673008 (diff) | |
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 <agardner@linuxfoundation.org>
| -rw-r--r-- | jjb/global/releng-macros.yml | 6 | ||||
| -rw-r--r-- | jjb/releng/opnfv-lint.yml | 14 |
2 files changed, 17 insertions, 3 deletions
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 |