<div dir="ltr">Thanks for the insight into lockutils-wrapper usage Ihar.  I had worked on getting the coverage target for VPNaaS repo working, and then have been working on a coverage target for functional testing. As part of that, I pulled the use of lockutils-wrapper.<div><br></div><div>If you can review it, I'd appreciate it very much:  <a href="https://review.openstack.org/#/c/155889/2">https://review.openstack.org/#/c/155889/2</a></div><div><br></div><div>I was thinking of doing this same coverage testing for FWaaS and LBaaS, if you want me to remove lockutils-wrapper there too, let me know, as I was planning on working on these today...</div><div><br></div><div>Regards,</div><div>PCM</div><div><br></div></div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature"><div dir="ltr"><div><div dir="ltr">PCM (Paul Michali)</div><div dir="ltr"><div><br></div><div>IRC............ pc_m (<a href="http://irc.freenode.com" target="_blank">irc.freenode.com</a>)<br>Twitter....... @pmichali</div><div><br></div></div></div></div></div></div>
<br><div class="gmail_quote">On Mon, Feb 16, 2015 at 8:16 AM, 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">-----BEGIN PGP SIGNED MESSAGE-----<br>
Hash: SHA1<br>
<span class=""><br>
On 02/13/2015 11:13 PM, Paul Michali wrote:<br>
> I see that in tox.ini, several commands have lockutils-wrapper<br>
> prefix on them in the neutron-vpnaas repo. Seems like this was<br>
> added as part of commit 88e2d801 for Migration to<br>
> oslo.concurrency.<br>
<br>
</span>Those would be interesting for unit tests only. And now that neutron<br>
BaseTestCase uses proper fixture [1], we can just remove the wrapper<br>
call from all *aas repos.<br>
<br>
That was actually one of the things I was going to do as a oslo<br>
liaison during Kilo [2] (see the 1st entry in the todo list.) But if<br>
you want to go with this before I reach this cleanup, I will be glad<br>
to review the changes. :)<br>
<br>
[1]:<br>
<a href="http://git.openstack.org/cgit/openstack/neutron/tree/neutron/tests/base.py#n89" target="_blank">http://git.openstack.org/cgit/openstack/neutron/tree/neutron/tests/base.py#n89</a><br>
[2]:<br>
<a href="http://lists.openstack.org/pipermail/openstack-dev/2015-January/054753.html" target="_blank">http://lists.openstack.org/pipermail/openstack-dev/2015-January/054753.html</a><br>
<br>
/Ihar<br>
-----BEGIN PGP SIGNATURE-----<br>
Version: GnuPG v1<br>
<br>
iQEcBAEBAgAGBQJU4e21AAoJEC5aWaUY1u57zKcH/iZ3SFi3BmyYaEwch5jipbzw<br>
Byxn2QxyjRNcQ6m/dr6ihpvXS2bIo75mrNajc+mdnKTqAdXebceSQRPAw4EX3c9r<br>
qtlaGzSrBqmSiyl/YnbqUiUf2zcXpFIpiTJswbdhv10P5Gi/Q64m6d+ipQsIUaMP<br>
4sY/0sjAV5Gn9cpkBZn9LY1/CrWnP7eqFMBYvFTsyEpGHdgJ4heAx2dLCqY2DE9H<br>
bVFexZK1yMqLzEIwmHtzSyifcFZkC39fa6bsxCVlLkbfU7+KC56FOOHARsjf+grd<br>
ReQuGH4QIS0aTMkrd7qmJRkaK7BudkX1yfOY68jsYSwrpKoia7pMZ+tbPosfUbk=<br>
=+HKf<br>
-----END PGP SIGNATURE-----<br>
<div class="HOEnZb"><div class="h5"><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>