Index: modules/adblockplus/manifests/init.pp |
=================================================================== |
--- a/modules/adblockplus/manifests/init.pp |
+++ b/modules/adblockplus/manifests/init.pp |
@@ -10,10 +10,6 @@ |
# The authorative domain or zone associated with the current environment, |
# similar to the deprecated and soon to be removed $base::zone. |
# |
-# [*hosts*] |
-# A hash of adblockplus::host $name => $parameter items to set up in this |
-# context, i.e. via Hiera. |
-# |
# [*users*] |
# A hash of adblockplus::user $name => $parameter items to set up in this |
# context, i.e. via Hiera. |
@@ -21,11 +17,6 @@ |
# === Examples: |
# |
# class {'adblockplus': |
-# hosts => { |
-# 'node1' => { |
-# # see adblockplus::host |
-# }, |
-# }, |
# users => { |
# 'pinocchio' => { |
# # see adblockplus::user |
@@ -35,30 +26,14 @@ |
# |
class adblockplus ( |
$authority = hiera('adblockplus::authority', 'adblockplus.org'), |
- $hosts = hiera_hash('adblockplus::hosts', {}), |
- $packages = hiera_array('adblockplus::packages', []), |
$users = hiera_hash('adblockplus::users', {}), |
) { |
- include postfix |
- include ssh |
- include stdlib |
- |
# See https://issues.adblockplus.org/ticket/3574#comment:8 |
class {'base': |
zone => $authority, |
} |
- # See https://issues.adblockplus.org/ticket/3575#comment:2 |
- class {'logrotate': |
- stage => 'runtime', |
- } |
- |
- # Class['apt'] cannot yet be configured to update on-demand |
- class {'apt': |
- always_apt_update => ($environment != 'development'), |
- } |
- |
# Used as internal constant within adblockplus::* resources |
$directory = '/var/adblockplus' |
@@ -70,58 +45,6 @@ |
owner => 'root', |
} |
- # A common time-zone shared by all hosts provisioned eases synchronization |
- # and debugging, i.e. log-file review and similar tasks, significantly |
- file { |
- '/etc/timezone': |
- content => 'UTC', |
- ensure => 'present', |
- group => 'root', |
- mode => 0644, |
- notify => Service['cron'], |
- owner => 'root'; |
- '/etc/localtime': |
- ensure => 'link', |
- target => '/usr/share/zoneinfo/UTC', |
- notify => Service['cron']; |
- } |
- |
- # Explicit resource required only to ensure cron(8) is running; |
- # there is no real requirement for a rationship with another resource |
- service {'cron': |
- ensure => 'running', |
- enable => true, |
- } |
- |
- # Work around https://issues.adblockplus.org/ticket/3479 |
- if $::environment == 'development' { |
- |
- file { |
- '/etc/ssh/ssh_host_rsa_key': |
- source => 'puppet:///modules/adblockplus/development_host_rsa_key', |
- mode => 600, |
- 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:19 |
- 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) |
} |