I noticed that on https://github.secureserver.net/cloudplatform/stackforge-puppet-nova/blob/master/manifests/cells.pp#L220-L224 , the rabbit_ variables here are referenced as ::nova::init::rabbit_*, instead of ::nova::rabbit_*.
On Puppet 3.7.3 with future parser, this causes the following errors:
Warning: Scope(Class[Nova::Cells]): Could not look up qualified variable '::nova::init::rabbit_userid'; class ::nova::init could not be found
Warning: Scope(Class[Nova::Cells]): Could not look up qualified variable '::nova::init::rabbit_password'; class ::nova::init could not be found
Warning: Scope(Class[Nova::Cells]): Could not look up qualified variable '::nova::init::rabbit_hosts'; class ::nova::init could not be found
Warning: Scope(Class[Nova::Cells]): Could not look up qualified variable '::nova::init::rabbit_port'; class ::nova::init could not be found
Warning: Scope(Class[Nova::Cells]): Could not look up qualified variable '::nova::init::rabbit_virtual_host'; class ::nova::init could not be found
I am not really up to speed with exported resources, so unless there is some special reason why this should be ::nova::init here, I think the init:: component of the variable names should be removed.
Fix proposed to branch: master
Review: https:/