Skip to content

Commit a6eb161

Browse files
committed
(MODULES-10971) - Ensure apt::keyserver is considered when creating a default apt:source
As it stands the source_key_defaults point towards the default `keyserver` value set within the params. This change makes it so that it will instead use the `keyserver` value set in the init, which will in turn default to the params value in none has been given.
1 parent 18fe46c commit a6eb161

File tree

2 files changed

+8
-8
lines changed

2 files changed

+8
-8
lines changed

manifests/init.pp

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -153,8 +153,15 @@
153153
String $preferences_d = $apt::params::preferences_d,
154154
String $apt_conf_d = $apt::params::apt_conf_d,
155155
Hash $config_files = $apt::params::config_files,
156-
Hash $source_key_defaults = $apt::params::source_key_defaults,
157156
Boolean $sources_list_force = $apt::params::sources_list_force,
157+
158+
Hash $source_key_defaults = {
159+
'server' => $keyserver,
160+
'options' => undef,
161+
'content' => undef,
162+
'source' => undef,
163+
}
164+
158165
) inherits apt::params {
159166

160167
if $facts['os']['family'] != 'Debian' {

manifests/params.pp

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -71,13 +71,6 @@
7171
'apt.conf.d' => false,
7272
}
7373

74-
$source_key_defaults = {
75-
'server' => $keyserver,
76-
'options' => undef,
77-
'content' => undef,
78-
'source' => undef,
79-
}
80-
8174
$include_defaults = {
8275
'deb' => true,
8376
'src' => false,

0 commit comments

Comments
 (0)