Comment 17 for bug 1714277

Revision history for this message
Ghanshyam Mann (ghanshyammann) wrote :

Thanks for bug report. I am partially agree here.

Agree on not forcing the dynamic cred on tests which does not change the test account like GET etc.

Disagree on removing the forcing of dynamic cred for tests which are changing the test account in some extend. Changing the test cred can effect the other tests execution as same cred are shared among other tests when using pre provisioned accounts. That was the main reason to making all the identity admin tests to use force_tenant_isolation by default.

So this bug is valid and we will go case by case and evaluate the above criteria to decide whether we can remove the force_tenant_isolation from identity tests. I will review the proposed patches.