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

Side by Side Diff: modules/adblockplus/manifests/init.pp

Issue 29367556: Issue 3065 - Refactor adblockplus::puppet module to include Puppet patches (Closed)
Patch Set: Created Dec. 15, 2016, 9:58 a.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | modules/adblockplus/manifests/puppet.pp » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # == Class: adblockplus 1 # == Class: adblockplus
2 # 2 #
3 # The adblockplus class and the associated adblockplus:: namespace are 3 # The adblockplus class and the associated adblockplus:: namespace are
4 # used to integrate Puppet modules with each other, in order to assemble 4 # used to integrate Puppet modules with each other, in order to assemble
5 # the setups used by the Adblock Plus project. 5 # the setups used by the Adblock Plus project.
6 # 6 #
7 # === Parameters: 7 # === Parameters:
8 # 8 #
9 # [*authority*] 9 # [*authority*]
10 # The authorative domain or zone associated with the current environment. 10 # The authorative domain or zone associated with the current environment.
(...skipping 25 matching lines...) Expand all
36 # }, 36 # },
37 # } 37 # }
38 # 38 #
39 class adblockplus ( 39 class adblockplus (
40 $authority = hiera('adblockplus::authority', 'adblockplus.org'), 40 $authority = hiera('adblockplus::authority', 'adblockplus.org'),
41 $hosts = hiera_hash('adblockplus::hosts', {}), 41 $hosts = hiera_hash('adblockplus::hosts', {}),
42 $packages = hiera_array('adblockplus::packages', []), 42 $packages = hiera_array('adblockplus::packages', []),
43 $users = hiera_hash('adblockplus::users', {}), 43 $users = hiera_hash('adblockplus::users', {}),
44 ) { 44 ) {
45 45
46 include adblockplus::puppet
47 include postfix 46 include postfix
48 include ssh 47 include ssh
49 include stdlib 48 include stdlib
50 49
51 # See https://issues.adblockplus.org/ticket/3575#comment:2 50 # See https://issues.adblockplus.org/ticket/3575#comment:2
52 class {'logrotate': 51 class {'logrotate':
53 stage => 'runtime', 52 stage => 'runtime',
54 } 53 }
55 54
56 # Class['apt'] cannot yet be configured to update on-demand 55 # Class['apt'] cannot yet be configured to update on-demand
(...skipping 50 matching lines...) Expand 10 before | Expand all | Expand 10 after
107 } 106 }
108 107
109 # Fix implicit package dependency Class['apt'] does not properly handle 108 # Fix implicit package dependency Class['apt'] does not properly handle
110 Exec['apt_update'] -> Package<|title != 'python-software-properties'|> 109 Exec['apt_update'] -> Package<|title != 'python-software-properties'|>
111 110
112 # https://issues.adblockplus.org/ticket/3574#comment:19 111 # https://issues.adblockplus.org/ticket/3574#comment:19
113 ensure_packages($packages) 112 ensure_packages($packages)
114 113
115 # https://projects.puppetlabs.com/issues/4145 114 # https://projects.puppetlabs.com/issues/4145
116 ensure_resource('file', '/etc/ssh/ssh_known_hosts', { 115 ensure_resource('file', '/etc/ssh/ssh_known_hosts', {
117 ensure => 'present', 116 'ensure' => 'present',
118 mode => 0644, 117 'group' => 'root',
118 'mode' => 0644,
119 'owner' => 'root',
mathias 2016/12/15 10:01:10 Accidental change, removed this one.
119 }) 120 })
120 121
121 # See modules/adblockplus/manifests/host.pp 122 # See modules/adblockplus/manifests/host.pp
122 create_resources('adblockplus::host', $hosts) 123 create_resources('adblockplus::host', $hosts)
123 124
124 # See modules/adblockplus/manifests/user.pp 125 # See modules/adblockplus/manifests/user.pp
125 create_resources('adblockplus::user', $users) 126 create_resources('adblockplus::user', $users)
126 } 127 }
OLDNEW
« no previous file with comments | « no previous file | modules/adblockplus/manifests/puppet.pp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld