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

Refs #7104 - ensure the qpidd is really running before configuring it #33

Merged
merged 1 commit into from
Aug 27, 2014

Conversation

iNecas
Copy link
Member

@iNecas iNecas commented Aug 25, 2014

No description provided.

@waldenraines
Copy link

I am testing this currently.

@waldenraines
Copy link

I applied the changes from this PR as well as theforeman/puppet-service_wait#4 and I get the following errors:

 qpid-config --ssl-certificate /etc/pki/katello/certs/java-client.crt --ssl-key /etc/pki/katello/private/java-client.key -b 'amqps://[redacted]:5671' add exchange topic event --durable returned 1 instead of one of [0]
 /Stage[main]/Certs::Candlepin/Exec[create candlepin qpid exchange]/returns: change from notrun to 0 failed: qpid-config --ssl-certificate /etc/pki/katello/certs/java-client.crt --ssl-key /etc/pki/katello/private/java-client.key -b 'amqps://[redacted]:5671' add exchange topic event --durable returned 1 instead of one of [0]
 /Stage[main]/Candlepin::Service/Exec[cpinit]: Failed to call refresh: /usr/bin/wget --timeout=30 --tries=5 --retry-connrefused -qO- http://localhost:8080/candlepin/admin/init >/var/log/candlepin/cpinit.log 2>&1 && touch /var/lib/candlepin/cpinit_done returned 4 instead of one of [0]
 /Stage[main]/Candlepin::Service/Exec[cpinit]: /usr/bin/wget --timeout=30 --tries=5 --retry-connrefused -qO- http://localhost:8080/candlepin/admin/init >/var/log/candlepin/cpinit.log 2>&1 && touch /var/lib/candlepin/cpinit_done returned 4 instead of one of [0]
 /Stage[main]/Foreman::Database/Foreman::Rake[db:seed]/Exec[foreman-rake-db:seed]: Failed to call refresh: /usr/sbin/foreman-rake db:seed returned 1 instead of one of [0]
 /Stage[main]/Foreman::Database/Foreman::Rake[db:seed]/Exec[foreman-rake-db:seed]: /usr/sbin/foreman-rake db:seed returned 1 instead of one of [0]

Here are my changes:

[walden@[redacted] ~]$ grep -n -B 3 qpidd /usr/share/katello-installer/modules/certs/manifests/candlepin.pp 
77-      command     => "certutil -A -d '${::certs::nss_db_dir}' -n 'amqp-client' -t ',,' -a -i '${client_cert}'",
78-      refreshonly => true,
79-      subscribe   => Exec['create-nss-db'],
80:      notify      => Service['qpidd'],
--
88-    exec { 'create candlepin qpid exchange':
89-      command => "qpid-config --ssl-certificate ${client_cert} --ssl-key ${client_key} -b 'amqps://${::fqdn}:5671' add exchange topic event --durable",
90-      unless  => "qpid-config --ssl-certificate ${client_cert} --ssl-key ${client_key} -b 'amqps://${::fqdn}:5671' exchanges event",
91:      require => Service['qpidd'],
[walden@[redacted] ~]$ cat /usr/share/katello-installer/modules/service_wait/plugins/qpidd.sh
confine [ $SERVICE = 'qpidd' ]

QPID_PORT=${QPID_PORT:-5671}

service-wait () {
    wait-for-port $QPID_PORT
}

@waldenraines
Copy link

This worked for me once I had everything set up correctly, ACK.

iNecas added a commit that referenced this pull request Aug 27, 2014
Refs #7104 - ensure the qpidd is really running before configuring it
@iNecas iNecas merged commit e33f09c into theforeman:master Aug 27, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants