Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

katello/certs 7.x compatibility & use certs::qpid_client variables #232

Merged
merged 2 commits into from Jan 20, 2020

Conversation

ekohl
Copy link
Member

@ekohl ekohl commented Jan 20, 2020

certs 7.0.0 started to create qpid files owned by the pulp group. This group is created by pulp-server which is in pulp::install. This more specific chaining avoids dependency cycles.

qpid_client_cert is the same:
$qpid_client_cert = $certs::qpid_client_cert,

The CA is a different path:
$qpid_client_ca_cert = $certs::qpid_client_ca_cert,

Which is defined as:
$qpid_client_ca_cert = "${pulp_pki_dir}/qpid/ca.crt"

This is different than what it used to be:
$ca_cert = "${pki_dir}/certs/${default_ca_name}.crt"

The contents should match.

qpid_client_cert is the same:
    $qpid_client_cert      = $certs::qpid_client_cert,

The CA is a different path:
    $qpid_client_ca_cert   = $certs::qpid_client_ca_cert,

Which is defined as:
    $qpid_client_ca_cert = "${pulp_pki_dir}/qpid/ca.crt"

This is different than what it used to be:
    $ca_cert = "${pki_dir}/certs/${default_ca_name}.crt"

The contents should match.
certs 7.0.0 started to create qpid files owned by the pulp group. This
group is created by pulp-server which is in pulp::install. This more
specific chaining avoids dependency cycles.
@ekohl
Copy link
Member Author

ekohl commented Jan 20, 2020

This is an alternative to theforeman/puppet-certs#270. Related to theforeman/puppet-certs#272.

@ehelms ehelms merged commit c55a889 into theforeman:master Jan 20, 2020
@ekohl ekohl deleted the certs-7 branch January 20, 2020 16:21
@ekohl ekohl mentioned this pull request Jan 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants