[openstack-dev] revert keystone eventlet changes?

Dan Prince dprince at redhat.com
Fri May 10 15:18:01 UTC 2013


After talking on IRC the fix (for today) seems to be:

Set EVENTLET_NO_GREENDNS=yes in tox.ini. (https://review.openstack.org/#/c/28802/)

And then we can do a nice clean (and safe) revert of the breaking eventlet patch.

Thanks for the help in working through this.

Dan

----- Original Message -----
> From: "Brant Knudson" <blk at acm.org>
> To: "OpenStack Development Mailing List" <openstack-dev at lists.openstack.org>
> Sent: Friday, May 10, 2013 9:19:32 AM
> Subject: Re: [openstack-dev] revert keystone eventlet changes?
> 
> 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
> >
> 
> _______________________________________________
> OpenStack-dev mailing list
> OpenStack-dev at lists.openstack.org
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
> 



More information about the OpenStack-dev mailing list