Issue 3574 - Introduce parameter $adblockplus::packages

modules/adblockplus/manifests/init.pp

 # == Class: adblockplus
 #
 # The adblockplus class and the associated adblockplus:: namespace are
 # used to integrate Puppet modules with each other, in order to assemble
 # the setups used by the Adblock Plus project.
 #
 # === Parameters:
 #
 # [*authority*]
 #   The authorative domain or zone associated with the current environment,
@@ skipping 18 matching lines @@
 #     users => {
 #       'pinocchio' => {
 #         # see adblockplus::user
 #       },
 #     },
 #   }
 #
 class adblockplus (
   $authority = hiera('adblockplus::authority', 'adblockplus.org'),
   $hosts = hiera_hash('adblockplus::hosts', {}),
+  $packages = hiera_array('adblockplus::packages', []),
   $users = hiera_hash('adblockplus::users', {}),
 ) {
 
   # See https://issues.adblockplus.org/ticket/3574#comment:8
   class {'base':
     zone => $authority,
   }
 
   # Class['apt'] cannot yet be configured to update on-demand
   class {'apt':
@@ skipping 44 matching lines @@
         notify => Service['ssh'];
       '/etc/ssh/ssh_host_rsa_key.pub':
         source => 'puppet:///modules/adblockplus/development_host_rsa_key.pub',
         mode => 644;
     }
   }
 
   # Fix implicit package dependency Class['apt'] does not properly handle
   Exec['apt_update'] -> Package<|title != 'python-software-properties'|>
 
+  # https://issues.adblockplus.org/ticket/3574#comment:18
+  ensure_packages($packages)
+
   # https://projects.puppetlabs.com/issues/4145
   ensure_resource('file', '/etc/ssh/ssh_known_hosts', {
     ensure => 'present',
     mode => 0644,
   })
 
   # See modules/adblockplus/manifests/host.pp
   create_resources('adblockplus::host', $hosts)
 
   # See modules/adblockplus/manifests/user.pp
   create_resources('adblockplus::user', $users)
 }