Bug 1003938

Summary:	impl_qpid changes to allow use of qpid federation
Product:	Red Hat OpenStack	Reporter:	Russell Bryant <rbryant>
Component:	openstack-neutron	Assignee:	Terry Wilson <twilson>
Status:	CLOSED ERRATA	QA Contact:	Rami Vaknin <rvaknin>
Severity:	medium	Docs Contact:
Priority:	high
Version:	3.0	CC:	ajeain, chrisw, ddomingo, hateya, lpeer, ndipanov, oblaut, twilson, yeylon
Target Milestone:	async
Target Release:	4.0
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:	openstack-neutron-2013.2-3.el6ost.rpm	Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:	1003937	Environment:
Last Closed:	2013-12-20 00:23:06 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:
Bug Depends On:	1003937, 1003942, 1003946, 1003951
Bug Blocks:

Description Russell Bryant 2013-09-03 14:34:19 UTC

+++ This bug was initially created as a clone of Bug #1003937 +++

A patch was recently merged upstream to the impl_qpid rpc driver to allow the use of qpid federation.  It also completes the fix for leaking qpid exchanges.  Lastly, it includes a couple of additional options to allow durable and auto-delete for queues to be configurable.  Pulling in these options make for a cleaner backport.

The commits from oslo-incubator are:

76972e2 Support a new qpid topology
5ff534d Add config for amqp durable/auto_delete queues


These changes should be backported for RHOS 3.0.z.

Comment 1 lpeer 2013-11-06 12:55:31 UTC

For testing this look at comment #9 on BZ 1003946

Comment 3 Rami Vaknin 2013-11-18 04:35:19 UTC

According to https://bugzilla.redhat.com/show_bug.cgi?id=1003946#c9, this bug should be tested by setting qpid_topology_version=2, I can find this config param only in heat, ceilometer, nova and cinder.

I've tried to set it also in neutron.conf, the server starts successsfully however I'm not sure whether this conf param is good for all OpenStack components, so:

1. Can I use the "qpid_topology_version=2" config param to all OpenStack components?
2. How can I verify that a component do use qpid federation?


# grep -R qpid_topology_version /etc/
/etc/cinder/cinder.conf:#qpid_topology_version=1
/etc/heat/heat.conf:#qpid_topology_version=1
/etc/heat/heat.conf.bak:#qpid_topology_version=1
/etc/ceilometer/ceilometer.conf:#qpid_topology_version=1
/etc/nova/nova.conf:#qpid_topology_version=1

Comment 4 Russell Bryant 2013-11-18 14:08:56 UTC

(In reply to Rami Vaknin from comment #3)
> According to https://bugzilla.redhat.com/show_bug.cgi?id=1003946#c9, this
> bug should be tested by setting qpid_topology_version=2, I can find this
> config param only in heat, ceilometer, nova and cinder.
> 
> I've tried to set it also in neutron.conf, the server starts successsfully
> however I'm not sure whether this conf param is good for all OpenStack
> components, so:
> 
> 1. Can I use the "qpid_topology_version=2" config param to all OpenStack
> components?

I can confirm that this option is supported in Neutron for RHOS 4.0.  Missing from the sample configuration file is a bug, though.


> 2. How can I verify that a component do use qpid federation?

Don't worry about this part.  Federation is still not going to work (according to William Henry).

Just verify that Neutron still works as usual with this option set.  You should notice no difference as a user.

Comment 5 Rami Vaknin 2013-11-19 16:01:44 UTC

I've added the the "qpid_topology_version=2" to nova, heat, ceilometer, cinder and neutron, I ran my neutron-robot test, no regression founds.

Tested on:
rhos 4.0 on rhel 6.5, puddle 2013-11-15.1
openstack-neutron-2013.2-7.el6ost

In addition, I've opened the following bug based on rbryant's comment#4:
https://bugzilla.redhat.com/show_bug.cgi?id=1032151

Ami, would your team run a further nova/heat/ceilometer/cinder tests? or do you prefer to set this bug as Verified relying on this comment?

Comment 6 Ami Jeain 2013-11-19 19:34:05 UTC

verified on latest puddle from Nov 18.
Ran basic nova tests, creating VM, instance, pausing and resuming an instance, creating volume and deleting volume.
Watched log files for error... no errors found

Comment 10 errata-xmlrpc 2013-12-20 00:23:06 UTC

Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

http://rhn.redhat.com/errata/RHEA-2013-1859.html