)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":17896,"name":"Rick Bartra","email":"rickbartra@microsoft.com","username":"rb560u"},"change_message_id":"07006a94831a35931c9c0fb5b7fdd67976269097","unresolved":false,"context_lines":[{"line_number":10,"context_line":"  * \"volume:get_volume_metadata\""},{"line_number":11,"context_line":"  * \"volume_extension:volume_image_metadata\""},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"The new tests use override_role introduced in [1]. The rest"},{"line_number":14,"context_line":"of the tests will be converted over in a follow-up PS."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"[0] https://github.com/openstack/cinder/blob/0ac4af182fdd55d4a97ce283070f7b8e6480540f/cinder/policies/volume_metadata.py"},{"line_number":17,"context_line":"[1] I670fba358bf321eae0d22d18cea6d2f530f00716"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"3fa0c359_7900f934","line":14,"range":{"start_line":13,"start_character":51,"end_line":14,"end_character":54},"updated":"2018-02-23 14:13:13.000000000","message":"nit: no longer true","commit_id":"db79c9a462ef68a5c06ea8b439eb37870a613e1f"}],"patrole_tempest_plugin/tests/api/volume/test_volume_metadata_rbac.py":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"952c199bb683ea90e8e660662039af818bfeb5eb","unresolved":false,"context_lines":[{"line_number":110,"context_line":"            resp_body \u003d self.volumes_client.list_volumes(detail\u003dTrue)["},{"line_number":111,"context_line":"                \u0027volumes\u0027]"},{"line_number":112,"context_line":"        expected_attr \u003d \u0027volume_image_metadata\u0027"},{"line_number":113,"context_line":"        if expected_attr not in resp_body[0]:"},{"line_number":114,"context_line":"            raise rbac_exceptions.RbacMalformedResponse("},{"line_number":115,"context_line":"                attribute\u003dexpected_attr)"},{"line_number":116,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"df7087c5_f8123f58","line":113,"range":{"start_line":113,"start_character":8,"end_line":113,"end_character":45},"updated":"2018-03-09 23:31:20.000000000","message":"yes, just checking attribute presence is ok as it can be empty depends on what cinder find in DB.","commit_id":"db79c9a462ef68a5c06ea8b439eb37870a613e1f"}]}