DavidPurcell | b25f93d | 2017-01-27 12:46:27 -0500 | [diff] [blame] | 1 | # Copyright 2017 AT&T Corporation. |
DavidPurcell | 029d8c3 | 2017-01-06 15:27:41 -0500 | [diff] [blame] | 2 | # All Rights Reserved. |
| 3 | # |
| 4 | # Licensed under the Apache License, Version 2.0 (the "License"); you may |
| 5 | # not use this file except in compliance with the License. You may obtain |
| 6 | # a copy of the License at |
| 7 | # |
| 8 | # http://www.apache.org/licenses/LICENSE-2.0 |
| 9 | # |
| 10 | # Unless required by applicable law or agreed to in writing, software |
| 11 | # distributed under the License is distributed on an "AS IS" BASIS, WITHOUT |
| 12 | # WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the |
| 13 | # License for the specific language governing permissions and limitations |
| 14 | # under the License. |
| 15 | |
Felipe Monteiro | 10e82fd | 2017-11-21 01:47:20 +0000 | [diff] [blame] | 16 | from contextlib import contextmanager |
DavidPurcell | 029d8c3 | 2017-01-06 15:27:41 -0500 | [diff] [blame] | 17 | import time |
Felipe Monteiro | 34a138c | 2017-03-02 17:01:37 -0500 | [diff] [blame] | 18 | |
Rajiv Kumar | 645dfc9 | 2017-01-19 13:48:27 +0530 | [diff] [blame] | 19 | from oslo_log import log as logging |
Felipe Monteiro | 2693bf7 | 2017-08-12 22:56:47 +0100 | [diff] [blame] | 20 | from oslo_utils import excutils |
Felipe Monteiro | fa01d5f | 2017-04-01 06:18:25 +0100 | [diff] [blame] | 21 | |
Felipe Monteiro | 3e14f47 | 2017-08-17 23:02:11 +0100 | [diff] [blame] | 22 | from tempest import clients |
Felipe Monteiro | e7e552e | 2017-05-02 17:04:12 +0100 | [diff] [blame] | 23 | from tempest.common import credentials_factory as credentials |
Felipe Monteiro | fa01d5f | 2017-04-01 06:18:25 +0100 | [diff] [blame] | 24 | from tempest import config |
DavidPurcell | 029d8c3 | 2017-01-06 15:27:41 -0500 | [diff] [blame] | 25 | |
Felipe Monteiro | 34a138c | 2017-03-02 17:01:37 -0500 | [diff] [blame] | 26 | from patrole_tempest_plugin import rbac_exceptions |
DavidPurcell | 029d8c3 | 2017-01-06 15:27:41 -0500 | [diff] [blame] | 27 | |
DavidPurcell | 029d8c3 | 2017-01-06 15:27:41 -0500 | [diff] [blame] | 28 | CONF = config.CONF |
Felipe Monteiro | 34a138c | 2017-03-02 17:01:37 -0500 | [diff] [blame] | 29 | LOG = logging.getLogger(__name__) |
DavidPurcell | 029d8c3 | 2017-01-06 15:27:41 -0500 | [diff] [blame] | 30 | |
| 31 | |
DavidPurcell | 029d8c3 | 2017-01-06 15:27:41 -0500 | [diff] [blame] | 32 | class RbacUtils(object): |
Felipe Monteiro | 10e82fd | 2017-11-21 01:47:20 +0000 | [diff] [blame] | 33 | """Utility class responsible for switching ``os_primary`` role. |
Felipe Monteiro | e8d93e0 | 2017-07-19 20:52:20 +0100 | [diff] [blame] | 34 | |
| 35 | This class is responsible for overriding the value of the primary Tempest |
Felipe Monteiro | 10e82fd | 2017-11-21 01:47:20 +0000 | [diff] [blame] | 36 | credential's role (i.e. ``os_primary`` role). By doing so, it is possible |
| 37 | to seamlessly swap between admin credentials, needed for setup and clean |
| 38 | up, and primary credentials, needed to perform the API call which does |
Felipe Monteiro | e8d93e0 | 2017-07-19 20:52:20 +0100 | [diff] [blame] | 39 | policy enforcement. The primary credentials always cycle between roles |
Felipe Monteiro | 10e82fd | 2017-11-21 01:47:20 +0000 | [diff] [blame] | 40 | defined by ``CONF.identity.admin_role`` and |
| 41 | ``CONF.patrole.rbac_test_role``. |
Felipe Monteiro | e8d93e0 | 2017-07-19 20:52:20 +0100 | [diff] [blame] | 42 | """ |
DavidPurcell | 029d8c3 | 2017-01-06 15:27:41 -0500 | [diff] [blame] | 43 | |
Felipe Monteiro | b35de58 | 2017-05-05 00:16:53 +0100 | [diff] [blame] | 44 | def __init__(self, test_obj): |
Felipe Monteiro | e8d93e0 | 2017-07-19 20:52:20 +0100 | [diff] [blame] | 45 | """Constructor for ``RbacUtils``. |
| 46 | |
Felipe Monteiro | 2693bf7 | 2017-08-12 22:56:47 +0100 | [diff] [blame] | 47 | :param test_obj: An instance of `tempest.test.BaseTestCase`. |
Felipe Monteiro | e8d93e0 | 2017-07-19 20:52:20 +0100 | [diff] [blame] | 48 | """ |
Felipe Monteiro | e8d93e0 | 2017-07-19 20:52:20 +0100 | [diff] [blame] | 49 | # Intialize the admin roles_client to perform role switching. |
Felipe Monteiro | 3e14f47 | 2017-08-17 23:02:11 +0100 | [diff] [blame] | 50 | admin_mgr = clients.Manager( |
| 51 | credentials.get_configured_admin_credentials()) |
Felipe Monteiro | e8d93e0 | 2017-07-19 20:52:20 +0100 | [diff] [blame] | 52 | if test_obj.get_identity_version() == 'v3': |
Felipe Monteiro | 3e14f47 | 2017-08-17 23:02:11 +0100 | [diff] [blame] | 53 | admin_roles_client = admin_mgr.roles_v3_client |
Felipe Monteiro | e8d93e0 | 2017-07-19 20:52:20 +0100 | [diff] [blame] | 54 | else: |
Felipe Monteiro | 3e14f47 | 2017-08-17 23:02:11 +0100 | [diff] [blame] | 55 | admin_roles_client = admin_mgr.roles_client |
Felipe Monteiro | e8d93e0 | 2017-07-19 20:52:20 +0100 | [diff] [blame] | 56 | |
| 57 | self.admin_roles_client = admin_roles_client |
Felipe Monteiro | 10e82fd | 2017-11-21 01:47:20 +0000 | [diff] [blame] | 58 | self._override_role(test_obj, False) |
Felipe Monteiro | b35de58 | 2017-05-05 00:16:53 +0100 | [diff] [blame] | 59 | |
Felipe Monteiro | fa01d5f | 2017-04-01 06:18:25 +0100 | [diff] [blame] | 60 | admin_role_id = None |
| 61 | rbac_role_id = None |
DavidPurcell | 029d8c3 | 2017-01-06 15:27:41 -0500 | [diff] [blame] | 62 | |
Felipe Monteiro | 10e82fd | 2017-11-21 01:47:20 +0000 | [diff] [blame] | 63 | @contextmanager |
| 64 | def override_role(self, test_obj): |
| 65 | """Override the role used by ``os_primary`` Tempest credentials. |
| 66 | |
| 67 | Temporarily change the role used by ``os_primary`` credentials to: |
Masayuki Igawa | 80b9aab | 2018-01-09 17:00:45 +0900 | [diff] [blame] | 68 | |
| 69 | * ``[patrole] rbac_test_role`` before test execution |
| 70 | * ``[identity] admin_role`` after test execution |
Felipe Monteiro | 10e82fd | 2017-11-21 01:47:20 +0000 | [diff] [blame] | 71 | |
| 72 | Automatically switches to admin role after test execution. |
| 73 | |
| 74 | :param test_obj: Instance of ``tempest.test.BaseTestCase``. |
| 75 | :returns: None |
| 76 | |
| 77 | .. warning:: |
| 78 | |
| 79 | This function can alter user roles for pre-provisioned credentials. |
| 80 | Work is underway to safely clean up after this function. |
| 81 | |
| 82 | Example:: |
| 83 | |
| 84 | @rbac_rule_validation.action(service='test', |
| 85 | rule='a:test:rule') |
| 86 | def test_foo(self): |
| 87 | # Allocate test-level resources here. |
| 88 | with self.rbac_utils.override_role(self): |
melissaml | 7cd2161 | 2018-05-23 21:00:50 +0800 | [diff] [blame] | 89 | # The role for `os_primary` has now been overridden. Within |
Felipe Monteiro | 10e82fd | 2017-11-21 01:47:20 +0000 | [diff] [blame] | 90 | # this block, call the API endpoint that enforces the |
| 91 | # expected policy specified by "rule" in the decorator. |
| 92 | self.foo_service.bar_api_call() |
| 93 | # The role is switched back to admin automatically. Note that |
| 94 | # if the API call above threw an exception, any code below this |
| 95 | # point in the test is not executed. |
| 96 | """ |
| 97 | self._override_role(test_obj, True) |
| 98 | try: |
| 99 | # Execute the test. |
| 100 | yield |
| 101 | finally: |
| 102 | # This code block is always executed, no matter the result of the |
| 103 | # test. Automatically switch back to the admin role for test clean |
| 104 | # up. |
| 105 | self._override_role(test_obj, False) |
| 106 | |
Felipe Monteiro | 10e82fd | 2017-11-21 01:47:20 +0000 | [diff] [blame] | 107 | def _override_role(self, test_obj, toggle_rbac_role=False): |
| 108 | """Private helper for overriding ``os_primary`` Tempest credentials. |
| 109 | |
Felipe Monteiro | 07a1c17 | 2017-12-10 04:26:08 +0000 | [diff] [blame] | 110 | :param test_obj: instance of :py:class:`tempest.test.BaseTestCase` |
Felipe Monteiro | 10e82fd | 2017-11-21 01:47:20 +0000 | [diff] [blame] | 111 | :param toggle_rbac_role: Boolean value that controls the role that |
| 112 | overrides default role of ``os_primary`` credentials. |
| 113 | * If True: role is set to ``[patrole] rbac_test_role`` |
| 114 | * If False: role is set to ``[identity] admin_role`` |
| 115 | """ |
Felipe Monteiro | e8d93e0 | 2017-07-19 20:52:20 +0100 | [diff] [blame] | 116 | self.user_id = test_obj.os_primary.credentials.user_id |
| 117 | self.project_id = test_obj.os_primary.credentials.tenant_id |
| 118 | self.token = test_obj.os_primary.auth_provider.get_token() |
DavidPurcell | 029d8c3 | 2017-01-06 15:27:41 -0500 | [diff] [blame] | 119 | |
Felipe Monteiro | 10e82fd | 2017-11-21 01:47:20 +0000 | [diff] [blame] | 120 | LOG.debug('Overriding role to: %s.', toggle_rbac_role) |
Felipe Monteiro | 2693bf7 | 2017-08-12 22:56:47 +0100 | [diff] [blame] | 121 | role_already_present = False |
| 122 | |
DavidPurcell | 029d8c3 | 2017-01-06 15:27:41 -0500 | [diff] [blame] | 123 | try: |
Felipe Monteiro | 2693bf7 | 2017-08-12 22:56:47 +0100 | [diff] [blame] | 124 | if not all([self.admin_role_id, self.rbac_role_id]): |
| 125 | self._get_roles_by_name() |
DavidPurcell | 029d8c3 | 2017-01-06 15:27:41 -0500 | [diff] [blame] | 126 | |
Felipe Monteiro | 2693bf7 | 2017-08-12 22:56:47 +0100 | [diff] [blame] | 127 | target_role = ( |
| 128 | self.rbac_role_id if toggle_rbac_role else self.admin_role_id) |
| 129 | role_already_present = self._list_and_clear_user_roles_on_project( |
| 130 | target_role) |
| 131 | |
Felipe Monteiro | 10e82fd | 2017-11-21 01:47:20 +0000 | [diff] [blame] | 132 | # Do not override roles if `target_role` already exists. |
Felipe Monteiro | 2693bf7 | 2017-08-12 22:56:47 +0100 | [diff] [blame] | 133 | if not role_already_present: |
| 134 | self._create_user_role_on_project(target_role) |
DavidPurcell | 029d8c3 | 2017-01-06 15:27:41 -0500 | [diff] [blame] | 135 | except Exception as exp: |
Felipe Monteiro | 2693bf7 | 2017-08-12 22:56:47 +0100 | [diff] [blame] | 136 | with excutils.save_and_reraise_exception(): |
| 137 | LOG.exception(exp) |
Felipe Monteiro | 34a138c | 2017-03-02 17:01:37 -0500 | [diff] [blame] | 138 | finally: |
Felipe Monteiro | e8d93e0 | 2017-07-19 20:52:20 +0100 | [diff] [blame] | 139 | test_obj.os_primary.auth_provider.clear_auth() |
Felipe Monteiro | 7be94e8 | 2017-07-26 02:17:08 +0100 | [diff] [blame] | 140 | # Fernet tokens are not subsecond aware so sleep to ensure we are |
| 141 | # passing the second boundary before attempting to authenticate. |
Felipe Monteiro | 2693bf7 | 2017-08-12 22:56:47 +0100 | [diff] [blame] | 142 | # Only sleep if a token revocation occurred as a result of role |
Felipe Monteiro | 10e82fd | 2017-11-21 01:47:20 +0000 | [diff] [blame] | 143 | # overriding. This will optimize test runtime in the case where |
Felipe Monteiro | b58c119 | 2017-11-20 01:50:24 +0000 | [diff] [blame] | 144 | # ``[identity] admin_role`` == ``[patrole] rbac_test_role``. |
Felipe Monteiro | 2693bf7 | 2017-08-12 22:56:47 +0100 | [diff] [blame] | 145 | if not role_already_present: |
Rick Bartra | 89f498f | 2017-03-20 15:54:45 -0400 | [diff] [blame] | 146 | time.sleep(1) |
Felipe Monteiro | e8d93e0 | 2017-07-19 20:52:20 +0100 | [diff] [blame] | 147 | test_obj.os_primary.auth_provider.set_auth() |
Felipe Monteiro | 34a138c | 2017-03-02 17:01:37 -0500 | [diff] [blame] | 148 | |
Felipe Monteiro | 2693bf7 | 2017-08-12 22:56:47 +0100 | [diff] [blame] | 149 | def _get_roles_by_name(self): |
| 150 | available_roles = self.admin_roles_client.list_roles() |
| 151 | admin_role_id = rbac_role_id = None |
Felipe Monteiro | fa01d5f | 2017-04-01 06:18:25 +0100 | [diff] [blame] | 152 | |
Felipe Monteiro | 2693bf7 | 2017-08-12 22:56:47 +0100 | [diff] [blame] | 153 | for role in available_roles['roles']: |
Felipe Monteiro | f6eb862 | 2017-08-06 06:08:02 +0100 | [diff] [blame] | 154 | if role['name'] == CONF.patrole.rbac_test_role: |
Felipe Monteiro | 2693bf7 | 2017-08-12 22:56:47 +0100 | [diff] [blame] | 155 | rbac_role_id = role['id'] |
| 156 | if role['name'] == CONF.identity.admin_role: |
| 157 | admin_role_id = role['id'] |
| 158 | |
| 159 | if not all([admin_role_id, rbac_role_id]): |
Felipe Monteiro | f6eb862 | 2017-08-06 06:08:02 +0100 | [diff] [blame] | 160 | msg = ("Roles defined by `[patrole] rbac_test_role` and " |
| 161 | "`[identity] admin_role` must be defined in the system.") |
Felipe Monteiro | 2693bf7 | 2017-08-12 22:56:47 +0100 | [diff] [blame] | 162 | raise rbac_exceptions.RbacResourceSetupFailed(msg) |
| 163 | |
| 164 | self.admin_role_id = admin_role_id |
| 165 | self.rbac_role_id = rbac_role_id |
| 166 | |
| 167 | def _create_user_role_on_project(self, role_id): |
Felipe Monteiro | e8d93e0 | 2017-07-19 20:52:20 +0100 | [diff] [blame] | 168 | self.admin_roles_client.create_user_role_on_project( |
Felipe Monteiro | fa01d5f | 2017-04-01 06:18:25 +0100 | [diff] [blame] | 169 | self.project_id, self.user_id, role_id) |
| 170 | |
Felipe Monteiro | 2693bf7 | 2017-08-12 22:56:47 +0100 | [diff] [blame] | 171 | def _list_and_clear_user_roles_on_project(self, role_id): |
Felipe Monteiro | e8d93e0 | 2017-07-19 20:52:20 +0100 | [diff] [blame] | 172 | roles = self.admin_roles_client.list_user_roles_on_project( |
Felipe Monteiro | fa01d5f | 2017-04-01 06:18:25 +0100 | [diff] [blame] | 173 | self.project_id, self.user_id)['roles'] |
Felipe Monteiro | fa01d5f | 2017-04-01 06:18:25 +0100 | [diff] [blame] | 174 | role_ids = [role['id'] for role in roles] |
Felipe Monteiro | 2693bf7 | 2017-08-12 22:56:47 +0100 | [diff] [blame] | 175 | |
| 176 | # NOTE(felipemonteiro): We do not use ``role_id in role_ids`` here to |
| 177 | # avoid over-permission errors: if the current list of roles on the |
| 178 | # project includes "admin" and "Member", and we are switching to the |
| 179 | # "Member" role, then we must delete the "admin" role. Thus, we only |
| 180 | # return early if the user's roles on the project are an exact match. |
| 181 | if [role_id] == role_ids: |
Felipe Monteiro | fa01d5f | 2017-04-01 06:18:25 +0100 | [diff] [blame] | 182 | return True |
Felipe Monteiro | b3b7bc8 | 2017-03-03 15:58:15 -0500 | [diff] [blame] | 183 | |
| 184 | for role in roles: |
Felipe Monteiro | e8d93e0 | 2017-07-19 20:52:20 +0100 | [diff] [blame] | 185 | self.admin_roles_client.delete_role_from_user_on_project( |
Felipe Monteiro | fa01d5f | 2017-04-01 06:18:25 +0100 | [diff] [blame] | 186 | self.project_id, self.user_id, role['id']) |
| 187 | |
| 188 | return False |
| 189 | |
Felipe Monteiro | 17e9b49 | 2017-05-27 05:45:20 +0100 | [diff] [blame] | 190 | |
Felipe Monteiro | 07a1c17 | 2017-12-10 04:26:08 +0000 | [diff] [blame] | 191 | class RbacUtilsMixin(object): |
| 192 | """Mixin class to be used alongside an instance of |
| 193 | :py:class:`tempest.test.BaseTestCase`. |
| 194 | |
| 195 | Should be used to perform Patrole class setup for a base RBAC class. Child |
| 196 | classes should not use this mixin. |
| 197 | |
| 198 | Example:: |
| 199 | |
| 200 | class BaseRbacTest(rbac_utils.RbacUtilsMixin, base.BaseV2ComputeTest): |
| 201 | |
| 202 | @classmethod |
| 203 | def skip_checks(cls): |
| 204 | super(BaseRbacTest, cls).skip_checks() |
| 205 | cls.skip_rbac_checks() |
| 206 | |
| 207 | @classmethod |
| 208 | def setup_clients(cls): |
| 209 | super(BaseRbacTest, cls).setup_clients() |
| 210 | cls.setup_rbac_utils() |
| 211 | """ |
| 212 | |
| 213 | @classmethod |
| 214 | def skip_rbac_checks(cls): |
| 215 | if not CONF.patrole.enable_rbac: |
| 216 | raise cls.skipException( |
Felipe Monteiro | d737199 | 2018-04-25 16:57:09 +0100 | [diff] [blame] | 217 | 'Patrole testing not enabled so skipping %s.' % cls.__name__) |
Felipe Monteiro | 07a1c17 | 2017-12-10 04:26:08 +0000 | [diff] [blame] | 218 | |
| 219 | @classmethod |
| 220 | def setup_rbac_utils(cls): |
| 221 | cls.rbac_utils = RbacUtils(cls) |
| 222 | |
| 223 | |
Felipe Monteiro | 8a043fb | 2017-08-06 06:29:05 +0100 | [diff] [blame] | 224 | def is_admin(): |
| 225 | """Verifies whether the current test role equals the admin role. |
| 226 | |
| 227 | :returns: True if ``rbac_test_role`` is the admin role. |
| 228 | """ |
Felipe Monteiro | f6eb862 | 2017-08-06 06:08:02 +0100 | [diff] [blame] | 229 | return CONF.patrole.rbac_test_role == CONF.identity.admin_role |