<div dir="ltr"><div><br></div>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</div>
<div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, May 10, 2013 at 8:06 AM, Dan Prince <span dir="ltr"><<a href="mailto:dprince@redhat.com" target="_blank">dprince@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im"><br>
<br>
----- Original Message -----<br>
> From: "Thierry Carrez" <<a href="mailto:thierry@openstack.org">thierry@openstack.org</a>><br>
> To: <a href="mailto:openstack-dev@lists.openstack.org">openstack-dev@lists.openstack.org</a><br>
> Sent: Friday, May 10, 2013 8:28:30 AM<br>
> Subject: Re: [openstack-dev] revert keystone eventlet changes?<br>
><br>
> Dan Prince wrote:<br>
> > We made a change to keystone eventlet last night that seems to have ground<br>
> > SmokeStack to a halt:<br>
> ><br>
> > <a href="https://review.openstack.org/#/c/28785/" target="_blank">https://review.openstack.org/#/c/28785/</a><br>
> > <a href="https://bugs.launchpad.net/keystone/+bug/1178646" target="_blank">https://bugs.launchpad.net/keystone/+bug/1178646</a><br>
> ><br>
> > Could we get a speedy revert on that patch until we get this figured out?<br>
><br>
> There is also a comment on the original review that observes the same<br>
> failure: <a href="https://review.openstack.org/#/c/28737/" target="_blank">https://review.openstack.org/#/c/28737/</a><br>
><br>
> Sounds like it should be reverted ASAP, and we need some tests to cover<br>
> that in the gate.<br>
><br>
> Dan: any idea why SmokeStack was happy with it on the review ?<br>
<br>
</div>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).<br>
<span class="HOEnZb"><font color="#888888"><br>
Dan<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
><br>
> --<br>
> Thierry Carrez (ttx)<br>
> Release Manager, OpenStack<br>
><br>
> _______________________________________________<br>
> OpenStack-dev mailing list<br>
> <a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a><br>
> <a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
><br>
<br>
_______________________________________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</div></div></blockquote></div><br></div>