On Fri, Mar 2, 2012 at 10:35 AM, Joshua Harlow <harlo...@yahoo-inc.com>wrote:
> Does anyone else feel that the following seems really “dirty”, or is it > just me. > 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. > > “adding a few sleep(0) calls in various places in the > > Nova codebase (as was recently added in the _sync_power_states() > periodic task) is an easy and simple win with pretty much no ill > side-effects. :)” > > Dirty in that it feels like there is something wrong from a design point > of view. > Sprinkling “sleep(0)” seems like its a band-aid on a larger problem imho. > But that’s just my gut feeling. > > *:-( > * > > On 3/2/12 8:26 AM, "Armando Migliaccio" <armando.migliac...@eu.citrix.com> > wrote: > > I knew you'd say that :P > > There you go: https://bugs.launchpad.net/nova/+bug/944145 > > Cheers, > Armando > > > -----Original Message----- > > From: Jay Pipes [mailto:jaypi...@gmail.com <jaypi...@gmail.com>] > > Sent: 02 March 2012 16:22 > > To: Armando Migliaccio > > Cc: openstack@lists.launchpad.net > > Subject: Re: [Openstack] eventlet weirdness > > > > On 03/02/2012 10:52 AM, Armando Migliaccio wrote: > > > I'd be cautious to say that no ill side-effects were introduced. I > found a > > race condition right in the middle of sync_power_states, which I assume > was > > exposed by "breaking" the task deliberately. > > > > Such a party-pooper! ;) > > > > Got a link to the bug report for me? > > > > Thanks! > > -jay > > _______________________________________________ > Mailing list: https://launchpad.net/~openstack > Post to : openstack@lists.launchpad.net > Unsubscribe : https://launchpad.net/~openstack > More help : https://help.launchpad.net/ListHelp > > > _______________________________________________ > Mailing list: https://launchpad.net/~openstack > Post to : openstack@lists.launchpad.net > Unsubscribe : https://launchpad.net/~openstack > More help : https://help.launchpad.net/ListHelp > >
_______________________________________________ Mailing list: https://launchpad.net/~openstack Post to : openstack@lists.launchpad.net Unsubscribe : https://launchpad.net/~openstack More help : https://help.launchpad.net/ListHelp
