On 13-09-2010 09:29, Vishvananda Ishaya wrote:
> Ah yes.  I do remember realizing that the destroy was in the wrong place at 
> some point.  In the orm branch I moved it until after the callbacks were done:
> 
>         for _index in xrange(total_slots):
>             volume_id = self._create_volume()
>             d = self.volume.create_volume(self.context, volume_id)
>             d.addCallback(_check)
>             d.addErrback(self.fail)
>             deferreds.append(d)
>         yield defer.DeferredList(deferreds)
>         for volume_id in volume_ids:
>             self.volume.delete_volume(self.context, volume_id)
> 
> Strange that it only fails some of the time in the old version, because that 
> destroy was definitely in the wrong place.

Well, it only failed it it happened to get an identical shelf/blade tuple.

-- 
Soren Hansen
Ubuntu Developer    http://www.ubuntu.com/
OpenStack Developer http://www.openstack.org/

_______________________________________________
Mailing list: https://launchpad.net/~nova
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~nova
More help   : https://help.launchpad.net/ListHelp

Reply via email to