[openstack-dev] [Openstack-operators] [glance] Remove `run_tests.sh` and `tools/*`

Flavio Percoco flavio at redhat.com
Fri Mar 4 17:41:42 UTC 2016


On 04/03/16 11:59 -0500, Nikhil Komawar wrote:
>I think the hard question to me here is:
>
>Do people care about testing code on system installs vs. virtual env?
>run_tests does that and for some cases when you want to be extra sure
>about your CICD nodes, packaging and upgrades, the problem is solved.
>
>Are packagers using tox to this purpose?

TBH, if you're testing things without venvs and sytemwide, I think it'd be far
easier to just call nosetests/testr directly in your system than calling the
run_tests script.

Some packages don't even ship tests.

Cheers,
Flavio

>On 3/4/16 11:16 AM, Steve Martinelli wrote:
>>
>> The keystone team did the same during Liberty while we were moving
>> towards using oslo.* projects instead of oslo-incubator [0]. We also
>> noticed that they were rarely used, and we did not go through a
>> deprecation process since these are developer tools. We're still
>> finding a few spots in our docs that need updating, but overall it was
>> an easy transition.
>>
>> [0]
>> https://github.com/openstack/keystone/commit/55e9514cbd4e712e2c317335294355cf1596d870
>>
>> stevemar
>>
>> Inactive hide details for Flavio Percoco ---2016/03/04 06:51:47
>> AM---Hey Folks, I'm looking at doing some cleanups in our repo Flavio
>> Percoco ---2016/03/04 06:51:47 AM---Hey Folks, I'm looking at doing
>> some cleanups in our repo and I would like to start by
>>
>> From: Flavio Percoco <flavio at redhat.com>
>> To: openstack-dev at lists.openstack.org
>> Cc: openstack-operators at lists.openstack.org
>> Date: 2016/03/04 06:51 AM
>> Subject: [Openstack-operators] [glance] Remove `run_tests.sh` and
>> `tools/*`
>>
>> ------------------------------------------------------------------------
>>
>>
>>
>> Hey Folks,
>>
>> I'm looking at doing some cleanups in our repo and I would like to
>> start by
>> deprecating the `run_tests` script and the contents in the `tools/` dir.
>>
>> As far as I can tell, no one is using this code - we're not even using
>> it in the
>> gate - as it was broken until recently, I believe. The recommended way
>> to run
>> tests is using `tox` and I believe having this script in the code base
>> misleads
>> new contributors and other users.
>>
>> So, before we do this. I wanted to get feedback from a broader
>> audience and give
>> a heads up to folks that might be using this code.
>>
>> Any objections? Something I'm missing?
>>
>> Flavio
>>
>> --
>> @flaper87
>> Flavio Percoco
>> [attachment "signature.asc" deleted by Steve Martinelli/Toronto/IBM]
>> _______________________________________________
>> OpenStack-operators mailing list
>> OpenStack-operators at lists.openstack.org
>> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-operators
>>
>>
>>
>>
>>
>> __________________________________________________________________________
>> OpenStack Development Mailing List (not for usage questions)
>> Unsubscribe: OpenStack-dev-request at lists.openstack.org?subject:unsubscribe
>> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>
>-- 
>
>Thanks,
>Nikhil
>

-- 
@flaper87
Flavio Percoco
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.openstack.org/pipermail/openstack-dev/attachments/20160304/71380c96/attachment.pgp>


More information about the OpenStack-dev mailing list