2012-04-04 13:01:39 |
Rohit Karajgi |
description |
This issue is related to the re-factorings introduced by the following patch
https://review.openstack.org/#change,5894
We have tests to be checked into tempest that require client API operations to be performed admin (privileged role)
as well as non admin (member role) users.
With this view, the earlier behavior, and signature of __init__ in Manager of openstack.py was suitable.
I could create two instances of manager, one for an admin user, and the other for non-admin user.
Also, the latest tempest.conf.sample has a [compute-admin] section that allows an admin test user
to be configured.
However this is not updated in config.py. The file needs a compute-admin class to define these properties,
so they can be used in test cases to execute APIs.
https://github.com/openstack/tempest/commit/3f981df854cb55679b883713165262d7c37e45c0#diff-3
It would be good if some of the above changes were reverted.
I'll submit a patch. |
This issue is related to the re-factorings introduced by the following patch
https://review.openstack.org/#change,5894
We have tests to be checked into tempest that require client API operations to be performed by admin (privileged role)
as well as non admin (member role) users.
With this view, the earlier behavior, and signature of __init__ in Manager of openstack.py was suitable.
I could create two instances of manager, one for an admin user, and the other for non-admin user.
Also, the latest tempest.conf.sample has a [compute-admin] section that allows an admin test user
to be configured.
However this is not updated in config.py. The file needs a compute-admin class to define these properties,
so they can be used in test cases to execute APIs.
https://github.com/openstack/tempest/commit/3f981df854cb55679b883713165262d7c37e45c0#diff-3
It would be good if some of the above changes were reverted.
I'll submit a patch. |
|