Issue 3065 - Introduce adblockplus::puppet::master module

modules/adblockplus/manifests/puppet/master.pp

Index: modules/adblockplus/manifests/puppet/master.pp
===================================================================
new file mode 100644
--- /dev/null
+++ b/modules/adblockplus/manifests/puppet/master.pp
@@ -0,0 +1,50 @@
+# == Class: adblockplus::puppet::master
+#
+# Manage Puppet (https://github.com/puppetlabs/puppet) master configuration.
+#
+# === Parameters:
+#
+# [*package*]
+#   Custom parameters for the implicit Package['puppetmaster'] resource.
+#
+# [*service*]
+#   Custom parameters for the Service['puppetmaster'] resource implicitly
+#   defined the package is ensured to be neither "absent" nor "purged".
+#
+# === Examples:
+#
+#   class {'adblockplus::puppet::master':
+#     package => {
+#       'ensure' => 'present',
+#       'name' => 'puppetmaster',
+#     },
+#     service => {
+#       'ensure' => 'running',
+#       'name' => 'puppetmaster',
+#     },
+#   }
+#
+class adblockplus::puppet::master (
+  $package = hiera('adblockplus::puppet::master::package', {}),
+  $service = hiera('adblockplus::puppet::master::service', {}),
+) {
+
+  include adblockplus
+  include puppetmaster
+  include stdlib
+
+  # https://forge.puppet.com/puppetlabs/stdlib#ensure_resource
+  ensure_resource('package', 'puppetmaster', $package)

[f.lopez, 2016/12/14 16:12:37]

+1 for using ensure_resources instead of ensure_packages, even tho I'm going to
upgrade the dependency

+
+  # https://forge.puppet.com/puppetlabs/stdlib#getparam
+  if getparam(Package['puppet'], 'ensure') !~ /^(absent|purged)$/ {
+
+    ensure_resource('service', 'puppetmaster', merge({
+      'ensure' => 'running',
+      'hasrestart' => true,
+      'hasstatus' => true,
+    }, $service))
+
+    Service['puppetmaster'] <- Package['puppetmaster']
+  }
+}