官术网_书友最值得收藏!

Checking your manifests with puppet-lint

The Puppet Labs 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. Here's how to use it:

Getting ready

Here's what you need to do to install puppet-lint:

  1. Run the following command (we'll install puppet-lint as a gem, because that version is much more up-to-date than the APT package available in the Ubuntu Precise repo):
    ubuntu@cookbook:~/puppet$ sudo gem install puppet-lint 
     --no-ri --no-rdoc
    Successfully installed puppet-lint-0.3.2
    1 gem installed
    

How to do it...

Follow these steps to use puppet-lint:

  1. Choose a Puppet manifest file that you want to check with puppet-lint, and run the following command:
    ubuntu@cookbook:~/puppet$ puppet-lint modules/admin/manifests/ntp.pp
    WARNING: indentation of => is not properly aligned on line 9
    ERROR: trailing whitespace found on line 13
    WARNING: double quoted string containing no variables
     on line 3
    
  2. As you can see, puppet-lint found a number of problems with the manifest. Correct them, save the file, and rerun puppet-lint to check that all is well. If so, you'll see no output:
    ubuntu@cookbook:~/puppet$ puppet-lint 
     modules/admin/manifests/ntp.pp
    ubuntu@cookbook:~/puppet$
    

There's more...

You can find out more about puppet-lint at the website

http://puppet-lint.com/

The website lists each of the style checks in detail, and explains what the error messages mean and what to do about them.

Should you follow the 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 that issued by Puppet Labs 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 codebase. 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

Run puppet-lint --help to see the complete list of check configuration commands.

See also

  • The Automatic syntax checking with Git hooks recipe in Chapter 1, Puppet Infrastructure
  • The Testing your manifests with rspec-puppet recipe in Chapter 8, External Tools and the Puppet Ecosystem
主站蜘蛛池模板: 特克斯县| 大方县| 都兰县| 霍州市| 滦南县| 龙门县| 普兰店市| 青铜峡市| 泊头市| 洪雅县| 桐柏县| 长治市| 六盘水市| 县级市| 兴山县| 乌鲁木齐县| 来宾市| 高安市| 米林县| 浑源县| 桃源县| 昌吉市| 绥芬河市| 永寿县| 方山县| 政和县| 丹棱县| 自治县| 龙山县| 沅江市| 社旗县| 梧州市| 东乌珠穆沁旗| 怀集县| 长寿区| 始兴县| 龙川县| 象山县| 青铜峡市| 桂东县| 抚宁县|