The tcp line assume that by default the qpid broker is on localhost. We could remove it and force them to always set it, i think the main reason it's here is the ssl portion of the url which can get tricky if you leave it out *and* you are using an SSL qpid broker.

The AMQP_KEYSTORE setting is the default location we expect to find the keystore to use for SSL. So I'm ok with leaving it as is.

I didn't question this portion from the original implementation, with the original intent being: install qpid, configure for SSL, and listening on the default port, all I'd have to do is ENABLE AMQP integration with a boolean and we'd be up and running. If it isn't there you do get a nasty exception stack trace in the error.log :smile: The katello installer work I'm doing is explicitly going to set it in candlepin.conf.