theforeman / puppet-pulp

Puppet module for setting up Pulp 2 as part of Katello installation
GNU General Public License v3.0
16 stars 66 forks source link

Fix https_ca_cert if a different location for ca_cert is used #382

Closed laugmanuel closed 4 years ago

laugmanuel commented 4 years ago

This fixes the bug introduced here https://github.com/theforeman/puppet-pulp/pull/381 which occurs if the default path of pulp::ca_cert is changed. The root cause is that pulp::https_ca_cert is set to the value of pulp::ca_cert in params.pp. If the value of pulp::ca_cert gets changed by passing an argument to the class, we have a missmatch if pulp::https_ca_cert is not defined at the same time.

@ekohl do you mind taking a look at the fix? @evgeni FYI

evgeni commented 4 years ago

thanks @laugmanuel!

ekohl commented 4 years ago

Thanks!