Felipe Monteiro | c458932 | 2017-06-09 19:42:50 +0100 | [diff] [blame] | 1 | ======================== |
| 2 | Team and repository tags |
| 3 | ======================== |
| 4 | |
shangxiaobj | 11b0232 | 2017-08-14 22:45:11 -0700 | [diff] [blame] | 5 | .. image:: https://governance.openstack.org/tc/badges/patrole.svg |
| 6 | :target: https://governance.openstack.org/tc/reference/tags/index.html |
Felipe Monteiro | c458932 | 2017-06-09 19:42:50 +0100 | [diff] [blame] | 7 | |
Felipe Monteiro | c458932 | 2017-06-09 19:42:50 +0100 | [diff] [blame] | 8 | Patrole - RBAC Integration Tempest Plugin |
| 9 | ========================================= |
DavidPurcell | 663aedf | 2017-01-03 10:01:14 -0500 | [diff] [blame] | 10 | |
Felipe Monteiro | e5ee4be | 2018-06-18 21:39:28 +0100 | [diff] [blame] | 11 | Patrole is a set of integration tests to be run against a live OpenStack |
| 12 | cluster. It has a battery of tests dedicated to validating the correctness and |
| 13 | integrity of the cloud's RBAC implementation. |
| 14 | |
| 15 | More importantly, Patrole is a security validation tool for verifying that |
| 16 | Role-Based Access Control is correctly configured and enforced in an OpenStack |
| 17 | cloud. It runs `Tempest`_-based API tests using specified RBAC roles, thus |
| 18 | allowing deployments to verify that only intended roles have access to those |
| 19 | APIs. |
DavidPurcell | 663aedf | 2017-01-03 10:01:14 -0500 | [diff] [blame] | 20 | |
Felipe Monteiro | 780210d | 2017-07-17 22:21:53 +0100 | [diff] [blame] | 21 | Patrole currently offers testing for the following OpenStack services: Nova, |
| 22 | Neutron, Glance, Cinder and Keystone. |
DavidPurcell | 663aedf | 2017-01-03 10:01:14 -0500 | [diff] [blame] | 23 | |
Felipe Monteiro | c287389 | 2017-11-15 06:09:02 +0000 | [diff] [blame] | 24 | Patrole is currently undergoing heavy development. As more projects move |
| 25 | toward policy in code, Patrole will align its testing with the appropriate |
| 26 | documentation. |
| 27 | |
Felipe Monteiro | e5ee4be | 2018-06-18 21:39:28 +0100 | [diff] [blame] | 28 | * Free software: Apache license |
| 29 | * Documentation: https://docs.openstack.org/patrole/latest |
| 30 | * Source: https://git.openstack.org/cgit/openstack/patrole |
| 31 | * Bugs: https://bugs.launchpad.net/patrole |
| 32 | * Release notes: https://docs.openstack.org/releasenotes/patrole/ |
Felipe Monteiro | 443d39c | 2018-04-08 17:05:33 -0400 | [diff] [blame] | 33 | |
Felipe Monteiro | e917655 | 2018-07-16 14:39:55 -0400 | [diff] [blame] | 34 | .. _design-principles: |
| 35 | |
Felipe Monteiro | c287389 | 2017-11-15 06:09:02 +0000 | [diff] [blame] | 36 | Design Principles |
| 37 | ----------------- |
| 38 | |
Felipe Monteiro | e5ee4be | 2018-06-18 21:39:28 +0100 | [diff] [blame] | 39 | As a `Tempest plugin`_, Patrole borrows some `design principles`_ from Tempest, |
| 40 | but not all, as its testing scope is confined to policies. |
Felipe Monteiro | c287389 | 2017-11-15 06:09:02 +0000 | [diff] [blame] | 41 | |
| 42 | * *Stability*. Patrole uses OpenStack public interfaces. Tests in Patrole |
| 43 | should only touch public OpenStack APIs. |
| 44 | * *Atomicity*. Patrole tests should be atomic: they should test policies in |
| 45 | isolation. Unlike Tempest, a Patrole test strives to only call a single |
Felipe Monteiro | e5ee4be | 2018-06-18 21:39:28 +0100 | [diff] [blame] | 46 | endpoint at a time. This is because it is important to validate each policy |
| 47 | is authorized correctly and the best way to do that is to validate each |
| 48 | policy alone, to avoid test contamination. |
Felipe Monteiro | 543f7b9 | 2018-06-10 13:38:31 -0400 | [diff] [blame] | 49 | * *Complete coverage*. Patrole should validate all policy in code defaults. For |
| 50 | testing, Patrole uses the API-to-policy mapping contained in each project's |
| 51 | `policy in code`_ documentation where applicable. |
| 52 | |
| 53 | For example, Nova's policy in code documentation is located in the |
| 54 | `Nova repository`_ under ``nova/policies``. Likewise, Keystone's policy in |
| 55 | code documentation is located in the `Keystone repository`_ under |
| 56 | ``keystone/common/policies``. The other OpenStack services follow the same |
| 57 | directory layout pattern with respect to policy in code. |
| 58 | |
| 59 | .. note:: |
| 60 | |
| 61 | Realistically this is not always possible because some services have |
| 62 | not yet moved to policy in code. |
| 63 | |
Felipe Monteiro | e917655 | 2018-07-16 14:39:55 -0400 | [diff] [blame] | 64 | * *Customizable*. Patrole should be able to validate custom policy overrides to |
| 65 | ensure that those overrides enhance rather than undermine the cloud's RBAC |
| 66 | configuration. In addition, Patrole should be able to validate any role. |
Felipe Monteiro | 543f7b9 | 2018-06-10 13:38:31 -0400 | [diff] [blame] | 67 | * *Self-cleaning*. Patrole should attempt to clean up after itself; whenever |
Felipe Monteiro | c287389 | 2017-11-15 06:09:02 +0000 | [diff] [blame] | 68 | possible we should tear down resources when done. |
| 69 | |
| 70 | .. note:: |
| 71 | |
| 72 | Patrole modifies roles dynamically in the background, which affects |
| 73 | pre-provisioned credentials. Work is currently underway to clean up |
| 74 | modifications made to pre-provisioned credentials. |
| 75 | |
Felipe Monteiro | 543f7b9 | 2018-06-10 13:38:31 -0400 | [diff] [blame] | 76 | * *Self-testing*. Patrole should be self-testing. |
| 77 | |
Felipe Monteiro | e5ee4be | 2018-06-18 21:39:28 +0100 | [diff] [blame] | 78 | .. _Tempest plugin: https://docs.openstack.org/tempest/latest/plugin.html |
| 79 | .. _design principles: https://docs.openstack.org/tempest/latest/overview.html#design-principles |
Felipe Monteiro | 543f7b9 | 2018-06-10 13:38:31 -0400 | [diff] [blame] | 80 | .. _policy in code: https://specs.openstack.org/openstack/oslo-specs/specs/newton/policy-in-code.html |
| 81 | .. _Nova repository: https://github.com/openstack/nova/tree/master/nova/policies |
| 82 | .. _Keystone repository: https://github.com/openstack/keystone/tree/master/keystone/common/policies |
Felipe Monteiro | c287389 | 2017-11-15 06:09:02 +0000 | [diff] [blame] | 83 | |
DavidPurcell | 663aedf | 2017-01-03 10:01:14 -0500 | [diff] [blame] | 84 | Features |
Felipe Monteiro | 780210d | 2017-07-17 22:21:53 +0100 | [diff] [blame] | 85 | -------- |
| 86 | * Validation of default policy definitions located in policy.json files. |
| 87 | * Validation of in-code policy definitions. |
| 88 | * Validation of custom policy file definitions that override default policy |
| 89 | definitions. |
| 90 | * Built-in positive and negative testing. Positive and negative testing |
| 91 | are performed using the same tests and role-switching. |
| 92 | * Valdation of custom roles as well as default OpenStack roles. |
Felipe Monteiro | 7bc35dc | 2017-04-19 21:11:46 +0100 | [diff] [blame] | 93 | |
| 94 | .. note:: |
| 95 | |
Felipe Monteiro | 780210d | 2017-07-17 22:21:53 +0100 | [diff] [blame] | 96 | Patrole does not yet support policy.yaml files, the new file format for |
| 97 | policy files in OpenStack. |
Felipe Monteiro | 7bc35dc | 2017-04-19 21:11:46 +0100 | [diff] [blame] | 98 | |
| 99 | How It Works |
Felipe Monteiro | 780210d | 2017-07-17 22:21:53 +0100 | [diff] [blame] | 100 | ------------ |
| 101 | Patrole leverages ``oslo.policy`` (OpenStack's policy enforcement engine) to |
| 102 | determine whether a given role is allowed to perform a policy action, given a |
| 103 | specific role and OpenStack service. The output from ``oslo.policy`` (the |
| 104 | expected result) and the actual result from test execution are compared to |
| 105 | each other: if both results match, then the test passes; else it fails. |
Felipe Monteiro | 7bc35dc | 2017-04-19 21:11:46 +0100 | [diff] [blame] | 106 | |
Felipe Monteiro | e5ee4be | 2018-06-18 21:39:28 +0100 | [diff] [blame] | 107 | Terminology |
| 108 | ^^^^^^^^^^^ |
| 109 | * Expected Result - The expected result of a given test. |
| 110 | * Actual Result - The actual result of a given test. |
| 111 | * Final Result - A match between both expected and actual results. A mismatch |
| 112 | in the expected result and the actual result will result in a test failure. |
| 113 | |
| 114 | * Expected: Pass | Actual: Pass - Test Case Success |
| 115 | * Expected: Pass | Actual: Fail - Test Case Under-Permission Failure |
| 116 | * Expected: Fail | Actual: Pass - Test Case Over-Permission Failure |
| 117 | * Expected: Fail | Actual: Fail (Expected exception) - Test Case Success |
| 118 | * Expected: Fail | Actual: Fail (Unexpected exception) - Test Case Failure |
Felipe Monteiro | 7bc35dc | 2017-04-19 21:11:46 +0100 | [diff] [blame] | 119 | |
Felipe Monteiro | 780210d | 2017-07-17 22:21:53 +0100 | [diff] [blame] | 120 | Quickstart |
Felipe Monteiro | 7c7b570 | 2017-07-21 01:43:42 +0100 | [diff] [blame] | 121 | ---------- |
Felipe Monteiro | e5ee4be | 2018-06-18 21:39:28 +0100 | [diff] [blame] | 122 | To run Patrole, you must first have `Tempest`_ installed and configured |
| 123 | properly. Please reference Tempest's `Quickstart`_ guide to do so. Follow all |
| 124 | the steps outlined therein. Afterward, proceed with the steps below. |
Felipe Monteiro | 7bc35dc | 2017-04-19 21:11:46 +0100 | [diff] [blame] | 125 | |
Felipe Monteiro | e5ee4be | 2018-06-18 21:39:28 +0100 | [diff] [blame] | 126 | #. You first need to install Patrole. This is done with pip after you check out |
| 127 | the Patrole repo:: |
Felipe Monteiro | 7bc35dc | 2017-04-19 21:11:46 +0100 | [diff] [blame] | 128 | |
Felipe Monteiro | e5ee4be | 2018-06-18 21:39:28 +0100 | [diff] [blame] | 129 | $ git clone https://git.openstack.org/openstack/patrole |
| 130 | $ pip install patrole/ |
Felipe Monteiro | 7bc35dc | 2017-04-19 21:11:46 +0100 | [diff] [blame] | 131 | |
Felipe Monteiro | e5ee4be | 2018-06-18 21:39:28 +0100 | [diff] [blame] | 132 | This can be done within a venv. |
Felipe Monteiro | 7bc35dc | 2017-04-19 21:11:46 +0100 | [diff] [blame] | 133 | |
Felipe Monteiro | e5ee4be | 2018-06-18 21:39:28 +0100 | [diff] [blame] | 134 | .. note:: |
Felipe Monteiro | 7bc35dc | 2017-04-19 21:11:46 +0100 | [diff] [blame] | 135 | |
Felipe Monteiro | e5ee4be | 2018-06-18 21:39:28 +0100 | [diff] [blame] | 136 | You may also install Patrole from source code by running:: |
Felipe Monteiro | 7bc35dc | 2017-04-19 21:11:46 +0100 | [diff] [blame] | 137 | |
Felipe Monteiro | e5ee4be | 2018-06-18 21:39:28 +0100 | [diff] [blame] | 138 | pip install -e patrole/ |
Felipe Monteiro | 7bc35dc | 2017-04-19 21:11:46 +0100 | [diff] [blame] | 139 | |
Felipe Monteiro | e5ee4be | 2018-06-18 21:39:28 +0100 | [diff] [blame] | 140 | #. Next you must properly configure Patrole, which is relatively |
| 141 | straightforward. For details on configuring Patrole refer to the |
| 142 | :ref:`patrole-configuration`. |
Felipe Monteiro | 7bc35dc | 2017-04-19 21:11:46 +0100 | [diff] [blame] | 143 | |
Felipe Monteiro | e5ee4be | 2018-06-18 21:39:28 +0100 | [diff] [blame] | 144 | #. Once the configuration is done you're now ready to run Patrole. This can |
| 145 | be done using the `tempest_run`_ command. This can be done by running:: |
Felipe Monteiro | 7bc35dc | 2017-04-19 21:11:46 +0100 | [diff] [blame] | 146 | |
Felipe Monteiro | e5ee4be | 2018-06-18 21:39:28 +0100 | [diff] [blame] | 147 | $ tempest run --regex '^patrole_tempest_plugin\.tests\.api' |
Felipe Monteiro | 7bc35dc | 2017-04-19 21:11:46 +0100 | [diff] [blame] | 148 | |
Felipe Monteiro | e5ee4be | 2018-06-18 21:39:28 +0100 | [diff] [blame] | 149 | There is also the option to use testr directly, or any `testr`_ based test |
| 150 | runner, like `ostestr`_. For example, from the workspace dir run:: |
Felipe Monteiro | 7bc35dc | 2017-04-19 21:11:46 +0100 | [diff] [blame] | 151 | |
Felipe Monteiro | e5ee4be | 2018-06-18 21:39:28 +0100 | [diff] [blame] | 152 | $ stestr --regex '(?!.*\[.*\bslow\b.*\])(^patrole_tempest_plugin\.tests\.api))' |
| 153 | |
| 154 | will run the same set of tests as the default gate jobs. |
| 155 | |
| 156 | You can also run Patrole tests using `tox`_. To do so, ``cd`` into the |
| 157 | **Tempest** directory and run:: |
| 158 | |
| 159 | $ tox -eall-plugin -- patrole_tempest_plugin.tests.api |
| 160 | |
| 161 | .. note:: |
| 162 | |
| 163 | It is possible to run Patrole via ``tox -eall`` in order to run Patrole |
| 164 | isolated from other plugins. This can be accomplished by including the |
| 165 | installation of services that currently use policy in code -- for example, |
| 166 | Nova and Keystone. For example:: |
| 167 | |
| 168 | $ tox -evenv-tempest -- pip install /opt/stack/patrole /opt/stack/keystone /opt/stack/nova |
| 169 | $ tox -eall -- patrole_tempest_plugin.tests.api |
| 170 | |
| 171 | #. Log information from tests is captured in ``tempest.log`` under the Tempest |
| 172 | repository. Some Patrole debugging information is captured in that log |
| 173 | related to expected test results and :ref:`role-overriding`. |
| 174 | |
| 175 | More detailed RBAC testing log output is emitted to ``patrole.log`` under |
| 176 | the Patrole repository. To configure Patrole's logging, see the |
| 177 | :ref:`patrole-configuration` guide. |
| 178 | |
| 179 | .. _Tempest: https://github.com/openstack/tempest |
| 180 | .. _Quickstart: https://docs.openstack.org/tempest/latest/overview.html#quickstart |
| 181 | .. _tempest_run: https://docs.openstack.org/tempest/latest/run.html |
| 182 | .. _testr: https://testrepository.readthedocs.org/en/latest/MANUAL.html |
| 183 | .. _ostestr: https://docs.openstack.org/os-testr/latest/ |
| 184 | .. _tox: https://tox.readthedocs.io/en/latest/ |
| 185 | |
| 186 | RBAC Tests |
| 187 | ---------- |
| 188 | |
| 189 | To change the role that the patrole tests are being run as, edit |
| 190 | ``rbac_test_role`` in the ``patrole`` section of tempest.conf: :: |
| 191 | |
| 192 | [patrole] |
| 193 | rbac_test_role = member |
| 194 | ... |
| 195 | |
| 196 | .. note:: |
| 197 | |
| 198 | The ``rbac_test_role`` is service-specific. member, for example, |
| 199 | is an arbitrary role, but by convention is used to designate the default |
| 200 | non-admin role in the system. Most Patrole tests should be run with |
| 201 | **admin** and **member** roles. However, other services may use entirely |
| 202 | different roles. |
| 203 | |
Felipe Monteiro | 4d4cb1e | 2018-07-29 13:44:10 -0400 | [diff] [blame^] | 204 | For more information about RBAC, reference the `rbac-overview`_ |
| 205 | documentation page. |
| 206 | |
| 207 | .. _rbac-overview: https://docs.openstack.org/patrole/latest/rbac-overview.html |
Felipe Monteiro | e5ee4be | 2018-06-18 21:39:28 +0100 | [diff] [blame] | 208 | |
| 209 | Unit Tests |
| 210 | ---------- |
| 211 | |
| 212 | Patrole also has a set of unit tests which test the Patrole code itself. These |
| 213 | tests can be run by specifying the test discovery path:: |
| 214 | |
| 215 | $ stestr --test-path ./patrole_tempest_plugin/tests/unit run |
| 216 | |
| 217 | By setting ``--test-path`` option to ``./patrole_tempest_plugin/tests/unit`` |
| 218 | it specifies that test discovery should only be run on the unit test directory. |
| 219 | |
| 220 | Alternatively, there are the py27 and py35 tox jobs which will run the unit |
| 221 | tests with the corresponding version of Python. |
| 222 | |
| 223 | One common activity is to just run a single test; you can do this with tox |
| 224 | simply by specifying to just run py27 or py35 tests against a single test:: |
| 225 | |
| 226 | $ tox -e py27 -- -n patrole_tempest_plugin.tests.unit.test_rbac_utils.RBACUtilsTest.test_override_role_with_missing_admin_role |
| 227 | |
| 228 | Or all tests in the test_rbac_utils.py file:: |
| 229 | |
| 230 | $ tox -e py27 -- -n patrole_tempest_plugin.tests.unit.test_rbac_utils |
| 231 | |
| 232 | You may also use regular expressions to run any matching tests:: |
| 233 | |
| 234 | $ tox -e py27 -- test_rbac_utils |
| 235 | |
| 236 | For more information on these options and details about stestr, please see the |
| 237 | `stestr documentation <http://stestr.readthedocs.io/en/latest/MANUAL.html>`_. |
Felipe Monteiro | 7bc35dc | 2017-04-19 21:11:46 +0100 | [diff] [blame] | 238 | |
Felipe Monteiro | 780210d | 2017-07-17 22:21:53 +0100 | [diff] [blame] | 239 | Release Versioning |
Felipe Monteiro | 7c7b570 | 2017-07-21 01:43:42 +0100 | [diff] [blame] | 240 | ------------------ |
Felipe Monteiro | 780210d | 2017-07-17 22:21:53 +0100 | [diff] [blame] | 241 | `Patrole Release Notes <https://docs.openstack.org/releasenotes/patrole/>`_ |
| 242 | shows which changes have been released for each version. |
Felipe Monteiro | 7bc35dc | 2017-04-19 21:11:46 +0100 | [diff] [blame] | 243 | |
Felipe Monteiro | 780210d | 2017-07-17 22:21:53 +0100 | [diff] [blame] | 244 | Patrole's release versioning follows Tempest's conventions. Like Tempest, |
| 245 | Patrole is branchless and uses versioning instead. |