<div dir="ltr">On Tue, Nov 21, 2017 at 4:47 PM MCCASLAND, TREVOR <<a href="mailto:tm2086@att.com">tm2086@att.com</a>> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hello QA Team,<br>
<br>
In many of the tempest tests there is a statement[1] that overrides the tempest.conf value for use_dynamic_credentials.<br>
In the event of an immutable user source, these test cases[2] and the one's inherited by them will fail.<br>
Can we remove or reduce the scope of the overrides? You can see my proposed solution for identity admin cases here[3] and my bug report here[4]<br>
<br>
What is the QA team's opinion on this?<br>
Is forcing dynamic creds too restrictive since it would prevent any tests being executed against clouds where credential creation is not allowed?<br></blockquote><div><br></div><div>The reason we force dynamic credentials for identity admin tests is that pre-provisioned credentials are generally not a good fit for admin test cases.</div><div>Admin tests are not a concern from an interoperability point of view, so forcing dynamic credentials on admin tests is not an issue for interoperability testing either.</div><div><br></div><div>That said there are probably cases where we could allow for pre-provisioned credentials without risking to break test isolation.</div><div>I'm fine with that in principle, I guess it has to be discussed on a test by test basis.</div><div><br></div><div>Andrea Frittoli (andreaf)</div><div><br></div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
[1] <a href="https://github.com/openstack/tempest/blob/eed21d7a1c0b3e5620960de9878ac9df0d2907fa/tempest/api/identity/base.py#L120-L127" rel="noreferrer" target="_blank">https://github.com/openstack/tempest/blob/eed21d7a1c0b3e5620960de9878ac9df0d2907fa/tempest/api/identity/base.py#L120-L127</a><br>
[2] <a href="http://codesearch.openstack.org/?q=force_tenant_isolation%20%3D%20True&i=nope&files=&repos=tempest" rel="noreferrer" target="_blank">http://codesearch.openstack.org/?q=force_tenant_isolation%20%3D%20True&i=nope&files=&repos=tempest</a><br>
[3] <a href="https://review.openstack.org/#/c/499756/" rel="noreferrer" target="_blank">https://review.openstack.org/#/c/499756/</a><br>
[4] <a href="https://bugs.launchpad.net/tempest/+bug/1714277" rel="noreferrer" target="_blank">https://bugs.launchpad.net/tempest/+bug/1714277</a><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>