Bug 1261597

Summary:	Unable to provision Swift proxy on Cloud Controllers
Product:	Red Hat OpenStack	Reporter:	Michael Solberg <msolberg>
Component:	openstack-foreman-installer	Assignee:	Jason Guiditta <jguiditt>
Status:	CLOSED UPSTREAM	QA Contact:	Shai Revivo <srevivo>
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	7.0 (Kilo)	CC:	mburns, morazi, msolberg, rhos-maint, srevivo
Target Milestone:	---	Keywords:	ZStream
Target Release:	8.0 (Liberty)
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2016-07-18 20:02:15 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Michael Solberg 2015-09-09 18:31:22 UTC

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

Comment 3 Michael Solberg 2015-09-09 18:36:00 UTC

https://github.com/redhat-openstack/astapor/pull/562 resolves the issue for me.

Comment 5 Jason Guiditta 2016-07-12 13:58:58 UTC

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.

Comment 6 Michael Solberg 2016-07-18 20:02:15 UTC

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.