Merge lp:~xianghui/ubuntu/trusty/neutron/lp1393391 into lp:ubuntu/trusty-proposed/neutron

Proposed by Xiang Hui
Status: Merged
Merge reported by: Marc Deslauriers
Merged at revision: not available
Proposed branch: lp:~xianghui/ubuntu/trusty/neutron/lp1393391
Merge into: lp:ubuntu/trusty-proposed/neutron
Diff against target: 66 lines (+42/-0)
3 files modified
debian/changelog (+9/-0)
debian/patches/fix-neutron-agent-fanout-queue-not-found-loop.patch (+32/-0)
debian/patches/series (+1/-0)
To merge this branch: bzr merge lp:~xianghui/ubuntu/trusty/neutron/lp1393391
Reviewer Review Type Date Requested Status
Ubuntu Development Team Pending
Review via email: mp+280943@code.launchpad.net

Description of the change

  * neutron-openvswitch-agent stuck on 'q-agent-notifier-port-update_fanout-x'
    is NOT_FOUND exception, this bug caused RabbitMQ HA(active + active) failover
    in Icehouse not work with large scaled nova compute nodes. (LP: #1393391):
    - d/p/fix-neutron-agent-fanout-queue-not-found-loop.patch

To post a comment you must log in.
Revision history for this message
Serge Hallyn (serge-hallyn) wrote :

Hi,

Could you please add DEP3 (http://dep.debian.net/deps/dep3/) headers to the patch?

37. By Xiang Hui

Add dep3

Revision history for this message
Xiang Hui (xianghui) wrote :

@Serge,

Thanks for looking at it, added the dep3 header.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== added file '.pc/disable-udev-tests.patch/.timestamp'
=== added file '.pc/fix-quantum-configuration.patch/.timestamp'
=== added file '.pc/skip-ipv6-tests.patch/.timestamp'
=== added file '.pc/use-concurrency.patch/.timestamp'
=== modified file 'debian/changelog'
--- debian/changelog 2015-06-22 10:14:52 +0000
+++ debian/changelog 2016-02-11 08:35:25 +0000
@@ -1,3 +1,12 @@
1neutron (1:2014.1.5-0ubuntu2) trusty; urgency=medium
2
3 * neutron-openvswitch-agent stuck on 'q-agent-notifier-port-update_fanout-x'
4 is NOT_FOUND exception, this bug caused RabbitMQ HA(active + active) failover
5 in Icehouse not work with large scaled nova compute nodes. (LP: #1393391):
6 - d/p/fix-neutron-agent-fanout-queue-not-found-loop.patch
7
8 -- Hui Xiang <hui.xiang@canonical.com> Mon, 25 Jan 2016 15:25:13 +0800
9
1neutron (1:2014.1.5-0ubuntu1) trusty; urgency=medium10neutron (1:2014.1.5-0ubuntu1) trusty; urgency=medium
211
3 * Resynchronize with stable/icehouse (877df58) (LP: #1467533):12 * Resynchronize with stable/icehouse (877df58) (LP: #1467533):
413
=== added file 'debian/patches/fix-neutron-agent-fanout-queue-not-found-loop.patch'
--- debian/patches/fix-neutron-agent-fanout-queue-not-found-loop.patch 1970-01-01 00:00:00 +0000
+++ debian/patches/fix-neutron-agent-fanout-queue-not-found-loop.patch 2016-02-11 08:35:25 +0000
@@ -0,0 +1,32 @@
1--- debian/patches/fix-neutron-agent-fanout-queue-not-found-loop.patch 2016-01-25 15:24:39.000000000 +0800
2+++ debian/patches/fix-neutron-agent-fanout-queue-not-found-loop.patch 1970-01-01 08:00:00.000000000 +0800
3@@ -1,29 +0,0 @@
4-Description: Fix-neutron-agent-fanout-queue-not-found-loop
5-
6- neutron-openvswitch-agent stuck on 'q-agent-notifier-port-update_fanout-x'
7- is NOT_FOUND exception, this bug caused RabbitMQ HA(active + active) failover
8- in Icehouse not work with large scaled nova compute nodes.
9-
10-Author: hui.xiang@canonical.com
11-Origin: other
12-Bug: https://bugs.launchpad.net/neutron/+bug/1393391
13----
14-This patch header follows DEP-3: http://dep.debian.net/deps/dep3/
15---- a/neutron/openstack/common/rpc/impl_kombu.py
16-+++ b/neutron/openstack/common/rpc/impl_kombu.py
17-@@ -792,6 +792,15 @@
18- self.consume()
19- except greenlet.GreenletExit:
20- return
21-+ except Exception as e:
22-+ if e.code == 404:
23-+ # Redeclare queue if it is deleted due to HA cluster
24-+ # disconnection, LP#1393391
25-+ LOG.warn(_("Queue NOT FOUND : %s"), e)
26-+ self.channel = self.connection.channel()
27-+ for consumer in self.consumers:
28-+ consumer.reconnect(self.channel)
29-+
30- if self.consumer_thread is None:
31- self.consumer_thread = eventlet.spawn(_consumer_thread)
32- return self.consumer_thread
033
=== modified file 'debian/patches/series'
--- debian/patches/series 2014-08-08 08:30:52 +0000
+++ debian/patches/series 2016-02-11 08:35:25 +0000
@@ -2,3 +2,4 @@
2disable-udev-tests.patch2disable-udev-tests.patch
3skip-ipv6-tests.patch3skip-ipv6-tests.patch
4use-concurrency.patch4use-concurrency.patch
5fix-neutron-agent-fanout-queue-not-found-loop.patch

Subscribers

People subscribed via source and target branches

to all changes: