The Puppet official style guide outlines a number of style conventions for Puppet code, some of which we've touched on in the preceding section. For example, according to the style guide, manifests:
- Must use two-space soft tabs
- Must not use literal tab characters
- Must not contain trailing white space
- Should not exceed an 80-character line width
- Should align parameter arrows (
=>
) within blocks
Following the style guide will make sure that your Puppet code is easy to read and maintain, and if you're planning to release your code to the public, style compliance is essential.
The puppet-lint
tool will automatically check your code against the style guide. The next section explains how to use it.
Here's what you need to do to install puppet-lint
:
- We'll install Puppet-lint using the gem provider because the gem version is much more up to date than the APT or RPM packages available. Create a
puppet-lint.pp
manifest as shown in the following code snippet:
package {'puppet-lint': ensure => 'installed', provider => 'gem' }
- Run
puppet apply
on thepuppet-lint.pp
manifest, as shown in the following command:
t@cookbook:manifests$ puppet apply puppet-lint.pp Notice: Compiled catalog for cookbook.example.com in environment production in 1.04 seconds Notice: /Stage[main]/Main/Package[puppet-lint]/ensure: created Notice: Applied catalog in 0.93 seconds
Follow these steps to use Puppet-lint:
t@cookbook:manifests$ puppet-lint puppet-lint.pp WARNING: indentation of => is not properly aligned (expected in column 12, but found it in column 10) on line 2 ERROR: trailing whitespace found on line 4
- As you can see,
Puppet-lint
found a number of problems with the manifest file. Correct the errors, save the file, and rerunPuppet-lint
to check that all is well. If successful, you'll see no output:
t@cookbook:manifests$ puppet-lint puppet-lint.pp t@cookbook:manifests$
Should you follow Puppet style guide and, by extension, keep your code lint-clean? It's up to you, but here are a couple of things to think about:
- It makes sense to use some style conventions, especially when you're working collaboratively on code. Unless you and your colleagues can agree on standards for whitespace, tabs, quoting, alignment, and so on, your code will be messy and difficult to read or maintain.
- If you're choosing a set of style conventions to follow, the logical choice would be those issued by Puppet and adopted by the community for use in public modules.
Having said that, it's possible to tell Puppet-lint
to ignore certain checks if you've chosen not to adopt them in your code base. For example, if you don't want puppet-lint
to warn you about code lines exceeding 80 characters, you can run puppet-lint
with the following option:
puppet-lint --no-80chars-check
Most developers have terminals with more than 80 characters now; the check for 80 characters is generally disabled in favor of a new 140-character limit. You may disable the 140 character check with the following:
puppet-lint --no-140chars-check
Run puppet-lint --help
to see the complete list of check configuration commands.
You can find out more about
Puppet-lint
athttps://github.com/rodjek/puppet-lint.The Automatic syntax checking with Git hooks recipe in Chapter 2, Puppet Infrastructure
- The Testing your manifests with rspec-puppet recipe in Chapter 9, External Tools and the Puppet Ecosystem