)]}' { "log": [ { "commit": "842845eaef1de6595e96ad253d0666555ce083f1", "tree": "7fd1bc429b88e420370ae30ea4fb1a7b1428a065", "parents": [ "f4cb74ce470b247f3f0e600200f9d893defbc738", "63d8602d359bf5f9492bf1f58c9edba89f01dd7f" ], "author": { "name": "Zuul", "email": "zuul@review.openstack.org", "time": "Fri Aug 24 19:49:39 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "review@openstack.org", "time": "Fri Aug 24 19:49:39 2018 +0000" }, "message": "Merge \"Add tests to cover flavor_service_profile\"" }, { "commit": "f4cb74ce470b247f3f0e600200f9d893defbc738", "tree": "3e6e1d70f5b59f257f3b416ec8f7c4764c07ef15", "parents": [ "2b93778e8cb0fe11f4b009a5b70df8efb92854c0" ], "author": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Sun Aug 19 17:18:56 2018 +0100" }, "committer": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Wed Aug 22 20:46:17 2018 +0000" }, "message": "add python 3.6 unit test job\n\nThis is a mechanically generated patch to add a unit test job running\nunder Python 3.6 as part of the python3-first goal.\n\nSee the python3-first goal document for details:\nhttps://governance.openstack.org/tc/goals/stein/python3-first.html\n\nChange-Id: I300eb6cdd44d5486e5eb4cc4bf9b05aa3678251f\nStory: #2002586\nTask: #24763\n" }, { "commit": "2b93778e8cb0fe11f4b009a5b70df8efb92854c0", "tree": "be8285cbc812ea8355b0b6ffbf51897cf25756c8", "parents": [ "33d0583e054002d684053c4390ae13a0563c7e95", "98437d41a1f34db48422310a3d73de8149147c86" ], "author": { "name": "Zuul", "email": "zuul@review.openstack.org", "time": "Wed Aug 22 05:32:18 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "review@openstack.org", "time": "Wed Aug 22 05:32:18 2018 +0000" }, "message": "Merge \"Remove override of \u0027expected_error_codes\u0027 with defaults\"" }, { "commit": "33d0583e054002d684053c4390ae13a0563c7e95", "tree": "ed62ad137e2033a25ea1b2bc094b78240974e7ac", "parents": [ "64f3943dadf34214192a7550e8a0dc9a34eb5173", "0f73e7ca9fa0deae2c88b0e9d7b3237397d92dda" ], "author": { "name": "Zuul", "email": "zuul@review.openstack.org", "time": "Wed Aug 22 04:53:50 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "review@openstack.org", "time": "Wed Aug 22 04:53:51 2018 +0000" }, "message": "Merge \"Add bandit python security scanning to pep8\"" }, { "commit": "64f3943dadf34214192a7550e8a0dc9a34eb5173", "tree": "0129e1eb91cac59a808e6b06ce5e0e56c7ee3947", "parents": [ "031b182470a33065e868b916e6a11f2dec8244ae", "0f45285601808eab599e5d31887e9503e17a8392" ], "author": { "name": "Zuul", "email": "zuul@review.openstack.org", "time": "Wed Aug 22 01:06:53 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "review@openstack.org", "time": "Wed Aug 22 01:06:53 2018 +0000" }, "message": "Merge \"Remove the usage of deprecated arg \u0027expected_error_code\u0027\"" }, { "commit": "0f73e7ca9fa0deae2c88b0e9d7b3237397d92dda", "tree": "9a687b977893d174dce4859a70041868a744c338", "parents": [ "b1e9d7a69337ccaa2395437e120e80bac4b5d662" ], "author": { "name": "Gage Hugo", "email": "gagehugo@gmail.com", "time": "Tue Aug 21 15:51:08 2018 -0500" }, "committer": { "name": "Gage Hugo", "email": "gagehugo@gmail.com", "time": "Tue Aug 21 15:51:08 2018 -0500" }, "message": "Add bandit python security scanning to pep8\n\nThis change adds bandit python security code scanning to the\npep8 zuul check.\n\nChange-Id: I8b6c52bd3fae87c5b49f3cde8bb9e63526155973\n" }, { "commit": "98437d41a1f34db48422310a3d73de8149147c86", "tree": "10ee4a74e38101d836928cd3192018223ca4fd29", "parents": [ "b1e9d7a69337ccaa2395437e120e80bac4b5d662" ], "author": { "name": "ghanshyam", "email": "gmann@ghanshyammann.com", "time": "Fri Aug 17 08:51:43 2018 +0000" }, "committer": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Tue Aug 21 18:50:02 2018 +0000" }, "message": "Remove override of \u0027expected_error_codes\u0027 with defaults\n\n\u0027expected_error_codes\u0027 are filled with defaults error code[1]\nin base class which is nothing but to fill required number\nof error codes with 403.\n\nFew tests override the \u0027expected_error_codes\u0027 with 403 itself which\nis confusing and inconsistent with other tests.\n\nThis commit removes those default overridden values.\n\n[1] https://github.com/openstack/patrole/blob/a6371ed510b318183e1c2cbafcc97bd85f6625f8/patrole_tempest_plugin/rbac_rule_validation.py#L286\n\nChange-Id: If607755fc09c392566fffad8f46e5f37cd33c2d2\n" }, { "commit": "0f45285601808eab599e5d31887e9503e17a8392", "tree": "5b7f96c685c9cc6f2eb921515bf7ca97e913f3b8", "parents": [ "b1e9d7a69337ccaa2395437e120e80bac4b5d662" ], "author": { "name": "ghanshyam", "email": "gmann@ghanshyammann.com", "time": "Fri Aug 17 09:30:07 2018 +0000" }, "committer": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Tue Aug 21 18:49:53 2018 +0000" }, "message": "Remove the usage of deprecated arg \u0027expected_error_code\u0027\n\n\u0027expected_error_code\u0027 arg to rbac_rule_validation.action is\ndeprecated in favor of \u0027expected_error_codes\u0027.\n\nMany test use the old deprecated arg and that\u0027s why the warning. We\nshould cleanup the usage in patrole to avoid the confusion and make\nall test to latest interface.\n\nThis commit migrate those usage to new arg and leave the deprecated\narg as it is which can be removed completely after deprecation phase.\n\nChange-Id: Ie284ae74ce6480a59406aecfe341d579142bd625\n" }, { "commit": "031b182470a33065e868b916e6a11f2dec8244ae", "tree": "4954b3f370976abc43fd36fb0f74e3faa6aa70e6", "parents": [ "b1e9d7a69337ccaa2395437e120e80bac4b5d662" ], "author": { "name": "Mykola Yakovliev", "email": "VegasQ@gmail.com", "time": "Sat Jul 28 13:14:24 2018 -0500" }, "committer": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Tue Aug 21 18:49:12 2018 +0000" }, "message": "Add tests to cover RBAC policies\n\nThis patch set adds tests to cver the neutron RBAC policies API [0].\nTest gets, creates, updates, and deletes RBAC policies.\nPart of \"Increase Neutron RBAC Coverage\" initiative [1]\n\n[0] https://developer.openstack.org/api-ref/network/v2/index.html#rbac-policies\n[1] https://storyboard.openstack.org/#!/story/2002641\n\nChange-Id: I37e07c1b37a4630024d45dc9e5ade25ef31c703a\nStory: 2002641\nTask: 22322\n" }, { "commit": "b1e9d7a69337ccaa2395437e120e80bac4b5d662", "tree": "f4f7fc1b737029a2bdedf844caed497e3cea39bd", "parents": [ "17385c78737ca5e5dcc50a378596f0c9efa4f19b", "6bffc5c5c6d1a52793e1e2091f4e87b2edac8f31" ], "author": { "name": "Zuul", "email": "zuul@review.openstack.org", "time": "Tue Aug 21 18:26:13 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "review@openstack.org", "time": "Tue Aug 21 18:26:13 2018 +0000" }, "message": "Merge \"Skip the deprecated API extensions policy tests\"" }, { "commit": "6bffc5c5c6d1a52793e1e2091f4e87b2edac8f31", "tree": "deb60b61fb01e112115f244effa1cd72fbe81f9a", "parents": [ "d063b409bb7a3d056ac9d55bb6200b1076e71422" ], "author": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Sun Aug 19 22:54:33 2018 +0100" }, "committer": { "name": "Ghanshyam Mann", "email": "gmann@ghanshyammann.com", "time": "Tue Aug 21 05:53:38 2018 +0000" }, "message": "Skip the deprecated API extensions policy tests\n\nA new policy feature flag called\n``[policy_feature_flag].removed_nova_policies_stein``\nhas been added to Patrole\u0027s config to handle Nova API\nextension policies removed in Stein [0].\n\nThe policy feature flag is applied to tests that validate\nresponse bodies for expected attributes previously returned\nfor the following policies that passed authorization:\n\n - os_compute_api:os-config-drive\n - os_compute_api:os-extended-availability-zone\n - os_compute_api:os-extended-status\n - os_compute_api:os-extended-volumes\n - os_compute_api:os-keypairs\n - os_compute_api:os-server-usage\n - os_compute_api:os-flavor-rxtx\n - os_compute_api:os-flavor-access (only from /flavors APIs)\n - os_compute_api:image-size\n\nNote that not all removed policies are included above because\ntest coverage is missing for them (like\nos_compute_api:os-security-groups).\n\nAlso fixes test flows associated with image_size tests:\n\n* endpoints are list images with details and show image (not\n list image)\n* both tests should check for OS-EXT-IMG-SIZE:size attribute\n\n[0] https://review.openstack.org/#/c/586872/8\n\nStory: 2003501\n\nChange-Id: Ia6f8d255a540f7063beedd80a3ca1833f3987490\n" }, { "commit": "17385c78737ca5e5dcc50a378596f0c9efa4f19b", "tree": "0da0cf53f8d6dae1aac6b02b052321130bd5b34e", "parents": [ "d063b409bb7a3d056ac9d55bb6200b1076e71422", "bd15460f14ec538d0be12ca8f1201478134a82fe" ], "author": { "name": "Zuul", "email": "zuul@review.openstack.org", "time": "Mon Aug 20 21:52:21 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "review@openstack.org", "time": "Mon Aug 20 21:52:21 2018 +0000" }, "message": "Merge \"Add release notes page for v0.4.0\"" }, { "commit": "d063b409bb7a3d056ac9d55bb6200b1076e71422", "tree": "0aa9261b3cc274b7bb8a06f36a3e23d41944f516", "parents": [ "9ee07cf523583ffa0b6988de54c8dfe9b7e6e645", "04b26288b36564fb6498e4457b69b879aec23b99" ], "author": { "name": "Zuul", "email": "zuul@review.openstack.org", "time": "Fri Aug 17 09:29:36 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "review@openstack.org", "time": "Fri Aug 17 09:29:36 2018 +0000" }, "message": "Merge \"Add tests to cover trunks\"" }, { "commit": "bd15460f14ec538d0be12ca8f1201478134a82fe", "tree": "47bdf6b2cc93b0591419754da35bac7d4c07b8b6", "parents": [ "9ee07cf523583ffa0b6988de54c8dfe9b7e6e645" ], "author": { "name": "ghanshyam", "email": "gmann@ghanshyammann.com", "time": "Fri Aug 17 05:42:02 2018 +0000" }, "committer": { "name": "ghanshyam", "email": "gmann@ghanshyammann.com", "time": "Fri Aug 17 05:42:02 2018 +0000" }, "message": "Add release notes page for v0.4.0\n\nPatrole 0.4.0 has been released [1].\nThis commit adds a new page for v0.4.0 release notes.\n\n[1] https://review.openstack.org/#/c/592277/\n\nChange-Id: I98519d5f4ff788fee485e735f4c284ea1dd00c28\n" }, { "commit": "9ee07cf523583ffa0b6988de54c8dfe9b7e6e645", "tree": "931c491b20a4bef3cf6e04c537937923f72e067f", "parents": [ "cad317ec400655b1d23bb0f3fc3a8edd0d6a8c40" ], "author": { "name": "ghanshyam", "email": "gmann@ghanshyammann.com", "time": "Thu Aug 16 08:15:42 2018 +0000" }, "committer": { "name": "Ghanshyam Mann", "email": "gmann@ghanshyammann.com", "time": "Thu Aug 16 08:49:40 2018 +0000" }, "message": "Fix \u0027ref\u0027 format errors in README file\n\nWhile releasing new version in pypi, it perform the\nvalidation on the package and give error if there is any\nformatting issue.\n\nREADME file is being used to upload on pypi and so validation\ncheck fails[1] on current README due to \u0027ref\u0027 used for internal\ndoc link. Those links would not be available for outside the built doc.\nThis can be checked locally by:\n python setup.py check -s -r\n\nThis commit fixes those format issue by removing the ref and provide the\nexternal links.\n\nChange-Id: I678e792eaf4e362f48864da761461c7779a09155\n" }, { "commit": "cad317ec400655b1d23bb0f3fc3a8edd0d6a8c40", "tree": "5d1bc155915a8dec79598b1156d46945b0f7553d", "parents": [ "bfcf1cc50e05c765f852044c679984c67589d736" ], "author": { "name": "ghanshyam", "email": "gmann@ghanshyammann.com", "time": "Thu Aug 16 03:13:18 2018 +0000" }, "committer": { "name": "ghanshyam", "email": "gmann@ghanshyammann.com", "time": "Thu Aug 16 03:13:18 2018 +0000" }, "message": "Add releasenote to tag the Patrole for Rocky release\n\nThis commit adds the releasenote to tag Patrole for Rocky\nand needed for version 0.4.0 to release.\n\nChange-Id: Ic6897aa3dd057eebc99e750d50ebb87a35391b2d\n" }, { "commit": "63d8602d359bf5f9492bf1f58c9edba89f01dd7f", "tree": "a045c6f5fa43d7738bd0235207951a0ea9073e44", "parents": [ "bfcf1cc50e05c765f852044c679984c67589d736" ], "author": { "name": "Mykola Yakovliev", "email": "VegasQ@gmail.com", "time": "Fri Aug 10 14:01:15 2018 -0500" }, "committer": { "name": "Mykola Yakovliev", "email": "VegasQ@gmail.com", "time": "Sat Aug 11 09:12:54 2018 -0500" }, "message": "Add tests to cover flavor_service_profile\n\nThis patch set adds tests to cover the flavor_service_profile API [0].\nTest creates and deletes flavor_service_profile.\nPart of \"Increase Neutron RBAC Coverage\" initiative [1]\n\n[0] https://developer.openstack.org/api-ref/network/v2/index.html#associate-flavor-with-a-service-profile\n[1] https://storyboard.openstack.org/#!/story/2002641\n\nChange-Id: I22b0467f698d548abb85f45c993bbe0165eb21b1\nStory: 2002641\nTask: 22324\n" }, { "commit": "bfcf1cc50e05c765f852044c679984c67589d736", "tree": "e699e9205211f01cadf484cf0aff1a71fae6acda", "parents": [ "c5dd1b4ef30f7d30b5424f75b850880352fba16d", "73d9f957b36eb68525655730fab48dc653f950cf" ], "author": { "name": "Zuul", "email": "zuul@review.openstack.org", "time": "Fri Aug 10 17:21:34 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "review@openstack.org", "time": "Fri Aug 10 17:21:34 2018 +0000" }, "message": "Merge \"Add tests to cover flavors\"" }, { "commit": "73d9f957b36eb68525655730fab48dc653f950cf", "tree": "17d106d0c98df73612ba0f4d3895dcda78a89f2f", "parents": [ "1d829781df157bc4a11b2489054997f09b4c54ee" ], "author": { "name": "Mykola Yakovliev", "email": "VegasQ@gmail.com", "time": "Sat Jul 28 11:03:32 2018 -0500" }, "committer": { "name": "Mykola Yakovliev", "email": "myakovliev@mirantis.com", "time": "Thu Aug 09 21:17:42 2018 +0000" }, "message": "Add tests to cover flavors\n\nThis patch set adds tests to cver the flavors API [0].\nTest lists, gets, creates, updates, and deletes flavors.\nPart of \"Increase Neutron RBAC Coverage\" initiative [1]\n\n[0] https://developer.openstack.org/api-ref/network/v2/index.html#networking-flavors-framework-v2-0-current-flavor-service-profile\n[1] https://storyboard.openstack.org/#!/story/2002641\n\nChange-Id: I526d1567b715c0a57e73fa6645d8a98b9b45c62e\nStory: 2002641\nTask: 22316\n" }, { "commit": "c5dd1b4ef30f7d30b5424f75b850880352fba16d", "tree": "d201d5f2995b1ac1e358bf0770af63147d46c1d9", "parents": [ "2cc271c75a85cf393b06b2054c302c3ce5ba7187", "a137576f949ff8c3ee1671bd3252a8f5c30219fa" ], "author": { "name": "Zuul", "email": "zuul@review.openstack.org", "time": "Thu Aug 09 05:03:24 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "review@openstack.org", "time": "Thu Aug 09 05:03:24 2018 +0000" }, "message": "Merge \"Improve tests for test_volume_basic_crud.py\"" }, { "commit": "2cc271c75a85cf393b06b2054c302c3ce5ba7187", "tree": "e69b07c7321c4599f7585a55bd0a1b7b650363c6", "parents": [ "3e98fe47b54dd75365e158e884ee7406a703c7d4" ], "author": { "name": "Mykola Yakovliev", "email": "VegasQ@gmail.com", "time": "Wed Aug 01 16:29:42 2018 -0500" }, "committer": { "name": "Mykola Yakovliev", "email": "myakovliev@mirantis.com", "time": "Tue Aug 07 18:34:32 2018 +0000" }, "message": "Add tests to cover auto_allocated_topology\n\nThis patch set adds tests to cver the neutron auto allocated topology API [0].\nPart of \"Increase Neutron RBAC Coverage\" initiative [1]\n\n[0] https://developer.openstack.org/api-ref/network/v2/index.html#id877\n[1] https://storyboard.openstack.org/#!/story/2002641\n\nChange-Id: I2341be8aa8c9057069dff24dfe718b6239e68817\nStory: 2002641\nTask: 22325\n" }, { "commit": "04b26288b36564fb6498e4457b69b879aec23b99", "tree": "393abaa3ef8e627bd93d6dc3a123b6d6f8c615af", "parents": [ "3e98fe47b54dd75365e158e884ee7406a703c7d4" ], "author": { "name": "Mykola Yakovliev", "email": "VegasQ@gmail.com", "time": "Mon Jul 23 12:13:02 2018 -0500" }, "committer": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Tue Aug 07 00:57:22 2018 +0000" }, "message": "Add tests to cover trunks\n\nThis patch set adds tests to cver the neutron trunks API [0].\nTest lists, gets, creates, updates, and deletes trunks.\nPart of \"Increase Neutron RBAC Coverage\" initiative [1]\n\n[0] https://developer.openstack.org/api-ref/network/v2/index.html#trunk-networking\n[1] https://storyboard.openstack.org/#!/story/2002641\n\nStory: 2002641\nTask: 22326\nChange-Id: Ie59f02f5ecc1d04a16fb178444f231e4791e2079\n" }, { "commit": "3e98fe47b54dd75365e158e884ee7406a703c7d4", "tree": "bf2d0480d34421a3fef091939d2e01c608aec386", "parents": [ "1d829781df157bc4a11b2489054997f09b4c54ee" ], "author": { "name": "Mykola Yakovliev", "email": "VegasQ@gmail.com", "time": "Thu Aug 02 17:25:53 2018 -0500" }, "committer": { "name": "Mykola Yakovliev", "email": "myakovliev@mirantis.com", "time": "Sat Aug 04 19:59:24 2018 +0000" }, "message": "Update segments rbac tests with error codes\n\nThis patchset updates tests with exact codes to prevent\nfalse positive results.\n\nChange-Id: I7d7f640c9adda06767eabb657fe739748a2beaf2\n" }, { "commit": "1d829781df157bc4a11b2489054997f09b4c54ee", "tree": "f2c8e15b981e7a082d4f7b33544e1e6022733540", "parents": [ "caffc23980285e4864085976fb482ca04d1ca7d2" ], "author": { "name": "Mykola Yakovliev", "email": "VegasQ@gmail.com", "time": "Fri Aug 03 14:37:37 2018 -0500" }, "committer": { "name": "Mykola Yakovliev", "email": "VegasQ@gmail.com", "time": "Sat Aug 04 11:40:51 2018 -0500" }, "message": "Reset ntp_client cached token in override_role context\n\nAs bug [0] shows we have incorrect behaviour with client, when its\nallowed to perform forbidden action beacause of cached token.\nThis patchset expands RbacUtilsMixin with `get_auth_providers` so\nwe can define additional auth_providers on testcase level.\nUsing it we reset cached auth data for neutron-tempest-plugin\nclient in *PluginRbacTest tests.\n\n[0] https://storyboard.openstack.org/#!/story/2003296\n\nChange-Id: Ie444f1e0be1c9a5ff7fc8a968269b73f510685e3\nStory: 2003296\nTask: 24245\n" }, { "commit": "a137576f949ff8c3ee1671bd3252a8f5c30219fa", "tree": "a28cc4021de0a0d2025d2782f955664b24e3aea1", "parents": [ "caffc23980285e4864085976fb482ca04d1ca7d2" ], "author": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Sun Jul 29 12:58:05 2018 -0400" }, "committer": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Sat Aug 04 05:20:08 2018 +0000" }, "message": "Improve tests for test_volume_basic_crud.py\n\nThis patch set addresses some quality of life improvements for\ntest_volume_basic_crud.py:\n\n* missing waiters\n* insufficient test granularity\n* add missing volume:get_all test (which is a fundamental API action)\n\n* Missing waiter is added for delete volume test.\n* create_volume test is broken up to be more granular, such that the\n waiter is outside the contextmanager inside which the role is switched.\n* test_list_volumes_with_details is added for complete coverage for the\n \"volume:get_all\" policy\n\nChange-Id: I5e11956afab2860a4d4fe0b8bc186b9fa967ce99\n" }, { "commit": "caffc23980285e4864085976fb482ca04d1ca7d2", "tree": "5dfccb57d6380b4167fbdb3148e86b21eda3901b", "parents": [ "839034139e8d47340f1edd75404bdbceb26e77bc", "d067148212ef95b938af5edbec359fa49156ddee" ], "author": { "name": "Zuul", "email": "zuul@review.openstack.org", "time": "Thu Aug 02 18:16:31 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "review@openstack.org", "time": "Thu Aug 02 18:16:32 2018 +0000" }, "message": "Merge \"Add tests to cover segments\"" }, { "commit": "839034139e8d47340f1edd75404bdbceb26e77bc", "tree": "57dcd0feec8c43e3d0e8c4fa9908e831dc6bd872", "parents": [ "9af45b13bca3564e758d0bd5ee83be9272dbb8e2" ], "author": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Mon Jul 09 16:33:55 2018 +0100" }, "committer": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Tue Jul 31 02:15:49 2018 -0400" }, "message": "Deprecate enable_rbac configuration option\n\nThe ``[patrole].enable_rbac`` option is deprecated and will be removed\nduring the \"S\" release. This is a legacy option that was meaningful\ndownstream when Patrole was a suite of tests inside Tempest itself. Now\nthat Patrole exists upstream as a Tempest plugin, it is paradoxical to\ninstall the Patrole plugin yet have an option that allows all Patrole\ntests to be skipped. This option is at odds with current Patrole\narchitecture.\n\nTo skip RBAC tests going forward, with Patrole Tempest plugin already\ninstalled, use an appropriate regex.\n\nChange-Id: I0cc39184c7891fb9133a45488a36ba220592915f\n" }, { "commit": "9af45b13bca3564e758d0bd5ee83be9272dbb8e2", "tree": "49057c741c0b9aea004fd4c5d034d173aff15fe0", "parents": [ "933c8922ef1c3b06c0a64391077c5ed82097a2ab", "d48d892355069aa8540d3b85d9a1c3dd5f7eda9d" ], "author": { "name": "Zuul", "email": "zuul@review.openstack.org", "time": "Mon Jul 30 17:39:48 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "review@openstack.org", "time": "Mon Jul 30 17:39:48 2018 +0000" }, "message": "Merge \"Remove misleading todo from config.py\"" }, { "commit": "d48d892355069aa8540d3b85d9a1c3dd5f7eda9d", "tree": "c8309d9ae3d359a6a59b592a8d8a27f347cfa054", "parents": [ "05a73499ce2eabb1f68013304d23ae09e38ba29c" ], "author": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Sun Jul 29 02:07:49 2018 -0400" }, "committer": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Sun Jul 29 02:07:49 2018 -0400" }, "message": "Remove misleading todo from config.py\n\nThis patch set removes a misleading todo from config.py which\nsuggests that it might be possible to leverage Keystone\u0027s\nPolicy API to derive policy information from remote hosts. But\nthis is not possible as the Policy API is deprecated [0].\n\nIf we want to get policy information from a remote host we will\nneed to pursue some SSH alternative instead, but this is not\nplanned anytime soon.\n\n[0] https://docs.openstack.org/releasenotes/keystone/queens.html#deprecation-notes\n\nChange-Id: I8a4da457c2d40ebddbfe5492dc1cbba87288c0da\n" }, { "commit": "d067148212ef95b938af5edbec359fa49156ddee", "tree": "996c038134eb1ac34b7d74e9f50273e381d41cdd", "parents": [ "05a73499ce2eabb1f68013304d23ae09e38ba29c" ], "author": { "name": "Mykola Yakovliev", "email": "VegasQ@gmail.com", "time": "Mon Jul 23 12:13:02 2018 -0500" }, "committer": { "name": "Mykola Yakovliev", "email": "VegasQ@gmail.com", "time": "Fri Jul 27 18:01:16 2018 -0500" }, "message": "Add tests to cover segments\n\nThis patch set adds tests to cver the neutron segments API [0].\nTest lists, gets, creates, updates, and deletes segments.\nPart of \"Increase Neutron RBAC Coverage\" initiative [1]\n\n[0] https://developer.openstack.org/api-ref/network/v2/index.html#segments\n[1] https://storyboard.openstack.org/#!/story/2002641\n\nStory: 2002641\nTask: 22306\nChange-Id: I8eef0dcd5355d6212ead2bcb109f23ca0b613109\n" }, { "commit": "933c8922ef1c3b06c0a64391077c5ed82097a2ab", "tree": "7fce291610ef8fbb2366b324ac04a0c9f827d61f", "parents": [ "1f923a1ae5dfb4c4441ab019eb39b0e6ef36e0e3", "2ee0db0726f80ffe0c29a5b24c91c06b988612b9" ], "author": { "name": "Zuul", "email": "zuul@review.openstack.org", "time": "Fri Jul 27 22:48:52 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "review@openstack.org", "time": "Fri Jul 27 22:48:52 2018 +0000" }, "message": "Merge \"Add list backups with details for backup:backup_project_attribute\"" }, { "commit": "1f923a1ae5dfb4c4441ab019eb39b0e6ef36e0e3", "tree": "5d0bf04afe3fb68275b3034e82257222e672b59d", "parents": [ "05a73499ce2eabb1f68013304d23ae09e38ba29c", "9d45e40db03daf4be48b3ce2dab607bf50a2d6c6" ], "author": { "name": "Zuul", "email": "zuul@review.openstack.org", "time": "Fri Jul 27 22:48:50 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "review@openstack.org", "time": "Fri Jul 27 22:48:50 2018 +0000" }, "message": "Merge \"Adds create_network tests for missing policy actions\"" }, { "commit": "2ee0db0726f80ffe0c29a5b24c91c06b988612b9", "tree": "80c509f35e7c6ee9f2c8d1c784f11a9aaa1ae7a4", "parents": [ "05a73499ce2eabb1f68013304d23ae09e38ba29c" ], "author": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Sun Jul 08 22:19:05 2018 -0400" }, "committer": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Fri Jul 27 00:29:31 2018 -0400" }, "message": "Add list backups with details for backup:backup_project_attribute\n\nThis patchset adds an RBAC test for validating that\n\"backup:backup_project_attribute\" is included in response body\nfor listing backups with details following successful policy\nauthorization. The policy in code documentation indicates that\nthis API enforces this policy [0] and the policy is enforced in\nthe code here [1]. The expected attribute comes from [2].\n\n[0] https://github.com/openstack/cinder/blob/8f7bc140052091d141e2cf436e989aa2d15dd6f4/cinder/api/v3/backups.py#L89\n[1] https://github.com/openstack/cinder/blob/8f7bc140052091d141e2cf436e989aa2d15dd6f4/cinder/policies/backups.py#L57\n[2] https://github.com/openstack/cinder/blob/8f7bc140052091d141e2cf436e989aa2d15dd6f4/cinder/api/v3/backups.py#L64\n\nChange-Id: I8285b25b97de90e1911adf5e97ffd051d10adf90\n" }, { "commit": "05a73499ce2eabb1f68013304d23ae09e38ba29c", "tree": "c99cf741bdba4230196ea26488e13a1a25a2b25d", "parents": [ "e737900e609783ff172dc86253f3af60e243be62", "1884c572ee472c4dfeb955f66c61a7a73474f682" ], "author": { "name": "Zuul", "email": "zuul@review.openstack.org", "time": "Fri Jul 27 03:35:48 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "review@openstack.org", "time": "Fri Jul 27 03:35:48 2018 +0000" }, "message": "Merge \"Increase test stability of backup RBAC tests\"" }, { "commit": "9d45e40db03daf4be48b3ce2dab607bf50a2d6c6", "tree": "0b4dfa88a1783438d93a94856d7d64c80365ec76", "parents": [ "e0c5c24e3bca25349894b8055589d88a16f4b894" ], "author": { "name": "jessegler", "email": "jess.egler@gmail.com", "time": "Tue Jul 17 13:54:41 2018 -0500" }, "committer": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Thu Jul 26 22:45:01 2018 -0400" }, "message": "Adds create_network tests for missing policy actions\n\n\u0027create_network:is_default\u0027\n\u0027create_network:provider:physical_network\u0027\n\nStory: 2002641\nTask: 22303\n\nChange-Id: I1aaa9b4cd9e0f34522e2e61191d1bae20ad6da48\n" }, { "commit": "e737900e609783ff172dc86253f3af60e243be62", "tree": "79088956200ea05bcee9b69a0999a26c71006dcc", "parents": [ "f8d1c74f51657cac279cf249ab47c782d1ec78db", "e91765592d5145744ba1c6f4c55637845a3833f4" ], "author": { "name": "Zuul", "email": "zuul@review.openstack.org", "time": "Tue Jul 24 06:35:17 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "review@openstack.org", "time": "Tue Jul 24 06:35:17 2018 +0000" }, "message": "Merge \"Docs: Add RBAC overview documentation\"" }, { "commit": "f8d1c74f51657cac279cf249ab47c782d1ec78db", "tree": "3abd4c13d37bcdc1b3440b6385b3c4ffdd435d04", "parents": [ "2a21633949555a79a34afe306d6ab299489a14a6" ], "author": { "name": "jessegler", "email": "jess.egler@gmail.com", "time": "Thu Jul 19 14:32:05 2018 -0500" }, "committer": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Sun Jul 22 02:14:40 2018 +0000" }, "message": "Adds the following tests for update_network:\n\n\u0027update_network:provider:network_type\u0027\n\u0027update_network:provider:physical_network\u0027\n\u0027update_network:provider:segmentation_id\u0027\n\nhttps://storyboard.openstack.org/#!/story/2002641\n\nChange-Id: I693746f2e2b816a1573b41a9e089ab9b7dd1e44a\n" }, { "commit": "1884c572ee472c4dfeb955f66c61a7a73474f682", "tree": "999e6dc41fad2c37b53f18208d8cee7e59462956", "parents": [ "2a21633949555a79a34afe306d6ab299489a14a6" ], "author": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Mon Mar 26 01:25:36 2018 -0400" }, "committer": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Fri Jul 20 21:22:35 2018 +0000" }, "message": "Increase test stability of backup RBAC tests\n\nThis is to increase test stability of backup RBAC tests which\nfail not only due to the rather error-prone nature of backing\nup volumes, but also because of data race conditions and\nmissing clean ups.\n\nThis adds some waiters following certain backup actions. This\nalso adds a clean up of the volume that is created when restoring\na backup [0].\n\nAdditional waiters are also added for waiting for volumes to\nbecome available following backup (because they briefly\ngo into backing-up status) [1].\n\n[0] https://developer.openstack.org/api-ref/block-storage/v3/index.html#restore-a-backup\n[1] https://review.openstack.org/#/c/569917/8\n\nChange-Id: Ifce15f69fcca38005b40785c42231ca164917345\n" }, { "commit": "e91765592d5145744ba1c6f4c55637845a3833f4", "tree": "76ed195c3a142cdb078ecec212433ae3be61ac52", "parents": [ "2e45d9b6929b3538d6d7b5bb04e95b3f822cdd7b" ], "author": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Mon Jul 16 14:39:55 2018 -0400" }, "committer": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Fri Jul 20 15:02:35 2018 +0000" }, "message": "Docs: Add RBAC overview documentation\n\nThis patchset adds an RBAC overview documentation section dedicated\nto:\n\n* Defining what RBAC is\n* Policy in code and validation info related to it\n* Custom policies and validation info related to it\n* Multiple policies and validation info related to it\n* Error codes\n* Glossary\n\nThis way, users can learn about what RBAC is from a high level\nand how Patrole uses validation to validate that it is working\ncorrectly.\n\nChange-Id: Ib411e4d06210135f7bd1cb90d5b6d59da2e5d076\n" }, { "commit": "2a21633949555a79a34afe306d6ab299489a14a6", "tree": "73ecd6fdec454691904e4c7f291fc6af3af5dd20", "parents": [ "853713df73e9a1715945ea7a03671493ce432da1", "fdc4514d749d90f4a7b0d0a9ada01bcf81db4c86" ], "author": { "name": "Zuul", "email": "zuul@review.openstack.org", "time": "Thu Jul 19 21:54:36 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "review@openstack.org", "time": "Thu Jul 19 21:54:36 2018 +0000" }, "message": "Merge \"trivial: Fix HACKING.rst nits causing rendering issues\"" }, { "commit": "853713df73e9a1715945ea7a03671493ce432da1", "tree": "3cf0a0f6a3302f0048c46b5067d3788275f00c81", "parents": [ "8779cf1ad1c368b9c1c924d3e7b44f06dd2fac44", "f6ffb8b1e0b5d5c2a3010d1fc053c48ec2471bb4" ], "author": { "name": "Zuul", "email": "zuul@review.openstack.org", "time": "Thu Jul 19 21:54:35 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "review@openstack.org", "time": "Thu Jul 19 21:54:35 2018 +0000" }, "message": "Merge \"Add README in the devstack folder\"" }, { "commit": "8779cf1ad1c368b9c1c924d3e7b44f06dd2fac44", "tree": "d88573b016355d2afdb0d7033b6e63091dd1c573", "parents": [ "07a79551e87c5fca43779578dcfe06afe5cf9c2b", "eb197db61f9248d7277884c2de6279c2fa163008" ], "author": { "name": "Zuul", "email": "zuul@review.openstack.org", "time": "Thu Jul 19 05:42:47 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "review@openstack.org", "time": "Thu Jul 19 05:42:47 2018 +0000" }, "message": "Merge \"Add REVIEWING documentation\"" }, { "commit": "fdc4514d749d90f4a7b0d0a9ada01bcf81db4c86", "tree": "54ab15db16dfd8e045f9441a6c95713701ff203c", "parents": [ "e0c5c24e3bca25349894b8055589d88a16f4b894" ], "author": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Wed Jul 18 20:54:20 2018 +0100" }, "committer": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Wed Jul 18 20:54:20 2018 +0100" }, "message": "trivial: Fix HACKING.rst nits causing rendering issues\n\nCurrently the HACKING.rst documentation is published here:\nhttps://docs.openstack.org/patrole/latest/HACKING.html\n\nAnd it is apparent that underneath the [0] section that\nsome of the hacking checks are incorrectly formatted\nlike [T109]. This fixes those issues.\n\n[0] https://docs.openstack.org/patrole/latest/HACKING.html#patrole-specific-commandments\n\nChange-Id: I306e972b23bc35672e8ef723bd5bc50f2f897882\n" }, { "commit": "f6ffb8b1e0b5d5c2a3010d1fc053c48ec2471bb4", "tree": "a7fcb34e4e551192b55161e86c2c8a883f2f7777", "parents": [ "e0c5c24e3bca25349894b8055589d88a16f4b894" ], "author": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Wed Jul 18 20:41:10 2018 +0100" }, "committer": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Wed Jul 18 20:52:05 2018 +0100" }, "message": "Add README in the devstack folder\n\nThis adds a README.rst in the devstack folder with information\nabout DevStack and how to install Patrole plugin in Devstack.\n\nChange-Id: I31a92351211a2f37403c08406215bc10f3c3222e\n" }, { "commit": "07a79551e87c5fca43779578dcfe06afe5cf9c2b", "tree": "728f7bb13d3b90ffd7b05c2603d12f9dcca115de", "parents": [ "51299a14146ec4f80fdd4eb42819dd0a4b8d5ee3", "299b9d467db7495835352e633538432685428d7d" ], "author": { "name": "Zuul", "email": "zuul@review.openstack.org", "time": "Wed Jul 18 19:01:37 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "review@openstack.org", "time": "Wed Jul 18 19:01:37 2018 +0000" }, "message": "Merge \"Bump hacking to 1.1.0\"" }, { "commit": "eb197db61f9248d7277884c2de6279c2fa163008", "tree": "fc358bd1e12348d710eb6b2b4e762fa730dd4a44", "parents": [ "561d028a636fc0982f57319a2fb78039dc9c700c" ], "author": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Sun Jun 10 12:21:49 2018 -0400" }, "committer": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Tue Jul 17 23:03:32 2018 -0400" }, "message": "Add REVIEWING documentation\n\nThis patchset adds reviewing documentation to Patrole which\nis very similar to Tempest\u0027s reviewing documentation, except that\nit omits sections that aren\u0027t so relevant (like requirements around\ndocstrings because currently Patrole has no such requirements) but\nadds sections related to policy concerns.\n\nChange-Id: I25c3a4b73f1d4f8beb7bce9c694f4bb3f904e038\n" }, { "commit": "51299a14146ec4f80fdd4eb42819dd0a4b8d5ee3", "tree": "9f6655b634cd87238cafadf4c53d1f48d7009f2a", "parents": [ "561d028a636fc0982f57319a2fb78039dc9c700c" ], "author": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Thu Jun 28 20:03:27 2018 -0400" }, "committer": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Wed Jul 18 01:18:35 2018 +0000" }, "message": "Improve exception that is raised following invalid service\n\nThis patchset changes the exception that is raised following\nan invalid service being passed in for testing. Instead of\na NotFound getting raised (which doesn\u0027t make sense), the\noriginal exception, RbacInvalidServiceException, is raised\ninstead. In addition \"Exception\" is appended to the original\nexception name for greater clarity and consistency with other\nexception names.\n\nThis patchset is similar to [0] which aims to simplify and\nconsolidate how service input is validated.\n\n[0] https://review.openstack.org/#/c/577963/\n\nChange-Id: Ib91a6581f89a528630d756840176b0e16663fa6f\n" }, { "commit": "561d028a636fc0982f57319a2fb78039dc9c700c", "tree": "9640832baa3e3c45b498698daf34dd6d751cb2c7", "parents": [ "2e45d9b6929b3538d6d7b5bb04e95b3f822cdd7b" ], "author": { "name": "ghanshyam", "email": "gmann@ghanshyammann.com", "time": "Tue Jul 17 04:45:49 2018 +0000" }, "committer": { "name": "ghanshyam", "email": "gmann@ghanshyammann.com", "time": "Tue Jul 17 04:45:49 2018 +0000" }, "message": "Cap additional nova-network tests at Queens\n\nAs noted in I7aea99d7c1818b8edcda41ba4eaa062a8ea069eb, nova is working\non removing the nova-network only compute REST APIs in the Rocky\nrelease, which means any requests to those APIs after Queens will result\nin a 410 response, regardless of microversion or whether or not neutron\nis being used.\n\nThe os-floating-ips-bulk API is being removed in nova change\nI89d081108b398d8efba9636279088c61349b21e6. As a result,\nos-floating-ips-bulk tests will no longer works against Rocky+ versions\nof nova. Skip this test using the check added in the previous change.\n\nRelated to blueprint remove-nova-network\n\nChange-Id: I1a2accebce29c3be6acec7ede67b1250552a42ca\n" }, { "commit": "2e45d9b6929b3538d6d7b5bb04e95b3f822cdd7b", "tree": "92d4067e8e1831f8c77bb6c333693ecea5dbc6b5", "parents": [ "68149b810243c96aca6a440b6255cd5047fc343d", "e5ee4be8b320ca4ed093edff2e7f2a090ce65575" ], "author": { "name": "Zuul", "email": "zuul@review.openstack.org", "time": "Mon Jul 16 18:12:17 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "review@openstack.org", "time": "Mon Jul 16 18:12:17 2018 +0000" }, "message": "Merge \"docs: Add symlink to readme from overview documentation\"" }, { "commit": "68149b810243c96aca6a440b6255cd5047fc343d", "tree": "e7be95b84cade6e57101c632df41af416c0c72db", "parents": [ "e0c5c24e3bca25349894b8055589d88a16f4b894", "1bbf48e617de3f5d515a4470d8ec2ac8c0bae04a" ], "author": { "name": "Zuul", "email": "zuul@review.openstack.org", "time": "Mon Jul 16 16:54:42 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "review@openstack.org", "time": "Mon Jul 16 16:54:42 2018 +0000" }, "message": "Merge \"followup: Include spec/discussion references\"" }, { "commit": "e0c5c24e3bca25349894b8055589d88a16f4b894", "tree": "7caf91d8bf0abd776741c255fd82f4de5fe8b2df", "parents": [ "94986d7e3c25b842f90036d6971f103403f0c3f2", "c0cb7eb2d49213afa209de87dfbeb376775c3977" ], "author": { "name": "Zuul", "email": "zuul@review.openstack.org", "time": "Tue Jul 10 20:57:10 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "review@openstack.org", "time": "Tue Jul 10 20:57:10 2018 +0000" }, "message": "Merge \"multi policy: Fix logging issues with multiple policies\"" }, { "commit": "94986d7e3c25b842f90036d6971f103403f0c3f2", "tree": "4f5592f69cf0061c5f4323c5125e967c705b8c37", "parents": [ "f800a1c5ca672033cd9e283c82972b87d0911648", "f16b6b3a1df654cfc98256ab1dd92ad4e719c369" ], "author": { "name": "Zuul", "email": "zuul@review.openstack.org", "time": "Tue Jul 10 19:54:19 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "review@openstack.org", "time": "Tue Jul 10 19:54:19 2018 +0000" }, "message": "Merge \"Update overpermission/underpermission rbac exceptions\"" }, { "commit": "299b9d467db7495835352e633538432685428d7d", "tree": "ff87cc98ff516c1a61b3119b65865dc6885cea8b", "parents": [ "556094d276d79b7f320e5557aee93d7f029a1363" ], "author": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Fri Jul 06 23:00:24 2018 -0400" }, "committer": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Tue Jul 10 18:40:19 2018 +0000" }, "message": "Bump hacking to 1.1.0\n\nhacking 1.0+ adds a couple of additional checks that are helpful as part\nof the transition to Python 3. However, hacking 1.1.0 incorporated\nflake8 2.6.x, which transitioned from pep8 to pycodestyle. Tempest\nprovides some hacking extensions and these use pep8 imports. As things\nstand, projects that wish to use these hacking extensions must either\nconstrain themselves to the same older hacking versions or install pep8\nmanually. Neither of these approaches is particularly attractive so\ninstead transition to hacking 1.1.0 ourselves. This has the benefit of\ngiving us the additional checks for Python 3.\n\nBased off https://review.openstack.org/#/c/580634/1/\n\nChange-Id: I8ed5aa0ccd02a792d26d3bef1949a7e1a3f600ed\n" }, { "commit": "1bbf48e617de3f5d515a4470d8ec2ac8c0bae04a", "tree": "ba35ef6bed5f375ee259e3326eda330a2d86cc49", "parents": [ "f800a1c5ca672033cd9e283c82972b87d0911648" ], "author": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Tue Jul 10 18:43:47 2018 +0100" }, "committer": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Tue Jul 10 18:43:47 2018 +0100" }, "message": "followup: Include spec/discussion references\n\nThis patchset follows up on [0] by addressing\nfeedback left in the review section related to\nadding background references:\n\n* https://review.openstack.org/#/c/382672/\n* https://specs.openstack.org/openstack/qa-specs/specs/tempest/rbac-policy-testing.html\n\nFor the purposes of corrborating the documentation\ncontent related to \"Why are these tests not in Tempest?\"\n\n[0] https://review.openstack.org/#/c/580903/4\n\nChange-Id: I69ca13cf16f0d033419a7ea086bf0605682c2dbc\n" }, { "commit": "f800a1c5ca672033cd9e283c82972b87d0911648", "tree": "467210e54a86028cb1c1fc5701268b460dea60be", "parents": [ "4671e3909def0fa75c7cdea2ea44b02f0fdd48fd" ], "author": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Mon Jul 09 00:23:46 2018 -0400" }, "committer": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Mon Jul 09 01:33:06 2018 -0400" }, "message": "Add documentation explaining why Patrole tests aren\u0027t in Tempest\n\nThere have been concerns raised as to why Patrole tests aren\u0027t\ncontained in Tempest (the concerns are not found in any written\ndiscussion online). This documentation puts such concerns to\nrest. It was agreed upon in the RBAC testing spec [0] that:\n\"rbac tests will live in separate tempest plugin.\" This\ndocumentation formalizes that reasoning.\n\n[0] comment in https://review.openstack.org/#/c/382672/\n\nChange-Id: I31d956b42440a5448a5be0a7e2c5b3b7ddacfab5\n" }, { "commit": "e5ee4be8b320ca4ed093edff2e7f2a090ce65575", "tree": "197a006fdc44f7429341adf89151b91d0ee83460", "parents": [ "556094d276d79b7f320e5557aee93d7f029a1363" ], "author": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Mon Jun 18 21:39:28 2018 +0100" }, "committer": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Thu Jul 05 15:38:20 2018 +0000" }, "message": "docs: Add symlink to readme from overview documentation\n\nThis patchset brings the README.rst and the overview.rst\ndocumentation together. Previously they were maintained\nseparately which causes maintenance issues. This adds\na symlink from the overview.rst to the README.rst so\nthat the overview documentation only needs to be maintained\nfor the README.\n\nThis also adds a Terminology subsection underneath the\n\"How it works\" section which expands on essential terminology\nneeded to understand Patrole testing architecture:\n\n* Expected Result - The expected result of a given test.\n* Actual Result - The actual result of a given test.\n* Final Result - A match between both expected and actual results. A mismatch\n in the expected result and the actual result will result in a test failure.\n\n * Expected: Pass | Actual: Pass - Test Case Success\n * Expected: Pass | Actual: Fail - Test Case Failure\n * Expected: Fail | Actual: Pass - Test Case Failure\n * Expected: Fail | Actual: Fail (Expected exception) - Test Case Success\n * Expected: Fail | Actual: Fail (Unexpected exception) - Test Case Failure\n\nChange-Id: I1d640200c55ce26cfd38197ec6face1161217b17\n" }, { "commit": "4671e3909def0fa75c7cdea2ea44b02f0fdd48fd", "tree": "61fe6a007f22f0253438e5dcc46edf98de84eeb1", "parents": [ "a85698bcb8d2be09359aa575f3a5e5e6d4eee73c", "1bed58be501ac22ef8f6c01f1e1e77fe2c6129a8" ], "author": { "name": "Zuul", "email": "zuul@review.openstack.org", "time": "Wed Jul 04 02:53:53 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "review@openstack.org", "time": "Wed Jul 04 02:53:53 2018 +0000" }, "message": "Merge \"Add fixed ip cleanup to test_add_fixed_ip\"" }, { "commit": "f16b6b3a1df654cfc98256ab1dd92ad4e719c369", "tree": "d757fcb089c63b21c7fffd8db1f8fc94c6f6e942", "parents": [ "556094d276d79b7f320e5557aee93d7f029a1363" ], "author": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Thu Jun 28 19:32:59 2018 -0400" }, "committer": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Fri Jun 29 04:51:33 2018 +0000" }, "message": "Update overpermission/underpermission rbac exceptions\n\nThis patchset updates rbac_exceptions by bringing the concept\nof under-permission and over-permission together. An over-permission\noccurs when an unauthorized role is allowed to perform an action\nand an under-permission occurs when an authorized role is not\nallowed to perform an action. Both of these are important failure\nscenarios.\n\nCurrent Patrole has an RbacOverPermission Exception but uses\na \"Forbidden\" as a pseudonym for the under-permission version\nbut this is not ideal for the following reasons:\n\n* Patrole can expect a 404 Not Found due to Neutron policy enforcement [0]\n* The naming is inconsistent with RbacOverPermission\n* It should have a Patrole wrapper exception (NotFound is used directly\n from Tempest)\n\nSo, this patchset:\n\n* renames RbacOverPermission to RbacOverPermissionException\n* replaces Forbidden exception with RbacUnderPermissionException\n* updates documentation, docstrings and unit tests\n\nIn addition, this patchset introduces a new exception called\nRbacExpectedWrongException which is raised when the expected\nexception does not match the actual exception and both are instances\nof 403 and 404, which means that the RBAC test uses the wrong\nexpected_error_codes.\n\nChange-Id: I681610448cbe0269f02c34ea6afaaaf29c306121\n" }, { "commit": "a85698bcb8d2be09359aa575f3a5e5e6d4eee73c", "tree": "a8f3f18c3ee0b6ed980cb4820a6aef25f1770265", "parents": [ "2ed9feac24eb6cf1919effe95313fe703486a2e3", "f5ed588fb55a9a4fc1f03e7f1e344266d9742b4f" ], "author": { "name": "Zuul", "email": "zuul@review.openstack.org", "time": "Thu Jun 28 23:24:51 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "review@openstack.org", "time": "Thu Jun 28 23:24:51 2018 +0000" }, "message": "Merge \"trivial: Drop empty services folder under patrole_tempest_plugin\"" }, { "commit": "2ed9feac24eb6cf1919effe95313fe703486a2e3", "tree": "75637309ae72b987b0d51ba0a032520b11a2e485", "parents": [ "a6371ed510b318183e1c2cbafcc97bd85f6625f8", "318fa3b80c8d3ff49ec3d26db98d063530e65b35" ], "author": { "name": "Zuul", "email": "zuul@review.openstack.org", "time": "Wed Jun 27 17:29:19 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "review@openstack.org", "time": "Wed Jun 27 17:29:19 2018 +0000" }, "message": "Merge \"Use tempest-multinode-full parent for multinode Zuul jobs\"" }, { "commit": "a6371ed510b318183e1c2cbafcc97bd85f6625f8", "tree": "1afd60ae5b235491f6c44feaaa549f7a1f7f791e", "parents": [ "556094d276d79b7f320e5557aee93d7f029a1363", "d91c1242c3396594495d2bd905bc42b24a8f29ba" ], "author": { "name": "Zuul", "email": "zuul@review.openstack.org", "time": "Wed Jun 27 17:03:16 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "review@openstack.org", "time": "Wed Jun 27 17:03:16 2018 +0000" }, "message": "Merge \"Change Neutron tests to use multi-policy support\"" }, { "commit": "d91c1242c3396594495d2bd905bc42b24a8f29ba", "tree": "d3a712fd1731701910803fe6e5fca5ea905755e0", "parents": [ "e965f90cafe8b45913ab51782afcc9c7c47a9fed" ], "author": { "name": "Cliff Parsons", "email": "cliff.parsons@att.com", "time": "Fri Jun 15 00:50:33 2018 -0500" }, "committer": { "name": "Cliff Parsons", "email": "cliff.parsons@att.com", "time": "Tue Jun 26 16:39:35 2018 +0000" }, "message": "Change Neutron tests to use multi-policy support\n\nThis change changes the test cases that require multi-policy and\nmultiple error code handling.\n\nPartially Implements: bp rbac-testing-multiple-policies\nStory: 2002620\n\nChange-Id: I40df70f7cd7c7f30822010323c5dd02697d50d2b\n" }, { "commit": "f5ed588fb55a9a4fc1f03e7f1e344266d9742b4f", "tree": "5c83134faf6f60c191c9e575a86d8185b099048c", "parents": [ "556094d276d79b7f320e5557aee93d7f029a1363" ], "author": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Mon Jun 25 21:29:11 2018 -0400" }, "committer": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Mon Jun 25 21:29:11 2018 -0400" }, "message": "trivial: Drop empty services folder under patrole_tempest_plugin\n\nThis patchset drops the empty services folder under\npatrole_tempest_plugin. While it is part of the cookie cutter\nTempest plugin setup, it is currently empty and serves no\npurpose in Patrole. If in the future we decide to add service\nclients there, we can reintroduce the folder. But there is\nno purpose in keeping around an empty folder -- it just\nincreases clutter.\n\nChange-Id: Ibf83a73f5dc2c38d737359221e9974a6562846a6\n" }, { "commit": "556094d276d79b7f320e5557aee93d7f029a1363", "tree": "ce280f2cd05bc190de13bfb0d2b5539c3afa7d7a", "parents": [ "e965f90cafe8b45913ab51782afcc9c7c47a9fed", "e7679f7f2d43e40f2d3399c495fba3e9be476acd" ], "author": { "name": "Zuul", "email": "zuul@review.openstack.org", "time": "Sat Jun 23 01:34:32 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "review@openstack.org", "time": "Sat Jun 23 01:34:32 2018 +0000" }, "message": "Merge \"Fix Neutron-related return values on some list APIs\"" }, { "commit": "318fa3b80c8d3ff49ec3d26db98d063530e65b35", "tree": "0f93d9b2e6e18d5b5294a06dac0dd2b8e4c7e47d", "parents": [ "8f45b364f722fa87be0e572108f98fc3d402a23c" ], "author": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Tue Jun 19 16:53:33 2018 -0400" }, "committer": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Wed Jun 20 00:25:06 2018 -0400" }, "message": "Use tempest-multinode-full parent for multinode Zuul jobs\n\nThis patchset uses the recently added [0] tempest-multinode-full\nparent for multinode Patrole Zuulv3 jobs. This patchset also\nfixes some trivial docstring errors related to [1].\n\n[0] https://review.openstack.org/#/c/545724/\n[1] https://review.openstack.org/#/c/570262/\n\nChange-Id: Ib13663b582f0c57a28ec4b18129b83ab5ba95cfe\n" }, { "commit": "e965f90cafe8b45913ab51782afcc9c7c47a9fed", "tree": "19ef5ec4fa5e62263089ad98bdebdcb92a46b28c", "parents": [ "25e2e71b95034743ddf26520ff369d941f39e2b3", "543f7b9b8a2ac880c9e86b1e916311721a05d33c" ], "author": { "name": "Zuul", "email": "zuul@review.openstack.org", "time": "Wed Jun 20 02:49:47 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "review@openstack.org", "time": "Wed Jun 20 02:49:47 2018 +0000" }, "message": "Merge \"docs: Add Patrole overview documentation\"" }, { "commit": "c0cb7eb2d49213afa209de87dfbeb376775c3977", "tree": "1719f342fed721ea8c7c888a7cf0102ec8980f1e", "parents": [ "25e2e71b95034743ddf26520ff369d941f39e2b3" ], "author": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Tue Jun 19 19:50:36 2018 -0400" }, "committer": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Tue Jun 19 19:50:36 2018 -0400" }, "message": "multi policy: Fix logging issues with multiple policies\n\nThis patchset mostly aims to fix logging issues surrounding\nmultiple policies introduced by [0]. Some of the LOG messages\nstill only log `rule` arg when they should log all the `rules`\narg instead to capture all the policies being enforced.\n\nIn addition, this patchset:\n\n* corrects docstring formatting with rbac_rule_validation following [1]\n* changes unit tests in test_rbac_rule_validation to test non-deprecated\n arg `rules` instead of `rule` (tests also exist for ensuring that\n `rule` is handled correctly until it is finally removed from code)\n* rewords some log messages to better conform to multi policy support\n\n[0] https://review.openstack.org/#/c/554764/\n[1] https://review.openstack.org/#/c/570262/\n\nPartially Implements: bp rbac-testing-multiple-policies\nStory: 2002620\nTask: 22250\n\nChange-Id: I3df69d5a476416378bde14ee6a258ff5ed40e363\n" }, { "commit": "25e2e71b95034743ddf26520ff369d941f39e2b3", "tree": "f8319d9d6e76f0c5ab207223ac1c848394b1cbf0", "parents": [ "8f45b364f722fa87be0e572108f98fc3d402a23c", "bc865dfecc319cbca8aeb31687ee0ba6dcea9d12" ], "author": { "name": "Zuul", "email": "zuul@review.openstack.org", "time": "Tue Jun 19 22:32:34 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "review@openstack.org", "time": "Tue Jun 19 22:32:34 2018 +0000" }, "message": "Merge \"Fix test_volume_upload BadRequest error\"" }, { "commit": "8f45b364f722fa87be0e572108f98fc3d402a23c", "tree": "1b08659d0a596cd79914c7e8938b5688a9019241", "parents": [ "f50f6f1384040ee4a1fa806bb9a39bfd6cf429e5", "35a77113fccfd5659c123adb2cc142836cdd764d" ], "author": { "name": "Zuul", "email": "zuul@review.openstack.org", "time": "Tue Jun 19 20:10:28 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "review@openstack.org", "time": "Tue Jun 19 20:10:28 2018 +0000" }, "message": "Merge \"Add support for handling multiple error codes\"" }, { "commit": "543f7b9b8a2ac880c9e86b1e916311721a05d33c", "tree": "691569f059be7d3d258e1c9aa70e2a69f6c5ea1c", "parents": [ "f50f6f1384040ee4a1fa806bb9a39bfd6cf429e5" ], "author": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Sun Jun 10 13:38:31 2018 -0400" }, "committer": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Tue Jun 19 16:05:36 2018 -0400" }, "message": "docs: Add Patrole overview documentation\n\nThis patchset adds a Patrole overview documentation\nsection which is sorely needed. It combines the previous\nusage documentation into it.\n\nChange-Id: Ia7412f2e99f33fbdfd2e60ba54ffdba757d1f886\n" }, { "commit": "f50f6f1384040ee4a1fa806bb9a39bfd6cf429e5", "tree": "d18407ddb99a8fdc1158d3afcfa491dab35a791b", "parents": [ "ddac38bbac813ac1b2857b23462fb13027956c48", "68f4c2f382ecf3f96d0759eef9d6cf0f7d257fe6" ], "author": { "name": "Zuul", "email": "zuul@review.openstack.org", "time": "Tue Jun 19 16:29:32 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "review@openstack.org", "time": "Tue Jun 19 16:29:32 2018 +0000" }, "message": "Merge \"fix tox python3 overrides\"" }, { "commit": "ddac38bbac813ac1b2857b23462fb13027956c48", "tree": "e387df0cdcea6ec9716a91dcfc4f6f76ad979adb", "parents": [ "6962e8afa2de51821c6392df6ae08d8ebec64299" ], "author": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Thu Jun 14 15:36:03 2018 -0400" }, "committer": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Mon Jun 18 23:49:38 2018 +0000" }, "message": "Limit .zuul.yaml gate jobs to most important jobs only\n\nThis patchset limits the jobs enqueued in the gate pipeline in\nZuul to the 2 most important jobs only:\n\n- patrole-admin\n- patrole-member\n\nEverything else will remain in the check pipeline. This means\nthat the chances of errant gate failures will be significantly\nreduced. At the same time, test coverage will remain the same.\nNow, however, all jobs will run once during the check pipeline\n(when commits are first submitted) and, after cores approve\nof a patchset, only the 2 jobs above (in addition to unit test\njobs, pep8, etc.) will run [0].\n\nIt is extremely unlikely that a patchset B that is merged between\npatchset A\u0027s check and gate pipelines will cause a breaking change\nto enter the source code -- and even so, this is an acceptable\nmargin of error when balanced against developer workflow being\nslowed down in some cases severely due to many, many spurious\nfailures across volume and server APIs related to driver instability\n(not RBAC), across multiple jobs enqueued in the gate pipeline.\n\n[0] https://docs.openstack.org/infra/system-config/zuul.html\n\nChange-Id: I57772e00d4e96a8ac841651b7f36379629940956\n" }, { "commit": "1bed58be501ac22ef8f6c01f1e1e77fe2c6129a8", "tree": "c2e5e79630ccd6bc08a1a6bb4ef1575c4fdfd26e", "parents": [ "ffc1ad8556b3aee842db5345ee4080d8cae7c303" ], "author": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Mon Jun 18 21:57:53 2018 +0100" }, "committer": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Mon Jun 18 21:57:53 2018 +0100" }, "message": "Add fixed ip cleanup to test_add_fixed_ip\n\nThere is a data race on clean up with the test test_add_fixed_ip\nbecause it can currently fail during resource_cleanup [0] with:\n\"One or more ports have an IP allocation from this subnet.\"\n\n[0] http://logs.openstack.org/36/575536/4/gate/patrole-admin/ccf48f1/testr_results.html.gz\n\nChange-Id: I71099c62cb9c49739ec21579f24d4fd8a9cbf432\n" }, { "commit": "6962e8afa2de51821c6392df6ae08d8ebec64299", "tree": "41a43c737bf262e6e264204311229ba6b47db0f3", "parents": [ "2fc2929882c211682219c5ab71f06e5954fc7b53", "ece376d3a4f760b8493e7d7d02f5174d01cdc7d3" ], "author": { "name": "Zuul", "email": "zuul@review.openstack.org", "time": "Mon Jun 18 17:49:41 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "review@openstack.org", "time": "Mon Jun 18 17:49:41 2018 +0000" }, "message": "Merge \"Remove duplicate test_unmanage_volume test\"" }, { "commit": "e7679f7f2d43e40f2d3399c495fba3e9be476acd", "tree": "a8a7e250b1e6fdc0045c75065b3b2b87a24c1aaa", "parents": [ "6962e8afa2de51821c6392df6ae08d8ebec64299" ], "author": { "name": "Cliff Parsons", "email": "cliff.parsons@att.com", "time": "Sat Jun 16 18:42:29 2018 -0500" }, "committer": { "name": "Cliff Parsons", "email": "cliff.parsons@att.com", "time": "Sun Jun 17 19:47:36 2018 -0500" }, "message": "Fix Neutron-related return values on some list APIs\n\nThere are a handful of APIs in Neutron where instead of returning\nan error code for a disallowed role, it returns an empty list. The\ntest cases for these APIs does not currently handle this.\n\nChange-Id: Ice15b4d27dae72963281a31ba58fee1f373a9690\n" }, { "commit": "35a77113fccfd5659c123adb2cc142836cdd764d", "tree": "b35dbf96aaec39fd83b3d07f7672b3bbbc8b4983", "parents": [ "2fc2929882c211682219c5ab71f06e5954fc7b53" ], "author": { "name": "Cliff Parsons", "email": "cliff.parsons@att.com", "time": "Mon May 07 14:03:40 2018 -0500" }, "committer": { "name": "Cliff Parsons", "email": "cliff.parsons@att.com", "time": "Sun Jun 17 04:37:17 2018 +0000" }, "message": "Add support for handling multiple error codes\n\nPatrole currently cannot handle the scenario where two possible\nerror codes can returned by Neutron policy enforcement for a\nfailed policy check (403 Forbidden and 404 NotFound), depending\non what role is being tested. Patrole framework can only handle\none expected_exception.\n\nThis change builds upon the recent multi-policy support to allow\nthe tester to specify multiple policy actions for one API test.\nFor each policy action, the tester would need to specify an\nerror code that is expected if the action should fail. If multiple\npolicy actions fail, the error code for the first policy action\nthat fails will be expected to be returned from the service.\n\nThis handles the cases in Neutron where Neutron may use a second\npolicy rule to determine whether or not to return a 403 error\ncode or a 404 error code. The tester is expected to list out\nwhich policy rules are being tested by the API endpoint test.\n\nChange-Id: I5cd861e184da90bb27f8ba454c94fa4d4f99c269\nCloses-Bug: #1772710\n" }, { "commit": "2fc2929882c211682219c5ab71f06e5954fc7b53", "tree": "cbd212581aefb64e4f4b99a8769c03a784f6d29c", "parents": [ "ffc1ad8556b3aee842db5345ee4080d8cae7c303" ], "author": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Fri Jun 15 18:26:27 2018 -0400" }, "committer": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Sat Jun 16 16:20:16 2018 -0400" }, "message": "Gate fix: Rename Member (legacy) to member role\n\nDue to a recent change [0], Member role is no longer\nbeing found, as it has been renamed to member. This is\ncausing all the member-based gates to fail. Because \"Member\"\nis legacy [1], this patchset uses \"member\" instead of \"Member\"\nduring the devstack Patrole plugin for master. For n-1\nand n-2 releases \"Member\" is still used.\n\nThis patchset also specifies which role was not found in\nthe system while trying to resolve roles CONF.identity.admin_role\nand CONF.patrole.rbac_test_role in order to make debugging\neasier.\n\n[0] https://review.openstack.org/#/c/572243/\n[1] http://git.openstack.org/cgit/openstack-dev/devstack/tree/lib/keystone#n355\n\nChange-Id: I7b59bab164041b26ed8a1a798546e493f22f6edd\n" }, { "commit": "ffc1ad8556b3aee842db5345ee4080d8cae7c303", "tree": "236a15893591e159e1102be706b5971deb3f82b3", "parents": [ "cd61bb685becf1f25f373067d683b1a15824e94a", "57ab8c46f2e2082c71d1e23f96f57472d5e216a0" ], "author": { "name": "Zuul", "email": "zuul@review.openstack.org", "time": "Fri Jun 15 02:38:21 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "review@openstack.org", "time": "Fri Jun 15 02:38:21 2018 +0000" }, "message": "Merge \"Remove duplicate subnet tests fromt test_networks_rbac module\"" }, { "commit": "cd61bb685becf1f25f373067d683b1a15824e94a", "tree": "0eeeb3c8e269f833eebf2531ad02ea7f90ae5a8b", "parents": [ "5da0aad1996835066a038194cd3811c52b222f60", "383960df5710c599334532c0177cbefee9c49d33" ], "author": { "name": "Zuul", "email": "zuul@review.openstack.org", "time": "Fri Jun 15 01:27:38 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "review@openstack.org", "time": "Fri Jun 15 01:27:38 2018 +0000" }, "message": "Merge \"Drop old README file\"" }, { "commit": "5da0aad1996835066a038194cd3811c52b222f60", "tree": "f263e151a5fbb25a57fa050ec4ce695016741ddd", "parents": [ "c9bc9db4b45b8112cee9d8e06304e620d0a5c13c" ], "author": { "name": "Rick Bartra", "email": "rb560u@att.com", "time": "Tue Jun 12 14:28:50 2018 -0400" }, "committer": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Thu Jun 14 11:26:21 2018 -0400" }, "message": "Gate fix: Cap nova-network tests at Queens\n\nNova is working on removing the nova-network only\ncompute REST APIs in the Rocky release, which means\nany requests to those APIs after Queens will result\nin a 410 response, regardless of microversion or\nwhether or not neutron is being used.\n\nThe os-virtual-interfaces API is being removed in\nnova change: https://review.openstack.org/569923/\n\nAs a result, os-virtual-interfaces tests will no longer\nworks against Rocky+ versions of nova.\n\nnova-network tests are skipped by adding\n`depends_on_nova_network \u003d True` to the test class. The\nbase compute API class does a check to determine, based\non the max compute API microversion, which release is\nbeing tested and if Rocky+ the nova-network specific\ntests will be skipped.\n\nSubsequent nova-network specific tests will be skipped\nas the API removal changes are proposed in nova.\n\nRelated to blueprint remove-nova-network\n\nChange-Id: I1d8d9aba6f44ee451e8e4d93a3e5bf5e70f7bbb3\n" }, { "commit": "c9bc9db4b45b8112cee9d8e06304e620d0a5c13c", "tree": "651896d395176ecd9b15f367a842e553beac81fb", "parents": [ "8d99172e2a55a2292ea0b733b43a4a7daefe5af3", "3056b9865bae5920dbd82e38f7b981fd497381d6" ], "author": { "name": "Zuul", "email": "zuul@review.openstack.org", "time": "Sun Jun 10 18:04:38 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "review@openstack.org", "time": "Sun Jun 10 18:04:38 2018 +0000" }, "message": "Merge \"Wait for interface to detach for all interface tests\"" }, { "commit": "383960df5710c599334532c0177cbefee9c49d33", "tree": "096bb1e83730d2898933c7ad01a9d26489309d07", "parents": [ "8d99172e2a55a2292ea0b733b43a4a7daefe5af3" ], "author": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Sun Jun 10 13:39:05 2018 -0400" }, "committer": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Sun Jun 10 13:39:05 2018 -0400" }, "message": "Drop old README file\n\nThis patchset drops an old README file (not the main README file)\nthat contains no content. If we want to include README content\nunder patrole_tempest_plugin package we can do so later, but we\nshould not keep an empty documentation page around for nothing.\n\nChange-Id: I7e524e849a02732e70c5109795f422ac7f7cba25\n" }, { "commit": "8d99172e2a55a2292ea0b733b43a4a7daefe5af3", "tree": "ba9a5c083444429aa535903b9fa442abd4e31fb0", "parents": [ "d0ddf7d45044b2c8a9395a8f0df0c6c6adec9399", "8251477dde1d12d9b325ed3f9ab8caa047f25b93" ], "author": { "name": "Zuul", "email": "zuul@review.openstack.org", "time": "Sun Jun 10 05:11:29 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "review@openstack.org", "time": "Sun Jun 10 05:11:29 2018 +0000" }, "message": "Merge \"Refactor `test_confirm_resize_server` to pass for all roles\"" }, { "commit": "ece376d3a4f760b8493e7d7d02f5174d01cdc7d3", "tree": "a532d787b8c228fc5d1040eb9f9a696bcbce1ac1", "parents": [ "9f42552b1aeb615cc7c705156fe0392184e581e8" ], "author": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Sat Jun 09 23:01:55 2018 -0400" }, "committer": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Sat Jun 09 23:01:55 2018 -0400" }, "message": "Remove duplicate test_unmanage_volume test\n\nThis patchset removes the duplicate test_umanage_volume test\nfrom test_volume_actions_rbac [0] because it is more appropriate\nto keep it in test_volumes_manage_rbac [1]. Besides the latter\nimplementation is more robust.\n\n[0] https://github.com/openstack/patrole/blob/1f2ba24f4df09ba3711aa520bdf1bedb63e7797a/patrole_tempest_plugin/tests/api/volume/test_volume_actions_rbac.py#L147\n[1] https://github.com/openstack/patrole/blob/7676a21077b5d8078f5beb32f7186970f02fc1a1/patrole_tempest_plugin/tests/api/volume/test_volumes_manage_rbac.py#L113\n\nChange-Id: I96705df2cfaf9e0885e53de15675581508dd7c3c\n" }, { "commit": "57ab8c46f2e2082c71d1e23f96f57472d5e216a0", "tree": "f0f44d49c174213c7649318287668bb5d234ff8b", "parents": [ "9f42552b1aeb615cc7c705156fe0392184e581e8" ], "author": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Sat Jun 09 22:16:59 2018 -0400" }, "committer": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Sat Jun 09 22:16:59 2018 -0400" }, "message": "Remove duplicate subnet tests fromt test_networks_rbac module\n\nThis patchset simply removes duplicate subnet tests from\ntest_networks_rbac [0] as they are already covered in\ntest_subnets_rbac [1].\n\n[0] e.g. https://github.com/openstack/patrole/blob/d0ddf7d45044b2c8a9395a8f0df0c6c6adec9399/patrole_tempest_plugin/tests/api/network/test_networks_rbac.py#L300\n[1] e.g. https://github.com/openstack/patrole/blob/d0ddf7d45044b2c8a9395a8f0df0c6c6adec9399/patrole_tempest_plugin/tests/api/network/test_subnets_rbac.py#L42\n\nChange-Id: Iea9eb984b06ab49822cf56021df3739c30a97d03\n" }, { "commit": "3056b9865bae5920dbd82e38f7b981fd497381d6", "tree": "659de49873f7c41fcff42b192351982d4d48d99d", "parents": [ "ea1c9e7cffb4b822d510d46dcaef76fab5999f20" ], "author": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Thu May 31 23:23:49 2018 -0400" }, "committer": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Sat Jun 09 15:21:01 2018 +0000" }, "message": "Wait for interface to detach for all interface tests\n\nThis patchset waits for interfaces to detach during\ntest clean up in MiscPolicyActionsNetworkRbacTest.\nThis is a follow up to [0], which missed clean up\nfor certain tests.\n\n[0] I7b1a095bb73f4814b756f9493f9e353a8f98e7da\n\nChange-Id: I041aaa7dce53f920b014c1cb586dc6bb3c10b489\n" }, { "commit": "d0ddf7d45044b2c8a9395a8f0df0c6c6adec9399", "tree": "8b991d5ac23c25a11f016af284098de25dbcb5b4", "parents": [ "ea1c9e7cffb4b822d510d46dcaef76fab5999f20" ], "author": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Thu May 31 02:04:11 2018 -0400" }, "committer": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Sat Jun 09 14:44:08 2018 +0000" }, "message": "Add RBAC tests for identity v3 application credentials API\n\nThis patchset adds RBAC tests for identity v3 application\ncredentials API [0], which enforces the following policies: [1].\n\n[0] https://developer.openstack.org/api-ref/identity/v3/index.html#application-credentials\n[1] https://github.com/openstack/keystone/blob/master/keystone/common/policies/application_credential.py\n\nChange-Id: I986e245a0bf7c4c37089a4a2dae7d9067c74162b\n" }, { "commit": "8251477dde1d12d9b325ed3f9ab8caa047f25b93", "tree": "23f63cb781bc8894719341232fe48009faad09b1", "parents": [ "ea1c9e7cffb4b822d510d46dcaef76fab5999f20" ], "author": { "name": "Rick Bartra", "email": "rb560u@att.com", "time": "Thu May 31 15:19:04 2018 -0400" }, "committer": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Sat Jun 09 14:43:49 2018 +0000" }, "message": "Refactor `test_confirm_resize_server` to pass for all roles\n\n`test_confirm_resize_server` fails for roles that do not have\npermission to perform the confirm resize API. It fails since\nthe cleanup tries to resize a server that is already in the resized\nstate [1]. This PS conditionally adds resize to addCleanup based on the\nstate of the server.\n\n[1] `Details: {u\u0027message\u0027: u\"Cannot \u0027resize\u0027 instance\ncb7c5ddc-959e-4238-9f79-a1963e60228f while it is in vm_state resized\",\nu\u0027code\u0027: 409}`\n\nChange-Id: I1abb13742ed931f15d6ae18ccdc6d790b41a02bf\n" }, { "commit": "ea1c9e7cffb4b822d510d46dcaef76fab5999f20", "tree": "3696d739de5d724192dded484984cd408dae11de", "parents": [ "3c9da68af76c27430224715a3e1e7ce6aba90a8e", "66d54a926674c1c60a2c0f3586b3dd6e938e2b89" ], "author": { "name": "Zuul", "email": "zuul@review.openstack.org", "time": "Sat Jun 09 14:27:06 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "review@openstack.org", "time": "Sat Jun 09 14:27:06 2018 +0000" }, "message": "Merge \"Docs: Add requirements authority module to documentation\"" }, { "commit": "68f4c2f382ecf3f96d0759eef9d6cf0f7d257fe6", "tree": "6f205f7c5636cb8c8f9b9fc396dc48b6f6fb5e33", "parents": [ "41e4d002ecebdccfb9a6a13fcbfce88cbb4a31d3" ], "author": { "name": "huang.zhiping", "email": "huang.zhiping@99cloud.net", "time": "Sat Jun 09 20:26:49 2018 +0800" }, "committer": { "name": "huang.zhiping", "email": "huang.zhiping@99cloud.net", "time": "Sat Jun 09 20:27:40 2018 +0800" }, "message": "fix tox python3 overrides\n\nWe want to default to running all tox environments under python 3, so\nset the basepython value in each environment.\n\nWe do not want to specify a minor version number, because we do not\nwant to have to update the file every time we upgrade python.\n\nWe do not want to set the override once in testenv, because that\nbreaks the more specific versions used in default environments like\npy35 and py36.\n\nChange-Id: I6e73db41d64343dc76ef33bed4638c93b131bd88\n" }, { "commit": "3c9da68af76c27430224715a3e1e7ce6aba90a8e", "tree": "3bd9d0c49de9506bcd67067f031764376b0a2ba7", "parents": [ "9c40f77e19df40ae0c4b2109aa3762705a1b6683", "778b7805483626ebcbd93614b21aa4b99951fc04" ], "author": { "name": "Zuul", "email": "zuul@review.openstack.org", "time": "Sat Jun 09 05:10:36 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "review@openstack.org", "time": "Sat Jun 09 05:10:36 2018 +0000" }, "message": "Merge \"Add docstrings for requirements_authority module\"" }, { "commit": "66d54a926674c1c60a2c0f3586b3dd6e938e2b89", "tree": "10f62e507af036b8bdc8b968b981516d59bb133a", "parents": [ "9f42552b1aeb615cc7c705156fe0392184e581e8" ], "author": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Thu May 31 20:08:35 2018 +0100" }, "committer": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Sat Jun 09 00:30:30 2018 -0400" }, "message": "Docs: Add requirements authority module to documentation\n\nThis patchset includes documentation on the rbac_authority\nand the requirements_authority modules. In addition,\nthe documentation for the policy_authority module is\nexpanded. All 3 modules are explained together, explaining\nthat the rbac_authority module contains an abstract\nclass consumed by the classes in the other two modules.\n\nThe use cases for each validation approach is also included\nin the documentation.\n\nFinally, some documentation syntax issues are corrected.\n\nChange-Id: I33bbe2da67683faafd0749b687b99237ac815009\n" }, { "commit": "9c40f77e19df40ae0c4b2109aa3762705a1b6683", "tree": "21fa431b8de191d3f7fefe66ba349cf96de1f636", "parents": [ "9f42552b1aeb615cc7c705156fe0392184e581e8", "ac3e85d89f393da48496d37983d72e8a3fe6f0cc" ], "author": { "name": "Zuul", "email": "zuul@review.openstack.org", "time": "Sat Jun 09 04:25:02 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "review@openstack.org", "time": "Sat Jun 09 04:25:02 2018 +0000" }, "message": "Merge \"Move VirtualInterfacesRbacTest into separate module\"" }, { "commit": "9f42552b1aeb615cc7c705156fe0392184e581e8", "tree": "03e0c15739d4a98b47f7abbf19756e78017eb64b", "parents": [ "55eeb3e6429a4d5ad7dc37f7449fa7371540a815", "31e308ecab6c6933b4e2830b53167416e867966c" ], "author": { "name": "Zuul", "email": "zuul@review.openstack.org", "time": "Sat Jun 09 04:20:14 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "review@openstack.org", "time": "Sat Jun 09 04:20:14 2018 +0000" }, "message": "Merge \"Move RbacAuthority class into separate module\"" }, { "commit": "55eeb3e6429a4d5ad7dc37f7449fa7371540a815", "tree": "88f95132a29d692a2d54cb0c5d86a003874a2eea", "parents": [ "39c66416c462aba747f61afe8b5f9b775aae1e5e", "44d77848b5e1be53072281f77e6324cde15c3caa" ], "author": { "name": "Zuul", "email": "zuul@review.openstack.org", "time": "Sat Jun 09 03:46:26 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "review@openstack.org", "time": "Sat Jun 09 03:46:26 2018 +0000" }, "message": "Merge \"Update rbac_rule_validation for multi-policy support\"" }, { "commit": "39c66416c462aba747f61afe8b5f9b775aae1e5e", "tree": "7119255f560f1e0beee197b4f5124eb32ee8d629", "parents": [ "c73b492669ce2d6f11efc71661603bb0e576ceba", "97117b0cc1b6c0f2462a66951bfbee2ee6310041" ], "author": { "name": "Zuul", "email": "zuul@review.openstack.org", "time": "Sat Jun 09 03:40:02 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "review@openstack.org", "time": "Sat Jun 09 03:40:02 2018 +0000" }, "message": "Merge \"Fix typos for custom requirements config options descriptions\"" }, { "commit": "bc865dfecc319cbca8aeb31687ee0ba6dcea9d12", "tree": "efde286d198ca6e5fb2342adada85c6c9b8c7fd9", "parents": [ "9ae705db1631172db370c2be48aeebc4f9a0b085" ], "author": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Fri Jun 08 19:47:28 2018 -0400" }, "committer": { "name": "Felipe Monteiro", "email": "felipe.monteiro@att.com", "time": "Fri Jun 08 21:36:27 2018 -0400" }, "message": "Fix test_volume_upload BadRequest error\n\nThis patchset fixes test_volume_upload raising a BadRequest\ndue to visibility being passed to upload_volume which is\nunexpected [0]. That is because this param is only\navailable for microversion 3.1+ [1].\n\nThe issue is fixed by moving the test into the 3.1 microversion\ntest class: VolumesActionsV310RbacTest.\n\n[0] http://logs.openstack.org/83/571383/2/check/patrole-multinode-admin/ccceab5/job-output.txt.gz#_2018-05-31_15_38_44_698855\n[1] https://github.com/openstack/cinder/blob/7d95fb79c42bdeeb82900b41acc70dead8a2ee6f/cinder/api/schemas/volume_actions.py#L150\n\nChange-Id: Ic92504cb0442e47eea3c0e5345498ecb49ffe44f\n" }, { "commit": "c73b492669ce2d6f11efc71661603bb0e576ceba", "tree": "75191271ec7fad06ded3ea15cc5f5a4fdffb4a34", "parents": [ "41e4d002ecebdccfb9a6a13fcbfce88cbb4a31d3", "94fc2ca66d1bce8eacab1f416fba3fd5ed2e50a5" ], "author": { "name": "Zuul", "email": "zuul@review.openstack.org", "time": "Fri Jun 08 19:36:04 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "review@openstack.org", "time": "Fri Jun 08 19:36:04 2018 +0000" }, "message": "Merge \"[trivial] Update short-hand doc reference with real link\"" }, { "commit": "41e4d002ecebdccfb9a6a13fcbfce88cbb4a31d3", "tree": "a752b5fc5ebb4ab4cdf759032c7886a64425560c", "parents": [ "9ae705db1631172db370c2be48aeebc4f9a0b085", "2880cad79c8c2bd090fc5387477f1c5f210d7500" ], "author": { "name": "Zuul", "email": "zuul@review.openstack.org", "time": "Wed Jun 06 16:28:22 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "review@openstack.org", "time": "Wed Jun 06 16:28:22 2018 +0000" }, "message": "Merge \"Remove RBAC test_volume_list_image_metadata test case\"" }, { "commit": "2880cad79c8c2bd090fc5387477f1c5f210d7500", "tree": "38bb06ef19553b2a77d2dd85ec38331a2fee101e", "parents": [ "f1ba5ae05ce075b30b2824f2631caa6ea2357f9b" ], "author": { "name": "Chi Lo", "email": "cl566n@att.com", "time": "Fri May 25 05:09:41 2018 -0500" }, "committer": { "name": "Chi Lo", "email": "cl566n@att.com", "time": "Mon Jun 04 16:21:10 2018 +0000" }, "message": "Remove RBAC test_volume_list_image_metadata test case\n\nRemove the test_volume_list_image_metadata test case as it duplicates with\ntest_list_volumes_details_image_metadata in test_volume_metadata_rbac.py. Also,\ntest_volume_list_image_metadata does not test list image metadata correctly\nsince the volume_image_metadata attribute has never been created in the first\nplace.\n\nChange-Id: I07c3971f5bdb4255105612538e58c79d69daa691\n" } ], "next": "ac3e85d89f393da48496d37983d72e8a3fe6f0cc" }