<div dir="ltr">Sean, Giulio,<div><br></div><div style>Thanks a lot for your explanations.</div><div style><br></div><div style>Giulio, from my opinion, as you said, the second solution would probably be better (the two backend names at the class level and skip the test if name1=name2). Let's see what others think about that.</div>

<div style><br></div><div style>Jérôme</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, May 31, 2013 at 9:38 PM, Giulio Fidente <span dir="ltr"><<a href="mailto:gfidente@redhat.com" target="_blank">gfidente@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="im">On 05/31/2013 09:23 PM, Giulio Fidente wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On 05/31/2013 07:07 PM, Sean Dague wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
This is why changing the variable later isn't any good (in the second<br>
patch), because the decorator was already evaluated. You may be able to<br>
build a trickier decorator which does a late load based on the class.<br>
</blockquote>
<br>
got it<br>
<br>
so would you prefer (and eventually +1) to evaluate the boolean within<br>
the test case or rather move at the class level the two backend names<br>
and skipIf name1=name2 '<br>
</blockquote>
<br></div>
I think the latter would work better but this raises another question: misconfiguring tempest by setting two backends with same name could result in a false positive<br>
<br>
shall we skip if misconfigured or what?<div class="HOEnZb"><div class="h5"><br>
-- <br>
Giulio Fidente<br>
GPG KEY: 08D733BA | IRC: giulivo<br>
<br>
______________________________<u></u>_________________<br>
openstack-qa mailing list<br>
<a href="mailto:openstack-qa@lists.openstack.org" target="_blank">openstack-qa@lists.openstack.<u></u>org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-qa" target="_blank">http://lists.openstack.org/<u></u>cgi-bin/mailman/listinfo/<u></u>openstack-qa</a><br>
</div></div></blockquote></div><br></div>