Illegal rabbitmq cluster nodes with Puppet

Question

I rely on Librarian-puppet to install rabbitmq in a vagrant box.

Puppet version is 3.4.0

My Puppetfile contains

forge "http://forge.puppetlabs.com"
[...]
mod 'rabbitmq', :git => 'git://github.com/puppetlabs/puppetlabs-rabbitmq'

By following the documentation, I intend to install RabbitMQ server with the next instruction:

class { '::rabbitmq':
    service_manage      => false,
    port                => '5672',
    delete_guest_user   => true,
}

By doing so, I've encountered the following error message:

Error: Illegal name. The given name _cluster_nodes does not conform to the naming rule
\A((::)?[a-z0-9]w*)(::[a-z0-9]w*)*\z at 
/etc/puppet/modules/rabbitmq/manifests/config.pp:45:5

Error: Illegal name. The given name _cluster_nodes does not conform to the naming rule
\A((::)?[a-z0-9]w*)(::[a-z0-9]w*)*\z at 
/etc/puppet/modules/rabbitmq/manifests/config.pp:47:5 

The actual config.pp file contains the next code block:

# Handle deprecated option.
if $cluster_disk_nodes != [] {
    notify { 'cluster_disk_nodes':
        message => 'WARNING: The cluster_disk_nodes is deprecated.
        Use cluster_nodes instead.',
    }
    $_cluster_nodes = $cluster_disk_nodes # line 45
} else {
    $_cluster_nodes = $cluster_nodes # line 47
}

Could anyone provide me with a valid instance of cluster node name? The regular expression apparently used for validation looks a bit cryptic to me.

I'm also wondering how the _cluster_nodes values are validated... Where does the regular expression used for validation comes from?

Answer 1

This issue was not about the value of variables ($cluster_nodes or $cluster_disk_nodes) and it was rather the variables names which were incorrect. A variable name shall not start with '_' anymore (as expressed by the regular expression and the actual error message). I just got lost in translation.

I've opened an issue on github and I've sent a PR, which attempts to fix it:

https://github.com/puppetlabs/puppetlabs-rabbitmq/issues/163

https://github.com/puppetlabs/puppetlabs-rabbitmq/pull/164

To follow up with this issue, it was equally fixed by another PR, which got actually merged:

https://github.com/puppetlabs/puppetlabs-rabbitmq/pull/160