-
Notifications
You must be signed in to change notification settings - Fork 39
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
Fixes #7104: Ensure qpidd is restarted before qpid config is run. #28
Conversation
On RHEL7, qpidd is not started fully before qpid-config is run due to adding the candlepin certs to the NSS database needing to refresh the qpidd service and qpid-config requiring that the qpidd service be running when called.
I'm testing this one. |
I get these errors when trying with the change from this PR:
When trying with this change only: diff --git a/manifests/candlepin.pp b/manifests/candlepin.pp
index 83d2314..22905a1 100644
--- a/manifests/candlepin.pp
+++ b/manifests/candlepin.pp
@@ -88,6 +88,7 @@ class certs::candlepin (
exec { 'create candlepin qpid exchange':
command => "qpid-config --ssl-certificate ${client_cert} --ssl-key ${client_key} -b 'amqps://${::fqdn}:5671' add exchange topic event --durable",
unless => "qpid-config --ssl-certificate ${client_cert} --ssl-key ${client_key} -b 'amqps://${::fqdn}:5671' exchanges event",
+ require => Service['qpidd'],
} ~>
exec { 'import CA into Candlepin truststore':
command => "keytool -import -v -keystore ${amqp_truststore} -storepass ${keystore_password} -alias ${certs::default_ca_name} -file ${ca_cert} -noprompt", I get these errors:
When trying with both changes, I get this:
I have all three of these VMs still if you'd like to have a look. |
And without the changes, on the latest, do you see the error? |
|
The alternative patches to fix the issue with qpid not being ready: |
I'm closing this PR as I just ACK'd @iNecas' PRs. |
On RHEL7, qpidd is not started fully before qpid-config is run due
to adding the candlepin certs to the NSS database needing to refresh
the qpidd service and qpid-config requiring that the qpidd service
be running when called.