Code Review
/
puppet-modules
/
puppetlabs-apt.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
review
|
tree
raw
|
inline
| side by side
do not specify file modes unless relevant
[puppet-modules/puppetlabs-apt.git]
/
manifests
/
init.pp
diff --git
a/manifests/init.pp
b/manifests/init.pp
index b8bc45b59dfc7b86ec97d76a798c93d9038397e8..d2aead96112f26cf31a8d11bb51c8d207317d367 100644
(file)
--- a/
manifests/init.pp
+++ b/
manifests/init.pp
@@
-148,6
+148,7
@@
class apt (
String $conf_d = $apt::params::conf_d,
String $preferences = $apt::params::preferences,
String $preferences_d = $apt::params::preferences_d,
String $conf_d = $apt::params::conf_d,
String $preferences = $apt::params::preferences,
String $preferences_d = $apt::params::preferences_d,
+ String $apt_conf_d = $apt::params::apt_conf_d,
Hash $config_files = $apt::params::config_files,
Hash $source_key_defaults = $apt::params::source_key_defaults,
) inherits apt::params {
Hash $config_files = $apt::params::config_files,
Hash $source_key_defaults = $apt::params::source_key_defaults,
) inherits apt::params {
@@
-184,6
+185,9
@@
class apt (
if $purge['preferences.d'] {
assert_type(Boolean, $purge['preferences.d'])
}
if $purge['preferences.d'] {
assert_type(Boolean, $purge['preferences.d'])
}
+ if $purge['apt.conf.d'] {
+ assert_type(Boolean, $purge['apt.conf.d'])
+ }
$_purge = merge($::apt::purge_defaults, $purge)
$_proxy = merge($apt::proxy_defaults, $proxy)
$_purge = merge($::apt::purge_defaults, $purge)
$_proxy = merge($apt::proxy_defaults, $proxy)
@@
-200,9
+204,9
@@
class apt (
}
}
}
}
- $sources_list_
content
= $_purge['sources.list'] ? {
- true =>
"# Repos managed by puppet.\n"
,
- default =>
undef
,
+ $sources_list_
ensure
= $_purge['sources.list'] ? {
+ true =>
absent
,
+ default =>
file
,
}
$preferences_ensure = $_purge['preferences'] ? {
}
$preferences_ensure = $_purge['preferences'] ? {
@@
-222,13
+226,11
@@
class apt (
}
file { 'sources.list':
}
file { 'sources.list':
- ensure => file,
- path => $::apt::sources_list,
- owner => root,
- group => root,
- mode => '0644',
- content => $sources_list_content,
- notify => Class['apt::update'],
+ ensure => $sources_list_ensure,
+ path => $::apt::sources_list,
+ owner => root,
+ group => root,
+ notify => Class['apt::update'],
}
file { 'sources.list.d':
}
file { 'sources.list.d':
@@
-236,7
+238,6
@@
class apt (
path => $::apt::sources_list_d,
owner => root,
group => root,
path => $::apt::sources_list_d,
owner => root,
group => root,
- mode => '0644',
purge => $_purge['sources.list.d'],
recurse => $_purge['sources.list.d'],
notify => Class['apt::update'],
purge => $_purge['sources.list.d'],
recurse => $_purge['sources.list.d'],
notify => Class['apt::update'],
@@
-247,7
+248,6
@@
class apt (
path => $::apt::preferences,
owner => root,
group => root,
path => $::apt::preferences,
owner => root,
group => root,
- mode => '0644',
notify => Class['apt::update'],
}
notify => Class['apt::update'],
}
@@
-256,12
+256,21
@@
class apt (
path => $::apt::preferences_d,
owner => root,
group => root,
path => $::apt::preferences_d,
owner => root,
group => root,
- mode => '0644',
purge => $_purge['preferences.d'],
recurse => $_purge['preferences.d'],
notify => Class['apt::update'],
}
purge => $_purge['preferences.d'],
recurse => $_purge['preferences.d'],
notify => Class['apt::update'],
}
+ file { 'apt.conf.d':
+ ensure => directory,
+ path => $::apt::apt_conf_d,
+ owner => root,
+ group => root,
+ purge => $_purge['apt.conf.d'],
+ recurse => $_purge['apt.conf.d'],
+ notify => Class['apt::update'],
+ }
+
if $confs {
create_resources('apt::conf', $confs)
}
if $confs {
create_resources('apt::conf', $confs)
}
@@
-306,5
+315,5
@@
class apt (
}
# required for adding GPG keys on Debian 9 (and derivatives)
}
# required for adding GPG keys on Debian 9 (and derivatives)
- ensure_packages(['
dirmngr
'])
+ ensure_packages(['
gnupg
'])
}
}