[openstack-dev] [puppet] Incompatible default parameters

2015-10-21 Thread Martin Mágr
Greetings, I'd like to discuss bug 1506061 ([1]). The main problem I'm trying to solve here is that using default parameters of classes cinder::api and nova::keystone::auth migration won't work, because Cinder will search for endpoint with different name ('Compute service' instead of

Re: [openstack-dev] [puppet] Incompatible default parameters

2015-10-21 Thread Mathieu Gagné
This change [1] introduced the nova_catalog_info and nova_catalog_admin_info parameters. Martin Mágr also mentioned that default values were wrong but the change got merged anyway. I agree that those default values should be changed to "nova" like initially suggested as the 2nd field is the

Re: [openstack-dev] [puppet] Incompatible default parameters

2015-10-21 Thread Clayton O'Neill
I think the only people that would be affected by the puppet-openstack module default not matching the keystone default are people that are trying to retrofit puppet into an already running environment. Those people already have a ton of work laid out in front of them, so I’m ok with making it