opendev/system-config
Code Issues Proposed changes

Add puppet-lint & syntax checks to puppet modules

Browse Source 
Add Puppet syntax checks to catch any errors that would have
otherwise not been discovered by missing rspec tests. Syntax checks
also ensure that ERB templates are syntactically correct.

Add Puppet lint checks to improve quality and uniformity.
Puppet lint jobs are marked as non-voting. It would otherwise
block all changes from being merged until we fix every lint errors
which isn't possible in the short-term. Puppet lint will be
an ongoing work for the following weeks and months.

Change-Id: I991f6c1179113dd241dab4679a27a2aa42936c66
Reviewed-on: https://review.openstack.org/32663
Reviewed-by: James E. Blair <corvus@inaugust.com>
Reviewed-by: Clark Boylan <clark.boylan@gmail.com>
Reviewed-by: Jeremy Stanley <fungi@yuggoth.org>
Reviewed-by: Elizabeth Krumbach Joseph <lyz@princessleia.com>
Approved: Clark Boylan <clark.boylan@gmail.com>
Tested-by: Jenkins
This commit is contained in:
Mathieu Gagné 2013-06-11 20:36:19 -04:00 committed by Jenkins
parent 6153f389e9
commit c7ebbbed16
2 changed files with 62 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
modules/openstack_project/files/jenkins_job_builder/config/projects.yaml
View File

@ -876,6 +876,8 @@
tarball-site: tarballs.openstack.org tarball-site: tarballs.openstack.org
jobs: jobs:
- gate-{name}-puppet-lint
- gate-{name}-puppet-syntax
- puppet-module-unit - puppet-module-unit
- project: - project:
@ -885,6 +887,8 @@
tarball-site: tarballs.openstack.org tarball-site: tarballs.openstack.org
jobs: jobs:
- gate-{name}-puppet-lint
- gate-{name}-puppet-syntax
- puppet-module-unit - puppet-module-unit
- project: - project:
@ -894,6 +898,8 @@
tarball-site: tarballs.openstack.org tarball-site: tarballs.openstack.org
jobs: jobs:
- gate-{name}-puppet-lint
- gate-{name}-puppet-syntax
- puppet-module-unit - puppet-module-unit
- project: - project:
@ -903,6 +909,8 @@
tarball-site: tarballs.openstack.org tarball-site: tarballs.openstack.org
jobs: jobs:
- gate-{name}-puppet-lint
- gate-{name}-puppet-syntax
- puppet-module-unit - puppet-module-unit
- project: - project:
@ -912,6 +920,8 @@
tarball-site: tarballs.openstack.org tarball-site: tarballs.openstack.org
jobs: jobs:
- gate-{name}-puppet-lint
- gate-{name}-puppet-syntax
- puppet-module-unit - puppet-module-unit
- project: - project:
@ -921,6 +931,8 @@
tarball-site: tarballs.openstack.org tarball-site: tarballs.openstack.org
jobs: jobs:
- gate-{name}-puppet-lint
- gate-{name}-puppet-syntax
- puppet-module-unit - puppet-module-unit
- project: - project:
@ -930,6 +942,8 @@
tarball-site: tarballs.openstack.org tarball-site: tarballs.openstack.org
jobs: jobs:
- gate-{name}-puppet-lint
- gate-{name}-puppet-syntax
- puppet-module-unit - puppet-module-unit
- project: - project:
@ -939,6 +953,8 @@
tarball-site: tarballs.openstack.org tarball-site: tarballs.openstack.org
jobs: jobs:
- gate-{name}-puppet-lint
- gate-{name}-puppet-syntax
- puppet-module-unit - puppet-module-unit
- project: - project:
@ -948,6 +964,8 @@
tarball-site: tarballs.openstack.org tarball-site: tarballs.openstack.org
jobs: jobs:
- gate-{name}-puppet-lint
- gate-{name}-puppet-syntax
- puppet-module-unit - puppet-module-unit
- project: - project:
@ -957,6 +975,8 @@
tarball-site: tarballs.openstack.org tarball-site: tarballs.openstack.org
jobs: jobs:
- gate-{name}-puppet-lint
- gate-{name}-puppet-syntax
- puppet-module-unit - puppet-module-unit
- project: - project:

42
modules/openstack_project/files/zuul/layout.yaml
View File

@ -123,6 +123,8 @@ jobs:
voting: false voting: false
- name: gate-savanna-pylint - name: gate-savanna-pylint
voting: false voting: false
- name: ^gate-puppet-.*-puppet-lint$
voting: false
projects: projects:
@ -1633,100 +1635,140 @@ projects:
- name: stackforge/puppet-ceilometer - name: stackforge/puppet-ceilometer
check: check:
- gate-puppet-ceilometer-puppet-lint
- gate-puppet-ceilometer-puppet-syntax
- gate-puppet-ceilometer-puppet-unit-2.7 - gate-puppet-ceilometer-puppet-unit-2.7
- gate-puppet-ceilometer-puppet-unit-3.0 - gate-puppet-ceilometer-puppet-unit-3.0
- gate-puppet-ceilometer-puppet-unit-3.1 - gate-puppet-ceilometer-puppet-unit-3.1
gate: gate:
- gate-puppet-ceilometer-puppet-lint
- gate-puppet-ceilometer-puppet-syntax
- gate-puppet-ceilometer-puppet-unit-2.7 - gate-puppet-ceilometer-puppet-unit-2.7
- gate-puppet-ceilometer-puppet-unit-3.0 - gate-puppet-ceilometer-puppet-unit-3.0
- gate-puppet-ceilometer-puppet-unit-3.1 - gate-puppet-ceilometer-puppet-unit-3.1
- name: stackforge/puppet-cinder - name: stackforge/puppet-cinder
check: check:
- gate-puppet-cinder-puppet-lint
- gate-puppet-cinder-puppet-syntax
- gate-puppet-cinder-puppet-unit-2.7 - gate-puppet-cinder-puppet-unit-2.7
- gate-puppet-cinder-puppet-unit-3.0 - gate-puppet-cinder-puppet-unit-3.0
- gate-puppet-cinder-puppet-unit-3.1 - gate-puppet-cinder-puppet-unit-3.1
gate: gate:
- gate-puppet-cinder-puppet-lint
- gate-puppet-cinder-puppet-syntax
- gate-puppet-cinder-puppet-unit-2.7 - gate-puppet-cinder-puppet-unit-2.7
- gate-puppet-cinder-puppet-unit-3.0 - gate-puppet-cinder-puppet-unit-3.0
- gate-puppet-cinder-puppet-unit-3.1 - gate-puppet-cinder-puppet-unit-3.1
- name: stackforge/puppet-glance - name: stackforge/puppet-glance
check: check:
- gate-puppet-glance-puppet-lint
- gate-puppet-glance-puppet-syntax
- gate-puppet-glance-puppet-unit-2.7 - gate-puppet-glance-puppet-unit-2.7
- gate-puppet-glance-puppet-unit-3.0 - gate-puppet-glance-puppet-unit-3.0
- gate-puppet-glance-puppet-unit-3.1 - gate-puppet-glance-puppet-unit-3.1
gate: gate:
- gate-puppet-glance-puppet-lint
- gate-puppet-glance-puppet-syntax
- gate-puppet-glance-puppet-unit-2.7 - gate-puppet-glance-puppet-unit-2.7
- gate-puppet-glance-puppet-unit-3.0 - gate-puppet-glance-puppet-unit-3.0
- gate-puppet-glance-puppet-unit-3.1 - gate-puppet-glance-puppet-unit-3.1
- name: stackforge/puppet-horizon - name: stackforge/puppet-horizon
check: check:
- gate-puppet-horizon-puppet-lint
- gate-puppet-horizon-puppet-syntax
- gate-puppet-horizon-puppet-unit-2.7 - gate-puppet-horizon-puppet-unit-2.7
- gate-puppet-horizon-puppet-unit-3.0 - gate-puppet-horizon-puppet-unit-3.0
- gate-puppet-horizon-puppet-unit-3.1 - gate-puppet-horizon-puppet-unit-3.1
gate: gate:
- gate-puppet-horizon-puppet-lint
- gate-puppet-horizon-puppet-syntax
- gate-puppet-horizon-puppet-unit-2.7 - gate-puppet-horizon-puppet-unit-2.7
- gate-puppet-horizon-puppet-unit-3.0 - gate-puppet-horizon-puppet-unit-3.0
- gate-puppet-horizon-puppet-unit-3.1 - gate-puppet-horizon-puppet-unit-3.1
- name: stackforge/puppet-keystone - name: stackforge/puppet-keystone
check: check:
- gate-puppet-keystone-puppet-lint
- gate-puppet-keystone-puppet-syntax
- gate-puppet-keystone-puppet-unit-2.7 - gate-puppet-keystone-puppet-unit-2.7
- gate-puppet-keystone-puppet-unit-3.0 - gate-puppet-keystone-puppet-unit-3.0
- gate-puppet-keystone-puppet-unit-3.1 - gate-puppet-keystone-puppet-unit-3.1
gate: gate:
- gate-puppet-keystone-puppet-lint
- gate-puppet-keystone-puppet-syntax
- gate-puppet-keystone-puppet-unit-2.7 - gate-puppet-keystone-puppet-unit-2.7
- gate-puppet-keystone-puppet-unit-3.0 - gate-puppet-keystone-puppet-unit-3.0
- gate-puppet-keystone-puppet-unit-3.1 - gate-puppet-keystone-puppet-unit-3.1
- name: stackforge/puppet-nova - name: stackforge/puppet-nova
check: check:
- gate-puppet-nova-puppet-lint
- gate-puppet-nova-puppet-syntax
- gate-puppet-nova-puppet-unit-2.7 - gate-puppet-nova-puppet-unit-2.7
- gate-puppet-nova-puppet-unit-3.0 - gate-puppet-nova-puppet-unit-3.0
- gate-puppet-nova-puppet-unit-3.1 - gate-puppet-nova-puppet-unit-3.1
gate: gate:
- gate-puppet-nova-puppet-lint
- gate-puppet-nova-puppet-syntax
- gate-puppet-nova-puppet-unit-2.7 - gate-puppet-nova-puppet-unit-2.7
- gate-puppet-nova-puppet-unit-3.0 - gate-puppet-nova-puppet-unit-3.0
- gate-puppet-nova-puppet-unit-3.1 - gate-puppet-nova-puppet-unit-3.1
- name: stackforge/puppet-quantum - name: stackforge/puppet-quantum
check: check:
- gate-puppet-quantum-puppet-lint
- gate-puppet-quantum-puppet-syntax
- gate-puppet-quantum-puppet-unit-2.7 - gate-puppet-quantum-puppet-unit-2.7
- gate-puppet-quantum-puppet-unit-3.0 - gate-puppet-quantum-puppet-unit-3.0
- gate-puppet-quantum-puppet-unit-3.1 - gate-puppet-quantum-puppet-unit-3.1
gate: gate:
- gate-puppet-quantum-puppet-lint
- gate-puppet-quantum-puppet-syntax
- gate-puppet-quantum-puppet-unit-2.7 - gate-puppet-quantum-puppet-unit-2.7
- gate-puppet-quantum-puppet-unit-3.0 - gate-puppet-quantum-puppet-unit-3.0
- gate-puppet-quantum-puppet-unit-3.1 - gate-puppet-quantum-puppet-unit-3.1
- name: stackforge/puppet-openstack - name: stackforge/puppet-openstack
check: check:
- gate-puppet-openstack-puppet-lint
- gate-puppet-openstack-puppet-syntax
- gate-puppet-openstack-puppet-unit-2.7 - gate-puppet-openstack-puppet-unit-2.7
- gate-puppet-openstack-puppet-unit-3.0 - gate-puppet-openstack-puppet-unit-3.0
- gate-puppet-openstack-puppet-unit-3.1 - gate-puppet-openstack-puppet-unit-3.1
gate: gate:
- gate-puppet-openstack-puppet-lint
- gate-puppet-openstack-puppet-syntax
- gate-puppet-openstack-puppet-unit-2.7 - gate-puppet-openstack-puppet-unit-2.7
- gate-puppet-openstack-puppet-unit-3.0 - gate-puppet-openstack-puppet-unit-3.0
- gate-puppet-openstack-puppet-unit-3.1 - gate-puppet-openstack-puppet-unit-3.1
- name: stackforge/puppet-swift - name: stackforge/puppet-swift
check: check:
- gate-puppet-swift-puppet-lint
- gate-puppet-swift-puppet-syntax
- gate-puppet-swift-puppet-unit-2.7 - gate-puppet-swift-puppet-unit-2.7
- gate-puppet-swift-puppet-unit-3.0 - gate-puppet-swift-puppet-unit-3.0
- gate-puppet-swift-puppet-unit-3.1 - gate-puppet-swift-puppet-unit-3.1
gate: gate:
- gate-puppet-swift-puppet-lint
- gate-puppet-swift-puppet-syntax
- gate-puppet-swift-puppet-unit-2.7 - gate-puppet-swift-puppet-unit-2.7
- gate-puppet-swift-puppet-unit-3.0 - gate-puppet-swift-puppet-unit-3.0
- gate-puppet-swift-puppet-unit-3.1 - gate-puppet-swift-puppet-unit-3.1
- name: stackforge/puppet-tempest - name: stackforge/puppet-tempest
check: check:
- gate-puppet-tempest-puppet-lint
- gate-puppet-tempest-puppet-syntax
- gate-puppet-tempest-puppet-unit-2.7 - gate-puppet-tempest-puppet-unit-2.7
- gate-puppet-tempest-puppet-unit-3.0 - gate-puppet-tempest-puppet-unit-3.0
- gate-puppet-tempest-puppet-unit-3.1 - gate-puppet-tempest-puppet-unit-3.1
gate: gate:
- gate-puppet-tempest-puppet-lint
- gate-puppet-tempest-puppet-syntax
- gate-puppet-tempest-puppet-unit-2.7 - gate-puppet-tempest-puppet-unit-2.7
- gate-puppet-tempest-puppet-unit-3.0 - gate-puppet-tempest-puppet-unit-3.0
- gate-puppet-tempest-puppet-unit-3.1 - gate-puppet-tempest-puppet-unit-3.1