<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Mar 25, 2014 at 1:49 AM, Maru Newby <span dir="ltr"><<a href="mailto:marun@redhat.com" target="_blank">marun@redhat.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5"><br>
On Mar 21, 2014, at 9:01 AM, David Kranz <<a href="mailto:dkranz@redhat.com">dkranz@redhat.com</a>> wrote:<br>
<br>
> On 03/20/2014 04:19 PM, Rochelle.RochelleGrober wrote:<br>
>><br>
>>> -----Original Message-----<br>
>>> From: Malini Kamalambal [mailto:<a href="mailto:malini.kamalambal@RACKSPACE.COM">malini.kamalambal@RACKSPACE.COM</a>]<br>
>>> Sent: Thursday, March 20, 2014 12:13 PM<br>
>>><br>
>>> 'project specific functional testing' in the Marconi context is<br>
>>> treating<br>
>>> Marconi as a complete system, making Marconi API calls & verifying the<br>
>>> response - just like an end user would, but without keystone. If one of<br>
>>> these tests fail, it is because there is a bug in the Marconi code ,<br>
>>> and<br>
>>> not because its interaction with Keystone caused it to fail.<br>
>>><br>
>>> "That being said there are certain cases where having a project<br>
>>> specific<br>
>>> functional test makes sense. For example swift has a functional test<br>
>>> job<br>
>>> that<br>
>>> starts swift in devstack. But, those things are normally handled on a<br>
>>> per<br>
>>> case<br>
>>> basis. In general if the project is meant to be part of the larger<br>
>>> OpenStack<br>
>>> ecosystem then Tempest is the place to put functional testing. That way<br>
>>> you know<br>
>>> it works with all of the other components. The thing is in openstack<br>
>>> what<br>
>>> seems<br>
>>> like a project isolated functional test almost always involves another<br>
>>> project<br>
>>> in real use cases. (for example keystone auth with api requests)<br>
>>><br>
>>> "<br>
<br>
<br>
<br>
>>><br>
>>> One of the concerns we heard in the review was 'having the functional<br>
>>> tests elsewhere (I.e within the project itself) does not count and they<br>
>>> have to be in Tempest'.<br>
>>> This has made us as a team wonder if we should migrate all our<br>
>>> functional<br>
>>> tests to Tempest.<br>
>>> But from Matt's response, I think it is reasonable to continue in our<br>
>>> current path & have the functional tests in Marconi coexist along with<br>
>>> the tests in Tempest.<br>
>>><br>
>> I think that what is being asked, really is that the functional tests could be a single set of tests that would become a part of the tempest repository and that these tests would have an ENV variable as part of the configuration that would allow either "no Keystone" or "Keystone" or some such, if that is the only configuration issue that separates running the tests isolated vs. integrated. The functional tests need to be as much as possible a single set of tests to reduce duplication and remove the likelihood of two sets getting out of sync with each other/development. If they only run in the integrated environment, that's ok, but if you want to run them isolated to make debugging easier, then it should be a configuration option and a separate test job.<br>
>><br>
>> So, if my assumptions are correct, QA only requires functional tests for integrated runs, but if the project QAs/Devs want to run isolated for dev and devtest purposes, more power to them. Just keep it a single set of functional tests and put them in the Tempest repository so that if a failure happens, anyone can find the test and do the debug work without digging into a separate project repository.<br>
>><br>
>> Hopefully, the tests as designed could easily take a new configuration directive and a short bit of work with OS QA will get the integrated FTs working as well as the isolated ones.<br>
>><br>
>> --Rocky<br>
> This issue has been much debated. There are some active members of our community who believe that all the functional tests should live outside of tempest in the projects, albeit with the same idea that such tests could be run either as part of today's "real" tempest runs or mocked in various ways to allow component isolation or better performance. Maru Newby posted a patch with an example of one way to do this but I think it expired and I don't have a pointer.<br>
<br>
</div></div>I think the best place for functional api tests to be maintained is in the projects themselves. The domain expertise required to write api tests is likely to be greater among project resources, and they should be tasked with writing api tests pre-merge. The current 'merge-first, test-later' procedure of maintaining api tests in the Tempest repo makes that impossible. Worse, the cost of developing functional api tests is higher in the integration environment that is the Tempest default.<br>
</blockquote><div><br></div><div><br></div><div>If an API is made and documented properly what domain expertise would be needed to use it? The opposite is true for tempest and the tests themselves. The tempest team focuses on just tests so they know how to write good tests and are able to leverage common underlying framework code.</div>
<div><br></div><div>Yes 'merge-first, test-later' is bad. But There are other ways of fixing this then moving the tests out of tempest. Hopefully cross project dependencies in zuul will help make this workflow easier.</div>
<div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
The patch in question [1] proposes allowing pre-merge functional api test maintenance and test reuse in an integration environment.<br>
<br>
<br>
m.<br>
<br>
1: <a href="https://review.openstack.org/#/c/72585/" target="_blank">https://review.openstack.org/#/c/72585/</a><br>
<div class=""><br>
> IMO there are valid arguments on both sides, but I hope every one could agree that functional tests should not be arbitrarily split between projects and tempest as they are now. The Tempest README states a desire for "complete coverage of the OpenStack API" but Tempest is not close to that. We have been discussing and then ignoring this issue for some time but I think the recent action to say that Tempest will be used to determine if something can use the OpenStack trademark will force more completeness on tempest (more tests, that is). I think we need to resolve this issue but it won't be easy and modifying existing api tests to be more flexible will be a lot of work. But at least new projects could get on the right path sooner.<br>
</div></blockquote><div><br></div><div><br></div><div>This touches on another point that I forgot to bring up before. As a cloud deployer I want a test suite that allows me to exercise all of my APIs to make sure everything is working. Although OpenStack runs extensive tests upstream, upstream cannot test every possible configuration, so as a deployer I would want rerun the same test suite. This is exactly why TripleO is getting ready to run tempest. Does this test suite have to just tempest? Perhaps not, but if we go down the road of moving more API tests into individual projects I think we still need easy way for a cloud deployer to run all the API tests against a live cloud from a single interface.</div>
<div> </div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">
<br>
<br>
<br>
</div> - to say nothing of the increased effort required to develop functional tests in an integration envirequired to maintain tests in Tempest splinters their efforts unnecessarily. I'm not suggesting that<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
><br>
> -David<br>
><br>
<br>
<br>
_______________________________________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</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>