Code review comment for lp:~termie/nova/eventlet_merge

Revision history for this message
termie (termie) wrote :

> Some notes on trying to get this branch to run. I'll update again just to be
> sure I have the latest.
>
> 1. logging is console only by default (assuming run as user). Needed to update
> service.Service to get file logging. Not a biggie.
>
> 2. Fails when trying to run a xenserver instance:
> http://paste.openstack.org/show/303/

Fixed the improt error, would be appreciative if you can test whether it runs, I currently don't have any way to test the xen stuff.

> 3. Why does nova-objectstore have to be run the old way and not integrated
> with nova-combined?

It is very reliant on twisted's implementations of things and as it is being deprecated I didn't want to spend time porting it over to eventlet.

>
> I'll see if I can fix #2 in the meanwhile.

« Back to merge proposal