<div dir="ltr">My opinions inline.<br><div class="gmail_extra"><br><div class="gmail_quote">On 17 February 2015 at 16:04, Ihar Hrachyshka <span dir="ltr"><<a href="mailto:ihrachys@redhat.com" target="_blank">ihrachys@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">-----BEGIN PGP SIGNED MESSAGE-----<br>
Hash: SHA1<br>
<br>
</span>Hi,<br>
<br>
response was huge so far :) so to add more traction, I have a question<br>
for everyone. Let's assume we want to move entry points for all<br>
services and agents into neutron/cmd/... If so,<br></blockquote><div><br></div><div>I don't have anything again this assumption. Also it seems other projects are already doing it this way so there is no "divergence" issue here. </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
- - Do we want all existing tools stored in the path to be monkey<br>
patched too? I would say 'yes', to make sure we run our unit tests in<br>
the same environment as in real life;<br></blockquote><div><br></div><div>I say yes but mildly here. If you're referring to the tools used for running flake8 or unit tests in theory it should not really matter whether they're patched or not. However, I'm aware of unit tests which spawn eventlet threadpools, so it's definitely better to ensure all these tools are patched.</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
- - Which parts of services we want to see there? Should they include<br>
any real main() or register_options() code, or should they be just a<br>
wrappers to call actual main() located somewhere in other parts of the<br>
tree? I lean toward leaving just a one liner main() under<br>
neutron/cmd/... that calls to 'real' main() located in a different<br>
place in the tree.<br></blockquote><div><br></div><div>My vote is for the one-liner.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Comments?<br>
<br>
/Ihar<br>
<span class=""><br>
<br>
On 02/13/2015 04:37 PM, Ihar Hrachyshka wrote:<br>
> On 02/13/2015 02:33 AM, Kevin Benton wrote:<br>
>> Why did the services fail with the stdlib patched? Are they<br>
>> incompatible with eventlet?<br>
><br>
> It's not like *service entry points* are not ready for neutron.* to<br>
> be monkey patched, but tools around it (flake8 that imports<br>
> neutron.hacking.checks, setuptools that import hooks from<br>
> neutron.hooks etc). It's also my belief that base library should<br>
> not be monkey patched not to put additional assumptions onto<br>
> consumers.<br>
><br>
> (Though I believe that all the code in the tree should be monkey<br>
> patched, including those agents that currently run without the<br>
> library patched - for consistency and to reflect the same test<br>
> environment for unit tests that will be patched from<br>
> neutron/tests/__init__.py).<br>
><br>
> /Ihar<br>
><br>
</span><span class="">> __________________________________________________________________________<br>
><br>
><br>
OpenStack Development Mailing List (not for usage questions)<br>
> Unsubscribe:<br>
> <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</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>
><br>
</span><span class="">-----BEGIN PGP SIGNATURE-----<br>
Version: GnuPG v1<br>
<br>
</span>iQEcBAEBAgAGBQJU41iWAAoJEC5aWaUY1u57zBYIAIuobIYMZ1NJmm+7sV+NW6LS<br>
ZS4PNKlwcYRrdfArGliUq7GLVi/ZRNPNgilF9RIJXQAiOXEc6PmKqpKw1JnwkQ7v<br>
l3/NeciYmkMhSNRv1vIrOBHegAYx9Js6o2lOBCF7BFKIpu88OsC95oobcLGtcrYU<br>
BxoBUM7DYvHssDhRp3NujNbyMrRkg4roer7+4qGE3a449tv4xViTcoUWg5MoNalY<br>
vD1ld/Gg8LfKPt7v7FbF2YnHkMG+UJSk47rRd0yv9KGABS69TkNuvJXeJ14sgw0O<br>
YqIY3oMO0nza+T8tdQGTrYv9N4rWOMFsJMyrOLIvoUyq526QQZ/K7Hrijj1IQjE=<br>
=ZtVP<br>
<div class="HOEnZb"><div class="h5">-----END PGP SIGNATURE-----<br>
<br>
__________________________________________________________________________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</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>
</div></div></blockquote></div><br></div></div>