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