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
1=== added file '.pc/disable-udev-tests.patch/.timestamp'
2=== added file '.pc/fix-quantum-configuration.patch/.timestamp'
3=== added file '.pc/skip-ipv6-tests.patch/.timestamp'
4=== added file '.pc/use-concurrency.patch/.timestamp'
5=== modified file 'debian/changelog'
6--- debian/changelog 2015-06-22 10:14:52 +0000
7+++ debian/changelog 2016-02-11 08:35:25 +0000
8@@ -1,3 +1,12 @@
9+neutron (1:2014.1.5-0ubuntu2) trusty; urgency=medium
10+
11+ * neutron-openvswitch-agent stuck on 'q-agent-notifier-port-update_fanout-x'
12+ is NOT_FOUND exception, this bug caused RabbitMQ HA(active + active) failover
13+ in Icehouse not work with large scaled nova compute nodes. (LP: #1393391):
14+ - d/p/fix-neutron-agent-fanout-queue-not-found-loop.patch
15+
16+ -- Hui Xiang <hui.xiang@canonical.com> Mon, 25 Jan 2016 15:25:13 +0800
17+
18 neutron (1:2014.1.5-0ubuntu1) trusty; urgency=medium
19
20 * Resynchronize with stable/icehouse (877df58) (LP: #1467533):
21
22=== added file 'debian/patches/fix-neutron-agent-fanout-queue-not-found-loop.patch'
23--- debian/patches/fix-neutron-agent-fanout-queue-not-found-loop.patch 1970-01-01 00:00:00 +0000
24+++ debian/patches/fix-neutron-agent-fanout-queue-not-found-loop.patch 2016-02-11 08:35:25 +0000
25@@ -0,0 +1,32 @@
26+--- debian/patches/fix-neutron-agent-fanout-queue-not-found-loop.patch 2016-01-25 15:24:39.000000000 +0800
27++++ debian/patches/fix-neutron-agent-fanout-queue-not-found-loop.patch 1970-01-01 08:00:00.000000000 +0800
28+@@ -1,29 +0,0 @@
29+-Description: Fix-neutron-agent-fanout-queue-not-found-loop
30+-
31+- neutron-openvswitch-agent stuck on 'q-agent-notifier-port-update_fanout-x'
32+- is NOT_FOUND exception, this bug caused RabbitMQ HA(active + active) failover
33+- in Icehouse not work with large scaled nova compute nodes.
34+-
35+-Author: hui.xiang@canonical.com
36+-Origin: other
37+-Bug: https://bugs.launchpad.net/neutron/+bug/1393391
38+----
39+-This patch header follows DEP-3: http://dep.debian.net/deps/dep3/
40+---- a/neutron/openstack/common/rpc/impl_kombu.py
41+-+++ b/neutron/openstack/common/rpc/impl_kombu.py
42+-@@ -792,6 +792,15 @@
43+- self.consume()
44+- except greenlet.GreenletExit:
45+- return
46+-+ except Exception as e:
47+-+ if e.code == 404:
48+-+ # Redeclare queue if it is deleted due to HA cluster
49+-+ # disconnection, LP#1393391
50+-+ LOG.warn(_("Queue NOT FOUND : %s"), e)
51+-+ self.channel = self.connection.channel()
52+-+ for consumer in self.consumers:
53+-+ consumer.reconnect(self.channel)
54+-+
55+- if self.consumer_thread is None:
56+- self.consumer_thread = eventlet.spawn(_consumer_thread)
57+- return self.consumer_thread
58
59=== modified file 'debian/patches/series'
60--- debian/patches/series 2014-08-08 08:30:52 +0000
61+++ debian/patches/series 2016-02-11 08:35:25 +0000
62@@ -2,3 +2,4 @@
63 disable-udev-tests.patch
64 skip-ipv6-tests.patch
65 use-concurrency.patch
66+fix-neutron-agent-fanout-queue-not-found-loop.patch

Subscribers

People subscribed via source and target branches

to all changes: