{"id":2846,"date":"2012-12-13T01:37:49","date_gmt":"2012-12-13T00:37:49","guid":{"rendered":"http:\/\/www.devco.net\/?p=2846"},"modified":"2012-12-13T19:05:38","modified_gmt":"2012-12-13T18:05:38","slug":"simple-puppet-module-structure-redux","status":"publish","type":"post","link":"https:\/\/www.devco.net\/archives\/2012\/12\/13\/simple-puppet-module-structure-redux.php","title":{"rendered":"Simple Puppet Module Structure Redux"},"content":{"rendered":"<p>Back in September 2009 I wrote a blog post titled <a href=\"http:\/\/www.devco.net\/archives\/2009\/09\/28\/simple_puppet_module_structure.php\">&#8220;Simple Puppet Module Structure&#8221;<\/a> which introduced a simple approach to writing Puppet Modules.  This post has been hugely popular in the community &#8211; but much has changed in Puppet since then so it is time for an updated version of that post.<\/p>\n<p>As before I will show a simple module for a common scenario. Rather than considering this module a blueprint for every module out there you should instead study its design and use it as a starting point when writing your own modules. You can build on it and adapt it but the basic approach should translate well to more complex modules.<\/p>\n<p>I should note that while I work for Puppet Labs I do not know if this reflect any kind of standard suggested approach by Puppet Labs &#8211; this is what I do when managing my own machines and no more.<\/p>\n<p><H2>The most important deliverables<\/H2><br \/>\nWhen writing a module I have a few things I keep in mind &#8211; these are all centered around down stream users of my module and future-me trying to figure out what is going on:<\/p>\n<ul>\n<li>A module should have a single entry point where someone reviewing it can get an overview of it&#8217;s behavior<\/li>\n<li>Modules that have configuration should be configurable in a single way and single place<\/li>\n<li>Modules should be made up of several single-responsibility classes. As far as possible these classes should be private details hidden from the user<\/li>\n<li>For the common use cases, users should not need to know individual resource names<\/li>\n<li>For the most common use case, users should not need to provide any parameters, defaults should be used<\/li>\n<li>Modules I write should have a consistant design and behaviour<\/li>\n<\/ul>\n<p>The module layout I will present below is designed so that someone who is curious about the behaviour of the module only have to look in the <em>init.pp<\/em> to see:<\/p>\n<ul>\n<li>All the parameters and their defaults used to configure the behaviour of the module<\/li>\n<li>Overview of the internal structure of the module by way of descriptive class names<\/li>\n<li>Relationships and notifications that exist inside the module and what classes they can notify<\/li>\n<\/ul>\n<p>This design will never remove the need for documenting your modules but a clear design will guide your users in discovering the internals of your module and how they interact with it.<\/p>\n<p>More important than what a module does is how accessible it is to you and others, how easy is it to understand, debug and extend.<\/p>\n<p><H2>Thinking about your module<\/H2><br \/>\nFor this post I will write a very simple module to manage NTP &#8211; it really is very simple, you should check the <a href=\"http:\/\/forge.puppetlabs.com\/\">Forge<\/a> for more complete ones.<\/p>\n<p>To go from nowhere to having NTP on your machine you would have to do:<\/p>\n<ul>\n<li>Install the packages and any dependencies<\/li>\n<li>Write out appropriate configuration files with some environment specific values<\/li>\n<li>Start the service or services you need once the configuration files are written.  Restart it if the config file change later.<\/li>\n<\/ul>\n<p>There is a clear implied dependency chain here and this basic pattern applies to most pieces of software.<\/p>\n<p>These 3 points basically translate to distinct groups of actions and sticking with the above principal of single function classes I will create a class for each group.<\/p>\n<p>To keep things clear and obvious I will call these class <em>install<\/em>, <em>config<\/em> and <em>service<\/em>.  The names don&#8217;t matter as long as they are descriptive &#8211; but you really should pick something and stick with it in all your modules.<\/p>\n<p><H2>Writing the module<\/H3><br \/>\nI&#8217;ll show the 3 classes that does the heavy lifting here and discuss parts of them afterwards:<\/p>\n<p><code><\/p>\n<pre LANG=\"puppet\">\r\nclass ntp::install {\r\n   package{'ntpd':\r\n      ensure => $ntp::version\r\n   }\r\n}\r\n\r\nclass ntp::config {\r\n   $ntpservers = $ntp::ntpservers\r\n\r\n   File{\r\n      owner   => root,\r\n      group   => root,\r\n      mode    => 644,\r\n   }\r\n\r\n   file{'\/etc\/ntp.conf':\r\n         content => template('ntp\/ntp.conf.erb');\r\n\r\n        '\/etc\/ntp\/step-tickers':\r\n         content => template('ntp\/step-tickers.erb');\r\n    }\r\n}\r\n\r\nclass ntp::service {\r\n   $ensure = $ntp::start ? {true => running, default => stopped}\r\n\r\n   service{\"ntp\":\r\n      ensure  => $ensure,\r\n      enable  => $ntp::enable,\r\n   }\r\n}\r\n<\/pre>\n<p><\/code><\/p>\n<p>Here I have 3 classes that serve a single purpose each and do not have any details like relationships, ordering or notifications in them.  They roughly just do the one thing they are supposed to do.<\/p>\n<p>Take a look at each class and you will see they use variables like <em>$ntp::version<\/em>, <em>$ntp::ntpservers<\/em> etc.  These are variables from the the main <em>ntp<\/em> class, lets take a quick look at that class:<\/p>\n<p><code><\/p>\n<pre LINE=\"1\" LANG=\"puppet\">\r\n# == Class: ntp\r\n#\r\n# A basic module to manage NTP\r\n#\r\n# === Parameters\r\n# [*version*]\r\n#   The package version to install\r\n#\r\n# [*ntpservers*]\r\n#   An array of NTP servers to use on this node\r\n#\r\n# [*enable*]\r\n#   Should the service be enabled during boot time?\r\n#\r\n# [*start*]\r\n#   Should the service be started by Puppet\r\nclass ntp(\r\n   $version = \"present\",\r\n   $ntpservers = [\"1.pool.ntp.org\", \"2.pool.ntp.org\"],\r\n   $enable = true,\r\n   $start = true\r\n) {\r\n   class{'ntp::install': } ->\r\n   class{'ntp::config': } ~>\r\n   class{'ntp::service': } ->\r\n   Class[\"ntp\"]\r\n}\r\n<\/pre>\n<p><\/code><\/p>\n<p>This is the main entry point into the module that was mentioned earlier.  All the variables the module use is documented in a single place, the basic design and parts of the module is clear and you can see that the service class can be notified and the relationships between the parts.<\/p>\n<p>I use the new chaining features to inject the dependencies and relationships here which surfaces these important interactions between the various classes back up to the main entry class for users to see easily.<\/p>\n<p>All this information is immediately available in the obvious place without looking at any additional files or by being bogged down with implementation details.<\/p>\n<p>Line 26 here requires some extra explanation &#8211; This ensures that all the NTP member classes are applied before this main NTP class so that cases where someone say <em>require => Class[&#8220;ntp&#8221;]<\/em> elsewhere they can be sure the associated tasks are completed.  This is a light weight version of the <a href=\"http:\/\/projects.puppetlabs.com\/projects\/puppet\/wiki\/Anchor_Pattern\">Anchor Pattern<\/a>.<\/p>\n<h2>Using the module<\/h2>\n<p>Let&#8217;s look at how you might use this module from knowing nothing.<\/p>\n<p>Ideally simply including the main entry point on a node should be enough:<\/p>\n<p><code><\/p>\n<pre LANG=\"puppet\">\r\ninclude ntp\r\n<\/pre>\n<p><\/code><\/p>\n<p>This does what you&#8217;d generally expect &#8211; installs, configures and starts the NTP service.<\/p>\n<p>After looking at the <em>init.pp<\/em> you can now supply some new values for some of the parameters to tune it for your needs:<\/p>\n<p><code><\/p>\n<pre LANG=\"puppet\">\r\nclass{\"ntp\": ntpservers => [\"ntp1.example.com\", \"ntp2.example.com\"]}\r\n<\/pre>\n<p><\/code><\/p>\n<p>Or you can use the new data bindings in Puppet 3 and supply new data in Hiera to override these variables by supplying data for the keys like <em>ntp::ntpservers<\/em>.<\/p>\n<p>Finally if for some or other related reason you need to restart the service you know from looking at the <em>ntp<\/em> class that you can notify the <em>ntp::service<\/em> class to achieve that.<\/p>\n<h2>Using classes for relationships<\/h2>\n<p>There&#8217;s a huge thing to note here in the main <em>ntp<\/em> class.  I specify all relationships and notifies on the classes and not the resources themselves.<\/p>\n<p>As personal style I only mention resources by name inside a class that contains that resource &#8211; if I ever have to access a resource outside of the class that it is contained in I access the class.<\/p>\n<p>I would not write:<\/p>\n<p><code><\/p>\n<pre LANG=\"puppet\">\r\nclass ntp::service {\r\n   service{\"ntp\": require => File[\"\/etc\/ntp.conf\"]}\r\n}\r\n<\/pre>\n<p><\/code><\/p>\n<p>These are many issues with this approach that mostly come down to maintenance headaches.  Here I require the ntp config file but what if a service have more than one file? Do you then list all the files? Do you later edit every class that reference these when another file gets managed?<\/p>\n<p>These issues quickly multiply in a large code base. By always acting on class names and by creating many small single purpose classes as here I effectively contain these by grouping names and not individual resource names.  This way any future refactoring of individual classes would not have an impact on other classes.<\/p>\n<p>So the above snippet would rather be something like this:<\/p>\n<p><code><\/p>\n<pre LANG=\"puppet\">\r\nclass ntp::service {\r\n   service{\"ntp\": require => Class[\"ntp::config\"]}\r\n}\r\n<\/pre>\n<p><\/code><\/p>\n<p>Here I require the <em>containing class<\/em> and not the resource. This has the effect of requiring all <em>resources<\/em> inside that class. This has the effect of isolating changes to that class and avoiding a situation where users have to worry about the internal implementation details of the other class. Along the same lines you can also notify a class &#8211; and all resources inside that class gets notified.<\/p>\n<p>I only include other classes at the top <em>ntp<\/em> level and never have include statements in my classes like <em>ntp::confg<\/em> and so forth &#8211; this means when I require the class <em>ntp::config<\/em> or notify <em>ntp::service<\/em> I get just what I want and no more.<\/p>\n<p>If you create big complex classes you run the risk of having refreshonly execs that relate to configuration or installation associated with services in the same class which would have disastrous consequences if you notify the wrong thing or if a user do not study your code before using it.<\/p>\n<p>A consistant style of small single purpose classes named descriptively avoid these and other problems.<\/p>\n<h2>What we learned and further links<\/h2>\n<p>There is a lot to learn here and much of it is about soft issues like the value of consistency and clarity of design and thinking about your users &#8211; and your future self.<\/p>\n<p>On the technical side you should learn about the effects of relationships and notifications based on containing classes and not by naming resources by name.<\/p>\n<p>And we came across a number of recently added Puppet features:<\/p>\n<ul>\n<li><a href=\"http:\/\/docs.puppetlabs.com\/guides\/parameterized_classes.html\">Parameterized classes<\/a><\/li>\n<li><a href=\"http:\/\/docs.puppetlabs.com\/puppet\/3\/reference\/lang_relationships.html#chaining-arrows\">Chaining<\/a> Arrows<\/li>\n<li>Data Bindings as introduced in Puppet 3<\/li>\n<\/ul>\n<p>Parameterized Classes are used to provide multiple convenient methods for supplying data to your module &#8211; defaults in the module, specifically in code, using Hiera and (not shown here) an ENC.<\/p>\n<p>Chaining Arrows are used in the main class to inject the dependencies and notifications in a way that is visible without having to study each individual class.<\/p>\n<p>These are important new additions to Puppet.  Some new features like Parameterised classes are not quite ready for prime time imho but in Puppet 3 when combined with the data bindings a lot of the pain points have been removed.<\/p>\n<p>Finally there are a number of useful things I did not mention here.  Specifically you should study the <a href=\"http:\/\/docs.puppetlabs.com\/guides\/style_guide.html\">Puppet Style Guide<\/a> and use the <a href=\"http:\/\/puppet-lint.com\/\">Puppet Lint<\/a> tool to validate your modules comply.  You should consider writing tests for your modules using <a href=\"http:\/\/rspec-puppet.com\/\">rspec-puppet<\/a> and finally share it on the <a href=\"http:\/\/forge.puppetlabs.com\/\">Puppet Forge<\/a>.<\/p>\n<p>And perhaps most importantly &#8211; do not reinvent the wheel, check the Forge first.<\/p>\n","protected":false},"excerpt":{"rendered":"<p>Back in September 2009 I wrote a blog post titled &#8220;Simple Puppet Module Structure&#8221; which introduced a simple approach to writing Puppet Modules. This post has been hugely popular in the community &#8211; but much has changed in Puppet since then so it is time for an updated version of that post. As before I [&hellip;]<\/p>\n","protected":false},"author":2,"featured_media":0,"comment_status":"open","ping_status":"open","sticky":false,"template":"","format":"standard","meta":{"_et_pb_use_builder":"","_et_pb_old_content":"","footnotes":""},"categories":[7],"tags":[121,85,21],"_links":{"self":[{"href":"https:\/\/www.devco.net\/wp-json\/wp\/v2\/posts\/2846"}],"collection":[{"href":"https:\/\/www.devco.net\/wp-json\/wp\/v2\/posts"}],"about":[{"href":"https:\/\/www.devco.net\/wp-json\/wp\/v2\/types\/post"}],"author":[{"embeddable":true,"href":"https:\/\/www.devco.net\/wp-json\/wp\/v2\/users\/2"}],"replies":[{"embeddable":true,"href":"https:\/\/www.devco.net\/wp-json\/wp\/v2\/comments?post=2846"}],"version-history":[{"count":28,"href":"https:\/\/www.devco.net\/wp-json\/wp\/v2\/posts\/2846\/revisions"}],"predecessor-version":[{"id":2875,"href":"https:\/\/www.devco.net\/wp-json\/wp\/v2\/posts\/2846\/revisions\/2875"}],"wp:attachment":[{"href":"https:\/\/www.devco.net\/wp-json\/wp\/v2\/media?parent=2846"}],"wp:term":[{"taxonomy":"category","embeddable":true,"href":"https:\/\/www.devco.net\/wp-json\/wp\/v2\/categories?post=2846"},{"taxonomy":"post_tag","embeddable":true,"href":"https:\/\/www.devco.net\/wp-json\/wp\/v2\/tags?post=2846"}],"curies":[{"name":"wp","href":"https:\/\/api.w.org\/{rel}","templated":true}]}}