Open Stack

Thanks Yatin - that is the change I am proposing in my patch


On Nov 25, 2013, at 9:09 AM, yatin kumbhare <yatinkumbhare at gmail.com> wrote:

> Hi,
> 
> http://debugopenstack.blogspot.in/
> 
> I have done Openstack remote debugging with eclipse and pydev.
> 
> only change is to exclude python thread library from monkey patch at service start up.
> 
> Regards,
> Yatin
> 
> 
> On Mon, Nov 25, 2013 at 9:10 PM, Russell Bryant <rbryant at redhat.com> wrote:
> On 11/25/2013 10:28 AM, Tracy Jones wrote:
> > Hi Folks - i am trying to add a patch to enable remote debugging in the
> > nova services.  I can make this work very simply, but it requires a
> > change to monkey_patching - i.e.
> >
> >     eventlet.monkey_patch(os=False, select=True, socket=True, thread=False,
> >                       time=True, psycopg=True)
> >
> > I’m working with some folks from the debugger vendor (pycharm) on why
> > this is needed.   However - i’ve been using it with nova-compute for a
> > month or so and do not see any issues which changing the
> > monkey-patching.  Since this is done only when someone wants to use the
> > debugger - is making this change so bad?
> >
> >  https://review.openstack.org/56287
> 
> Last I looked at the review, it wasn't known that thread=False was
> specifically what was needed IIRC>  That's good progress and is a bit
> less surprising than the change before.
> 
> I suppose if the options that enable this come with a giant warning,
> it'd be OK.  Something like:
> 
>   WARNING: Using this option changes how Nova uses the eventlet
>   library to support async IO. This could result in failures that do
>   not occur under normal opreation. Use at your own risk.
> 
> --
> Russell Bryant
> 
> _______________________________________________
> OpenStack-dev mailing list
> OpenStack-dev at lists.openstack.org
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
> 

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstack.org/pipermail/openstack-dev/attachments/20131125/265f269c/attachment.html>