Linting should be done on a build server, not on master 07/51207/3
authoragardner <agardner@linuxfoundation.org>
Fri, 26 Jan 2018 15:32:04 +0000 (10:32 -0500)
committeragardner <agardner@linuxfoundation.org>
Fri, 26 Jan 2018 15:44:22 +0000 (10:44 -0500)
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>
jjb/global/releng-macros.yml
jjb/releng/opnfv-lint.yml

index 25e4e4a..1c74732 100644 (file)
     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)
     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)
index c0da8ae..8cf6474 100644 (file)
       - 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
       - 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