jenkins-bot has submitted this change and it was merged.

Change subject: Drop lenient puppet-lint job
......................................................................


Drop lenient puppet-lint job

All our repositories are nowadays passing puppet-lint and emit neither
errors nor warnings.  The lenient job was meant to accept warnings and
is no more needed.

Remove template {name}-puppetlint-lenient

Delete the sole job it realize: operations-puppet-puppetlint-lenient
last build was in October 2015.

Change-Id: I646dd2cf1e11a26c27ada4e7444b7960a70ad021
---
M jjb/job-templates.yaml
M jjb/operations-puppet.yaml
M tests/test_zuul_scheduler.py
M zuul/layout.yaml
4 files changed, 1 insertion(+), 24 deletions(-)

Approvals:
  Hashar: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/jjb/job-templates.yaml b/jjb/job-templates.yaml
index f1b4260..8a188b9 100644
--- a/jjb/job-templates.yaml
+++ b/jjb/job-templates.yaml
@@ -204,26 +204,6 @@
      - castor-save
 
 - job-template:
-    name: '{name}-puppetlint-lenient'
-    node: contintLabsSlave && UbuntuTrusty
-    defaults: use-remote-zuul-no-submodules
-    concurrent: true
-    triggers:
-     - zuul
-    builders:
-     - shell: |
-         rm -f puppet-lint.log
-         puppet-lint --version
-         set -o pipefail
-         puppet-lint \
-            --error-level error \
-            --log-format '%{{path}}:%{{line}} %{{KIND}} %{{message}} 
(%{{check}})' \
-            . | tee puppet-lint.log
-         set +o pipefail
-    publishers:
-     - puppet-lint
-
-- job-template:
     name: '{name}-puppetlint-strict'
     node: contintLabsSlave && UbuntuTrusty
     defaults: use-remote-zuul-no-submodules
diff --git a/jjb/operations-puppet.yaml b/jjb/operations-puppet.yaml
index 8bfdb4d..f5fa0d5 100644
--- a/jjb/operations-puppet.yaml
+++ b/jjb/operations-puppet.yaml
@@ -86,7 +86,6 @@
      - pep8
      - py27
     jobs:
-     - '{name}-puppetlint-lenient'
      - '{name}-puppetlint-strict'
      - '{name}-tox-{toxenv}-jessie'
      - '{name}-typos'
diff --git a/tests/test_zuul_scheduler.py b/tests/test_zuul_scheduler.py
index 6f4eff0..b9b643a 100644
--- a/tests/test_zuul_scheduler.py
+++ b/tests/test_zuul_scheduler.py
@@ -295,7 +295,7 @@
             '.*-(tabs|typos)',
             'operations-puppet-pep8',
             '.*-puppet-validate',
-            '.*-puppetlint-(strict|lenient)',
+            '.*-puppetlint-strict',
             '.*-whitespaces',
             'noop',
             'composer-validate',
diff --git a/zuul/layout.yaml b/zuul/layout.yaml
index d0d096e..2aacb08 100644
--- a/zuul/layout.yaml
+++ b/zuul/layout.yaml
@@ -2013,7 +2013,6 @@
     check:
       - erblint-HEAD
       - pplint-HEAD
-      #- operations-puppet-puppetlint-lenient
       - operations-puppet-puppetlint-strict
       - operations-puppet-pep8
       - operations-puppet-typos
@@ -2025,7 +2024,6 @@
       # Same as `check` pipeline:
       - erblint-HEAD
       - pplint-HEAD
-      #- operations-puppet-puppetlint-lenient
       - operations-puppet-puppetlint-strict
       - operations-puppet-pep8
       - operations-puppet-typos

-- 
To view, visit https://gerrit.wikimedia.org/r/288606
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I646dd2cf1e11a26c27ada4e7444b7960a70ad021
Gerrit-PatchSet: 1
Gerrit-Project: integration/config
Gerrit-Branch: master
Gerrit-Owner: Hashar <[email protected]>
Gerrit-Reviewer: Hashar <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to