That fix was put in to get Keystone to pass the python26 gate on centos, so reverting this change is going to fail Jenkins. Sorry for not looking close enough yesterday to see that smokestack hadn't passed (smokestack has usually finished before Jenkins). I'll investigate a fix for the swift issue. - Brant On Fri, May 10, 2013 at 8:06 AM, Dan Prince <dprince at redhat.com> wrote: > > > ----- Original Message ----- > > From: "Thierry Carrez" <thierry at openstack.org> > > To: openstack-dev at lists.openstack.org > > Sent: Friday, May 10, 2013 8:28:30 AM > > Subject: Re: [openstack-dev] revert keystone eventlet changes? > > > > Dan Prince wrote: > > > We made a change to keystone eventlet last night that seems to have > ground > > > SmokeStack to a halt: > > > > > > https://review.openstack.org/#/c/28785/ > > > https://bugs.launchpad.net/keystone/+bug/1178646 > > > > > > Could we get a speedy revert on that patch until we get this figured > out? > > > > There is also a comment on the original review that observes the same > > failure: https://review.openstack.org/#/c/28737/ > > > > Sounds like it should be reverted ASAP, and we need some tests to cover > > that in the gate. > > > > Dan: any idea why SmokeStack was happy with it on the review ? > > ttx: From what I can see only patchset 3 ran fine in SmokeStack. Patchset > 4 did not get a +1 verified (and I didn't get around to approving the > negative result so it would post a -1). > > Dan > > > > > -- > > Thierry Carrez (ttx) > > Release Manager, OpenStack > > > > _______________________________________________ > > OpenStack-dev mailing list > > OpenStack-dev at lists.openstack.org > > http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev > > > > _______________________________________________ > OpenStack-dev mailing list > OpenStack-dev at lists.openstack.org > http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev > -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.openstack.org/pipermail/openstack-dev/attachments/20130510/f721ea49/attachment.html>