Description of problem: Setting "include_swift" to True in quickstack::pacemaker::params.pp causes the following error: Could not retrieve catalog from remote server: Error 400 on SERVER: Puppet::Parser::AST::Resource failed with error ArgumentError: Could not find declared class ::swift::proxy::proxy-logging at /etc/puppet/environments/KT_salab_Development_OpenStack_3/modules/quickstack/manifests/swift/proxy.pp:41 Version-Release number of selected component (if applicable): openstack-puppet-modules-2015.1.8-8.el7ost.noarch master branch of quickstack from https://github.com/redhat-openstack/astapor How reproducible: 100% Steps to Reproduce: 1. Create a Controller host group which contains the following classes: quickstack::ntp quickstack::openstack_common quickstack::pacemaker::ceilometer quickstack::pacemaker::cinder quickstack::pacemaker::common quickstack::pacemaker::galera quickstack::pacemaker::glance quickstack::pacemaker::heat quickstack::pacemaker::horizon quickstack::pacemaker::keystone quickstack::pacemaker::load_balancer quickstack::pacemaker::memcached quickstack::pacemaker::neutron quickstack::pacemaker::nosql quickstack::pacemaker::nova quickstack::pacemaker::params quickstack::pacemaker::qpid quickstack::pacemaker::rabbitmq quickstack::pacemaker::swift 2. Set "include_swift" to True in quickstack::pacemaker::params 3. Apply host group to systems. Actual results: 400 Error Expected results: Puppet classes are applied. Additional info: The name of the proxy_logging class was changed in puppet-swift with this commit: https://github.com/openstack/puppet-swift/commit/0f7601a3587511b633d0a04ffa24f42f19b3786c
https://github.com/redhat-openstack/astapor/pull/562 resolves the issue for me.
Michael, sorry this BZ sat for so long, I think it was not noticed because it was targeted at liberty (by rules engine), where we no longer ship ofi. I am not sure if this is worth escalating to get into an ofi build, but I looked and your patch, and that looks fine to me. Keep in mind, especially if you are using staypuft, swift was not officially supported, I don't think any QA was pointed at it. That said, I could at least merge your patch to master if you want, and if it is still important for you, we could try to get acks to have a build. If not, I propose we close this (just let me know, as well as if you want the patch merged). I know of no current plans to do another ofi build at this time.
No need for an OFI build - I'd appreciate having the patches merged to master though. I'll close as upstream (understanding what that means for this). Michael.