<html><head><meta http-equiv="Content-Type" content="text/html charset=windows-1252"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;">Thanks Yatin - that is the change I am proposing in my patch<div><br></div><div><br><div style=""><div>On Nov 25, 2013, at 9:09 AM, yatin kumbhare <<a href="mailto:yatinkumbhare@gmail.com">yatinkumbhare@gmail.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div dir="ltr">Hi,<div><br></div><div><a href="http://debugopenstack.blogspot.in/">http://debugopenstack.blogspot.in/</a><br></div><div><br></div><div>I have done Openstack remote debugging with eclipse and pydev.</div><div>
<br></div><div>only change is to exclude python thread library from monkey patch at service start up.</div><div><br></div><div>Regards,</div><div>Yatin</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">
On Mon, Nov 25, 2013 at 9:10 PM, Russell Bryant <span dir="ltr"><<a href="mailto:rbryant@redhat.com" target="_blank">rbryant@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5">On 11/25/2013 10:28 AM, Tracy Jones wrote:<br>
> Hi Folks - i am trying to add a patch to enable remote debugging in the<br>
> nova services. I can make this work very simply, but it requires a<br>
> change to monkey_patching - i.e.<br>
><br>
> eventlet.monkey_patch(os=False, select=True, socket=True, thread=False,<br>
> time=True, psycopg=True)<br>
><br>
> I’m working with some folks from the debugger vendor (pycharm) on why<br>
> this is needed. However - i’ve been using it with nova-compute for a<br>
> month or so and do not see any issues which changing the<br>
> monkey-patching. Since this is done only when someone wants to use the<br>
> debugger - is making this change so bad?<br>
><br>
> <a href="https://review.openstack.org/56287" target="_blank">https://review.openstack.org/56287</a><br>
<br>
</div></div>Last I looked at the review, it wasn't known that thread=False was<br>
specifically what was needed IIRC> That's good progress and is a bit<br>
less surprising than the change before.<br>
<br>
I suppose if the options that enable this come with a giant warning,<br>
it'd be OK. Something like:<br>
<br>
WARNING: Using this option changes how Nova uses the eventlet<br>
library to support async IO. This could result in failures that do<br>
not occur under normal opreation. Use at your own risk.<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Russell Bryant<br>
<br>
_______________________________________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</font></span></blockquote></div><br></div>
</blockquote></div><br></div></body></html>