<br><br><div class="gmail_quote">On Fri, Mar 2, 2012 at 10:35 AM, Joshua Harlow <span dir="ltr"><<a href="mailto:harlowja@yahoo-inc.com">harlowja@yahoo-inc.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">




<div>
<font face="Calibri, Verdana, Helvetica, Arial"><span style="font-size:11pt">Does anyone else feel that the following seems really “dirty”, or is it just me.<br></span></font></div></blockquote><div><br></div><div>Any feeling of dirtiness is just due to it being called "sleep," all you are doing is yielding control to allow another co-routine to schedule itself. Blocking code is still blocking code, you have to give it some break points if you are going to run a loop that waits on something else.</div>
<div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><font face="Calibri, Verdana, Helvetica, Arial"><span style="font-size:11pt">
<br>
“adding a few sleep(0) calls in various places in the<div class="im"><br>
Nova codebase (as was recently added in the _sync_power_states()<br>
periodic task) is an easy and simple win with pretty much no ill<br></div>
side-effects. :)”<br>
<br>
Dirty in that it feels like there is something wrong from a design point of view.<br>
Sprinkling “sleep(0)” seems like its a band-aid on a larger problem imho. <br>
But that’s just my gut feeling.<br>
<br>
<i>:-(<br>
</i><div><div class="h5"><br>
On 3/2/12 8:26 AM, "Armando Migliaccio" <<a href="http://Armando.Migliaccio@eu.citrix.com" target="_blank">Armando.Migliaccio@eu.citrix.com</a>> wrote:<br>
<br>
</div></div></span></font><div><div class="h5"><blockquote><font face="Calibri, Verdana, Helvetica, Arial"><span style="font-size:11pt">I knew you'd say that :P<br>
<br>
There you go: <a href="https://bugs.launchpad.net/nova/+bug/944145" target="_blank">https://bugs.launchpad.net/nova/+bug/944145</a><br>
<br>
Cheers,<br>
Armando<br>
<br>
> -----Original Message-----<br>
> From: Jay Pipes [<a href="mailto:jaypipes@gmail.com" target="_blank">mailto:jaypipes@gmail.com</a>]<br>
> Sent: 02 March 2012 16:22<br>
> To: Armando Migliaccio<br>
> Cc: <a href="http://openstack@lists.launchpad.net" target="_blank">openstack@lists.launchpad.net</a><br>
> Subject: Re: [Openstack] eventlet weirdness<br>
><br>
> On 03/02/2012 10:52 AM, Armando Migliaccio wrote:<br>
> > I'd be cautious to say that no ill side-effects were introduced. I found a<br>
> race condition right in the middle of sync_power_states, which I assume was<br>
> exposed by "breaking" the task deliberately.<br>
><br>
> Such a party-pooper! ;)<br>
><br>
> Got a link to the bug report for me?<br>
><br>
> Thanks!<br>
> -jay<br>
<br>
_______________________________________________<br>
Mailing list: <a href="https://launchpad.net/~openstack" target="_blank">https://launchpad.net/~openstack</a><br>
Post to     : <a href="http://openstack@lists.launchpad.net" target="_blank">openstack@lists.launchpad.net</a><br>
Unsubscribe : <a href="https://launchpad.net/~openstack" target="_blank">https://launchpad.net/~openstack</a><br>
More help   : <a href="https://help.launchpad.net/ListHelp" target="_blank">https://help.launchpad.net/ListHelp</a><br>
<br>
</span></font></blockquote>
</div></div></div>


<br>_______________________________________________<br>
Mailing list: <a href="https://launchpad.net/~openstack" target="_blank">https://launchpad.net/~openstack</a><br>
Post to     : <a href="mailto:openstack@lists.launchpad.net">openstack@lists.launchpad.net</a><br>
Unsubscribe : <a href="https://launchpad.net/~openstack" target="_blank">https://launchpad.net/~openstack</a><br>
More help   : <a href="https://help.launchpad.net/ListHelp" target="_blank">https://help.launchpad.net/ListHelp</a><br>
<br></blockquote></div><br>