<div dir="ltr">+1. Especially if you don't use "fake" mode of Nailgun, I don't know why would you even be copying legacy code to the new repo..<div><br></div><div>Thanks!<br><div><div class="gmail_quote"><div dir="ltr"><br></div><div dir="ltr">On Mon, Aug 17, 2015 at 6:25 AM Jay Pipes <<a href="mailto:jaypipes@gmail.com" target="_blank">jaypipes@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 08/17/2015 08:50 AM, Roman Prykhodchenko wrote:<br>
> Hi Fuelers!<br>
><br>
> I was working on enabling Python tests in Fuel Client to run on<br>
> OpenStack CI and I figured out that we actually have a piece of<br>
> legacy code which can be removed now. That piece is run_tests.sh<br>
> file. For those who’s not aware, that script allows to run different<br>
> tests under different environments. I don’t know how it was a<br>
> thousand years ago when I was not involved to Fuel project, but the<br>
> situation at this particular moment looks like that:<br>
><br>
> - Tests are actually orchestrated by tox - The biggest job of<br>
> run_tests.sh is to translate its options to tox’es options - The only<br>
> useful job of run_tests.sh is to start Nailgun correctly for<br>
> functional tests<br>
><br>
> As you can see the profit of that script is tiny. However, the<br>
> problems it brings are pretty much big and looks as follows:<br>
><br>
> - It is unstable — tiniest changes to tests require big changes to<br>
> the script - The CLI it provides is confusing - Working on that file<br>
> looks like doing the same job that is already done in tox - Among the<br>
> active Fuel Client’s community there are only a few guys who are<br>
> proficient in bash enough, to support that script effectively<br>
><br>
><br>
> My proposal is to extract the code responsible for starting Nailgun<br>
> into to a small utility script and let tox do the rest by removing<br>
> run_test.sh completely. That will bring us the following advantages:<br>
><br>
> - No need to support a complex bash script. - Closer to being able to<br>
> run functional tests on DSVM gates. - Test CLI will be more<br>
> compatible with other OpenStack projects.<br>
><br>
> I foresee a few questions and the answers for them follow:<br>
><br>
> Q: How is verify-job from FuelCI going to run tests without that<br>
> file? A: Fuel Client has its own job on FuelCI, so it will be just<br>
> necessary to change the invocation there.<br>
><br>
> Q: But run_test.sh is in all Fuel projects, shouldn’t we keep them<br>
> all similar. A: Why does it have to be similar? This kind of<br>
> difference is minor and it brings more advantages, than just having<br>
> the same file. In fact the set of options in run_tests.sh is already<br>
> different from run_tests.sh in fuel-web.<br>
><br>
> Q: Why should we look ad other OpenStack projects? A: Fuel is living<br>
> in the OpenStack ecosystem so being compatible with it is a big<br>
> advantage. It’s also a must for going big tent.<br>
<br>
+1.<br>
<br>
Just make sure any documentation that might refer to run_tests.sh is<br>
updated accordingly :)<br>
<br>
Best,<br>
-jay<br>
<br>
__________________________________________________________________________<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.openstack.org?subject:unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</blockquote></div></div></div></div><div dir="ltr">-- <br></div><div dir="ltr">Mike Scherbakov<br>#mihgen</div>