Update patch set 6

Browse Source

Patch Set 6:

Adding some notes about --non-admin and --create confusion:

--create creates resources like tempest users, flavors and images - if those resources are not created before running tempestconf and --create is specified, it tries to create them. If --create is not specified and those resources does not exist, tempestconf fails, because it needs them.
So --create can use any user, who have permissions to create resources.

--non-admin - I believe this argument is meant to define a user, who doesn't have permissions to create resources. Basically what it does, it removes default admin credentials from tempest.conf which are set from default-overrides.conf (admin_username, admin_password, admin_project/tenant_name). Then it's understandable, that it can't work with --create together. 

So I believe even demo user can use --create, if he has permissions to do so. So if I understand it right, in the gate jobs, we want to test credentials of demo user as a user, who doesn't have rights to create new resources. So we doesn't need any change in arguments or the code. 

This are just my thoughts, what do you think?

Patch-set: 6

...

This commit is contained in:

Gerrit User 22873 2018-03-28 12:28:31 +00:00 committed by Gerrit Code Review

parent ec82e9d590

commit 832bd4a321

Show all changes Ignore whitespace when comparing lines Ignore changes in amount of whitespace Ignore changes in whitespace at EOL

Show Stats Download Patch File Download Diff File Expand all files Collapse all files

Diff Content Not Available