erlang_cookie detection was better when NOT facter based about puppet-rabbitmq HOT 7 CLOSED

To sum it up; using puppet 2.7.22-1puppetlabs1, the custom fact rabbitmq_erlang_cookie seems to be not available to the client so if $erlang_cookie != $::rabbitmq_erlang_cookie { is always evaluating to true. A fact should just be available so just like a the $hostname fact I should be able to reference it with $rabbitmq_erlang_cookie....... but its always ""/null. I am very confused. :)

from puppet-rabbitmq.

Before I start digging into this I just want to check the very obvious case of making sure pluginsync is enabled for your 2.7 clients, right?

from puppet-rabbitmq.

confirmed, pluginsync is on:

info: Retrieving plugin
info: Loading facts in /var/lib/puppet/lib/facter/puppet_vardir.rb
info: Caching catalog for node0.shop.com
info: Applying configuration version '1374851492'
notice: /Stage[main]/Rabbitmq::Config/Exec[wipe_db]/returns: executed successfully
info: Class[Rabbitmq::Config]: Scheduling refresh of Class[Rabbitmq::Service]
info: Class[Rabbitmq::Service]: Scheduling refresh of Service[rabbitmq-server]
notice: /Stage[main]/Rabbitmq::Service/Service[rabbitmq-server]/ensure: ensure changed 'stopped' to 'running'
notice: /Stage[main]/Rabbitmq::Service/Service[rabbitmq-server]: Triggered 'refresh' from 1 events
notice: Finished catalog run in 27.07 seconds
root@node0:~# cat /etc/puppet/puppet.conf
[main]
server=middleware.shop.com
listen=true
logdir=/var/log/puppet
vardir=/var/lib/puppet
ssldir=/var/lib/puppet/ssl
rundir=/var/run/puppet
factpath=$vardir/lib/facter
templatedir=$confdir/templates
pluginsync=true

from puppet-rabbitmq.

Urgh, so embarrassing - #97 fixes this and I'll merge it tomorrow as I'm about to head out of the door. Sorry about this. I'm going to add spec tests for the fact as soon as I can get them to work properly!

from puppet-rabbitmq.

no need to be embarrassed. :)

after merging your commit I received the error: err: Failed to apply catalog: Could not find dependency Exec[wipe_db] for File[erlang_cookie] at /etc/puppet/modules/rabbitmq/manifests/config.pp:87

I assume the exec is never executed/processed when the if statement evaluates to true so you end up with this catch 22. I just removed the require.

from puppet-rabbitmq.

I'll dig into this today, I'm working through this with a cold and conjunctivitis so hang in there while I stumble around and fix it up. :)

from puppet-rabbitmq.

This should be fixed now in master.

from puppet-rabbitmq.