Skip to content

Commit

Permalink
Do not let foreman_proxy_content inherit from params
Browse files Browse the repository at this point in the history
  • Loading branch information
ekohl committed Sep 26, 2018
1 parent 89880a7 commit e331477
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 5 deletions.
8 changes: 4 additions & 4 deletions manifests/foreman_proxy_content.pp
Original file line number Diff line number Diff line change
Expand Up @@ -13,10 +13,10 @@
#
class certs::foreman_proxy_content (
String[1] $parent_fqdn = $::fqdn,
String $foreman_proxy_fqdn = $::certs::params::node_fqdn,
Array[String] $foreman_proxy_cname = $::certs::params::cname,
String[1] $certs_tar = $::certs::params::certs_tar,
) inherits certs::params {
String $foreman_proxy_fqdn = $::fqdn,
Array[String] $foreman_proxy_cname = [],
String[1] $certs_tar = undef,
) {

if $foreman_proxy_fqdn == $::fqdn {
fail('The hostname is the same as the provided hostname for the foreman-proxy')
Expand Down
1 change: 0 additions & 1 deletion manifests/params.pp
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,6 @@
$candlepin_amqp_truststore = "${candlepin_amqp_store_dir}/candlepin.truststore"
$candlepin_amqp_keystore = "${candlepin_amqp_store_dir}/candlepin.jks"

$certs_tar = undef
# Settings for uploading packages to Katello
$katello_user = undef
$katello_password = undef
Expand Down

0 comments on commit e331477

Please sign in to comment.