Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: modules/adblockplus/manifests/init.pp

Issue 29341151: Issue 4019 - Added "Edge" to platform choices in Issues tracker at issues1. (Closed)
Patch Set: Created May 10, 2016, 3:35 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « modules/adblockplus/manifests/host/monitoringserver.pp ('k') | modules/adblockplus/manifests/log.pp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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)
}
« no previous file with comments | « modules/adblockplus/manifests/host/monitoringserver.pp ('k') | modules/adblockplus/manifests/log.pp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld