FFE for 2 stable backports (QPID reconnect delay)
Greetings, Here I'm asking for a FFE on 2 stable patches. These 2 patches fix the same issue in 2 different projects (nova and cinder). The fix is for having a shorter delay when trying to reconnect to a qpid server. Current delay can get to 60s, which is a really bad scenario for HA deployments. Here are the 2 reviews: https://review.openstack.org/#/c/83686/ https://review.openstack.org/#/c/83689/ Thanks Flavio -- @flaper87 Flavio Percoco
On 28/03/14 14:04 +0100, Flavio Percoco wrote:
Greetings,
Here I'm asking for a FFE on 2 stable patches.
These 2 patches fix the same issue in 2 different projects (nova and cinder). The fix is for having a shorter delay when trying to reconnect to a qpid server. Current delay can get to 60s, which is a really bad scenario for HA deployments.
Here are the 2 reviews:
https://review.openstack.org/#/c/83686/ https://review.openstack.org/#/c/83689/
And 2 more stable exceptions. Same fix 2 more projects (keystone and ceilometer) https://review.openstack.org/#/c/83738/ https://review.openstack.org/#/c/83739/ Neutron's backport was already done! -- @flaper87 Flavio Percoco
I'm okay with all of these going in post-freeze. It's just a minimal oslo code sync that made it into to Neutron pre-freeze and makes sense to update the others as well. Approving now.. Thanks, Adam On Fri, Mar 28, 2014 at 6:19 AM, Flavio Percoco <flavio@redhat.com> wrote:
On 28/03/14 14:04 +0100, Flavio Percoco wrote:
Greetings,
Here I'm asking for a FFE on 2 stable patches.
These 2 patches fix the same issue in 2 different projects (nova and cinder). The fix is for having a shorter delay when trying to reconnect to a qpid server. Current delay can get to 60s, which is a really bad scenario for HA deployments.
Here are the 2 reviews:
https://review.openstack.org/#/c/83686/ https://review.openstack.org/#/c/83689/
And 2 more stable exceptions. Same fix 2 more projects (keystone and ceilometer)
https://review.openstack.org/#/c/83738/ https://review.openstack.org/#/c/83739/
Neutron's backport was already done!
-- @flaper87 Flavio Percoco
_______________________________________________ Openstack-stable-maint mailing list Openstack-stable-maint@lists.openstack.org http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-stable-maint
Actually none of these have been fixed in Icehouse yet. And https://bugs.launchpad.net/oslo.messaging/+bug/1281148 doesnt even have tasks against keystone, ceilometer, etc. I'm in favor of merging these post-freeze but I'd feel more comfortable doing so once they've been fixed in trunk. Added a -2 until then. Thanks, Adam On Fri, Mar 28, 2014 at 6:19 AM, Flavio Percoco <flavio@redhat.com> wrote:
On 28/03/14 14:04 +0100, Flavio Percoco wrote:
Greetings,
Here I'm asking for a FFE on 2 stable patches.
These 2 patches fix the same issue in 2 different projects (nova and cinder). The fix is for having a shorter delay when trying to reconnect to a qpid server. Current delay can get to 60s, which is a really bad scenario for HA deployments.
Here are the 2 reviews:
https://review.openstack.org/#/c/83686/ https://review.openstack.org/#/c/83689/
And 2 more stable exceptions. Same fix 2 more projects (keystone and ceilometer)
https://review.openstack.org/#/c/83738/ https://review.openstack.org/#/c/83739/
Neutron's backport was already done!
-- @flaper87 Flavio Percoco
_______________________________________________ Openstack-stable-maint mailing list Openstack-stable-maint@lists.openstack.org http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-stable-maint
Actually none of these have been fixed in Icehouse yet.
For projects migrated to oslo.messaging library there is nothing to fix, I've marked those tasks as Invalid.
And https://bugs.launchpad.net/oslo.messaging/+bug/1281148 doesnt even have tasks against keystone, ceilometer
etc. I'm in favor of merging these post-freeze but I'd feel more comfortable doing so once they've been fixed in trunk. Added a -2 until
Yes, those should be added as havana-only tasks too. then. Please reconsider, trunk criteria do not apply here. Cheers, Alan
Ack. My bad. :) re-+2'd all but ceilometer and added any missing bug tasks. On Fri, Mar 28, 2014 at 3:15 PM, Alan Pevec <apevec@gmail.com> wrote:
Actually none of these have been fixed in Icehouse yet.
For projects migrated to oslo.messaging library there is nothing to fix, I've marked those tasks as Invalid.
And https://bugs.launchpad.net/oslo.messaging/+bug/1281148 doesnt even have tasks against keystone, ceilometer
Yes, those should be added as havana-only tasks too.
etc. I'm in favor of merging these post-freeze but I'd feel more comfortable doing so once they've been fixed in trunk. Added a -2 until then.
Please reconsider, trunk criteria do not apply here.
Cheers, Alan
participants (3)
-
Adam Gandelman
-
Alan Pevec
-
Flavio Percoco