<div dir="ltr"><div><div><div>As for downstream you can do whatever you want, but it looks like this issue should be solved in upstream. I mean if "tests" directory is located at the top level of the repo, no one will use it.<br></div><div>Also, setuptools supports `exclude` option which should solve the issue as well.<br></div><br></div></div><div><div><div><div><br></div></div></div></div></div><div class="gmail_extra"><br><div class="gmail_quote">2018-02-19 17:41 GMT+02:00 Michael Bayer <span dir="ltr"><<a href="mailto:mbayer@redhat.com" target="_blank">mbayer@redhat.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Mon, Feb 19, 2018 at 10:39 AM, Andrey Kurilin <<a href="mailto:andr.kurilin@gmail.com">andr.kurilin@gmail.com</a>> wrote:<br>
> Can someone explain me the reason for including "tests" module into<br>
> packages?<br>
<br>
</span>the "tests" module should not be inside packages.   Downstream we have<br>
CI running Cinder's test suite against packaged dependencies, which<br>
fails because we don't package oslo_db.tests.<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
><br>
><br>
> 2018-02-19 17:00 GMT+02:00 Michael Bayer <<a href="mailto:mbayer@redhat.com">mbayer@redhat.com</a>>:<br>
>><br>
>> Hi list -<br>
>><br>
>> Apparently Cinder was misled by my deprecations within the<br>
>> oslo_db.sqlalchemy.test_base package of DbFixture and DbTestCase, and<br>
>> in <a href="https://review.openstack.org/#/c/522290/" rel="noreferrer" target="_blank">https://review.openstack.org/#<wbr>/c/522290/</a> the assumption was made<br>
>> that these should be imported from oslo_db.tests.sqlalchemy.    This<br>
>> is an immense mistake on my part that I did not expect people to go<br>
>> looking for the same names elsewhere in private packages and now we<br>
>> have a serious downstream issue as these modules are not packaged, as<br>
>> well as the possibility that the oslo_db.tests. package is now locked<br>
>> in time and I have to add deprecations there also.<br>
>><br>
>> If anyone knows of projects (or feels like helping me search) that are<br>
>> importing *anything* from oslo_db.tests these must be reverted ASAP.<br>
>><br>
>> ______________________________<wbr>______________________________<wbr>______________<br>
>> OpenStack Development Mailing List (not for usage questions)<br>
>> Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.<wbr>openstack.org?subject:<wbr>unsubscribe</a><br>
>> <a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/<wbr>cgi-bin/mailman/listinfo/<wbr>openstack-dev</a><br>
><br>
><br>
><br>
><br>
> --<br>
> Best regards,<br>
> Andrey Kurilin.<br>
><br>
> ______________________________<wbr>______________________________<wbr>______________<br>
> OpenStack Development Mailing List (not for usage questions)<br>
> Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.<wbr>openstack.org?subject:<wbr>unsubscribe</a><br>
> <a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/<wbr>cgi-bin/mailman/listinfo/<wbr>openstack-dev</a><br>
><br>
<br>
______________________________<wbr>______________________________<wbr>______________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.<wbr>openstack.org?subject:<wbr>unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/<wbr>cgi-bin/mailman/listinfo/<wbr>openstack-dev</a><br>
</div></div></blockquote></div><br><br clear="all"><br>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr">Best regards,<br>Andrey Kurilin.<br></div></div>
</div>