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

Delta Between Two Patch Sets: modules/adblockplus/manifests/init.pp

Issue 29338036: Issue 3574 - Move APT setup from class base to module adblockplus (Closed)
Left Patch Set: Created March 9, 2016, 7:30 p.m.
Right Patch Set: Issue 3574 - Fix typo: $enfironment => $environment Created March 9, 2016, 7:33 p.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
Left: Side by side diff | Download
Right: Side by side diff | Download
« no previous file with change/comment | « no previous file | modules/base/manifests/init.pp » ('j') | no next file with change/comment »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
LEFTRIGHT
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 18 matching lines...) Expand all
29 $users = hiera_hash('adblockplus::users', {}), 29 $users = hiera_hash('adblockplus::users', {}),
30 ) { 30 ) {
31 31
32 # See https://issues.adblockplus.org/ticket/3574#comment:8 32 # See https://issues.adblockplus.org/ticket/3574#comment:8
33 class {'base': 33 class {'base':
34 zone => $authority, 34 zone => $authority,
35 } 35 }
36 36
37 # Class['apt'] cannot yet be configured to update on-demand 37 # Class['apt'] cannot yet be configured to update on-demand
38 class {'apt': 38 class {'apt':
39 always_apt_update => ($enfironment != 'development'), 39 always_apt_update => ($environment != 'development'),
40 } 40 }
41 41
42 # Used as internal constant within adblockplus::* resources 42 # Used as internal constant within adblockplus::* resources
43 $directory = '/var/adblockplus' 43 $directory = '/var/adblockplus'
44 44
45 # A common location for directories specific to the adblockplus:: setups, 45 # A common location for directories specific to the adblockplus:: setups,
46 # managed via Puppet, but accessible by all users with access to the system 46 # managed via Puppet, but accessible by all users with access to the system
47 @file {$directory: 47 @file {$directory:
48 ensure => 'directory', 48 ensure => 'directory',
49 mode => 0755, 49 mode => 0755,
(...skipping 29 matching lines...) Expand all
79 mode => 644; 79 mode => 644;
80 } 80 }
81 } 81 }
82 82
83 # Fix implicit package dependency Class['apt'] does not properly handle 83 # Fix implicit package dependency Class['apt'] does not properly handle
84 Exec['apt_update'] -> Package<|title != 'python-software-properties'|> 84 Exec['apt_update'] -> Package<|title != 'python-software-properties'|>
85 85
86 # See modules/adblockplus/manifests/user.pp 86 # See modules/adblockplus/manifests/user.pp
87 create_resources('adblockplus::user', $users) 87 create_resources('adblockplus::user', $users)
88 } 88 }
LEFTRIGHT

Powered by Google App Engine
This is Rietveld