<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Mar 13, 2014 at 8:14 AM, Koderer, Marc <span dir="ltr"><<a href="mailto:m.koderer@telekom.de" target="_blank">m.koderer@telekom.de</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi folks,<br>
<br>
I can't make it to the QA meeting for today so I wanted to summarize the issue<br>
that we have with the pep8 and tempest gate. An example for the issue you can<br>
find here:<br>
  <a href="https://review.openstack.org/#/c/79256/" target="_blank">https://review.openstack.org/#/c/79256/</a><br>
  <a href="http://logs.openstack.org/56/79256/1/gate/gate-tempest-pep8/088cc12/console.html" target="_blank">http://logs.openstack.org/56/79256/1/gate/gate-tempest-pep8/088cc12/console.html</a><br>
<br>
pep8 check shows an error but the check itself is marked as success.<br>
<br>
For me this show two issues. First flake8 should return with an exit code !=0.<br>
I will have a closer look into hacking and what went wrong here.<br></blockquote><div><br></div><div>This was an intentional compromise.  At the time of doing that we didn't have consensus that any module should be importable without a stacktrace.</div>

<div><br></div><div>If we want to make that a rule, something which I am favor of, we can just add a new hacking rule. </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>
Second issue is the current implementation of the negative testing framework:<br>
we are using the testscenarios lib with the "load_tests" variable interpreted<br>
by the test runner. This forces us to build the scenario at import time and if<br>
we want to have tempest configurations for this (like introduced in<br>
<a href="https://review.openstack.org/#/c/73982/" target="_blank">https://review.openstack.org/#/c/73982/</a>) the laziness for the config doesn't<br>
work.<br>
<br>
Although it seems like if I remove the inheritance of the xml class to the<br>
json class (<a href="https://github.com/openstack/tempest/blob/master/tempest/api/compute/admin/test_flavors_negative_xml.py#L24" target="_blank">https://github.com/openstack/tempest/blob/master/tempest/api/compute/admin/test_flavors_negative_xml.py#L24</a>)<br>


that error doesn't appear any longer, I see a general problem with<br>
the usage of "import-time" code and we may think about a better solution in general.<br>
<br>
I'll try to address the missing pieces tomorrow.<br>
Bug: <a href="https://bugs.launchpad.net/tempest/+bug/1291826" target="_blank">https://bugs.launchpad.net/tempest/+bug/1291826</a><br>
<br>
Regards,<br>
Marc<br>
<br>
DEUTSCHE TELEKOM AG<br>
Digital Business Unit, Cloud Services (P&I)<br>
Marc Koderer<br>
Cloud Technology Software Developer<br>
T-Online-Allee 1, 64211 Darmstadt<br>
E-Mail: <a href="mailto:m.koderer@telekom.de">m.koderer@telekom.de</a><br>
<a href="http://www.telekom.com" target="_blank">www.telekom.com</a><br>
<br>
LIFE IS FOR SHARING.<br>
<br>
DEUTSCHE TELEKOM AG<br>
Supervisory Board: Prof. Dr. Ulrich Lehner (Chairman)<br>
Board of Management: René Obermann (Chairman),<br>
Reinhard Clemens, Niek Jan van Damme, Timotheus Höttges,<br>
Dr. Thomas Kremer, Claudia Nemat, Prof. Dr. Marion Schick<br>
Commercial register: Amtsgericht Bonn HRB 6794<br>
Registered office: Bonn<br>
<br>
BIG CHANGES START SMALL – CONSERVE RESOURCES BY NOT PRINTING EVERY E-MAIL.<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>
</blockquote></div><br></div></div>