)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"47116a8f02c7e84db272e6e53e3dd86845145aae","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"fff378d6_05baf6da","updated":"2023-01-25 14:20:19.000000000","message":"LGTM, thanks Vida!\nSome other changes have merged - Could you please rebase the change?","commit_id":"6a0d51d703df99fdcefd0681262a3c115c9a876b"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"e9769dd6326593d6ad05f894f880ee31aad9726b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"94134507_9b769edf","updated":"2023-01-26 23:48:35.000000000","message":"Thanks Vida, couple of comments inline","commit_id":"35d6ab42e2f49448cbd52e88e9830e061f72d4fb"},{"author":{"_account_id":30025,"name":"Vida Haririan","display_name":"Vida Haririan","email":"vhariria@redhat.com","username":"vhari"},"change_message_id":"e0520c92b4b91a5ad4d790a9b35eb028a2635f03","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"67fc9d97_f2da71a8","updated":"2023-01-26 15:52:21.000000000","message":"recheck, unrelated failure","commit_id":"35d6ab42e2f49448cbd52e88e9830e061f72d4fb"}],"manila_tempest_tests/tests/rbac/test_security_services.py":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"e9769dd6326593d6ad05f894f880ee31aad9726b","unresolved":true,"context_lines":[{"line_number":210,"context_line":"    @tc.attr(base.TAG_POSITIVE, base.TAG_API)"},{"line_number":211,"context_line":"    @decorators.idempotent_id(\u002721c170da-a7e1-4347-888b-c234b3c9ec41\u0027)"},{"line_number":212,"context_line":"    def test_get_security_service(self):"},{"line_number":213,"context_line":"        super(TestProjectReaderTests, self).test_get_security_service()"},{"line_number":214,"context_line":""},{"line_number":215,"context_line":"    @tc.attr(base.TAG_POSITIVE, base.TAG_API)"},{"line_number":216,"context_line":"    @decorators.idempotent_id(\u0027f82c3cf1-1dd4-4b5c-ac5d-670c7c3f5664\u0027)"}],"source_content_type":"text/x-python","patch_set":8,"id":"a85ea79e_5e9c6ca0","line":213,"range":{"start_line":213,"start_character":7,"end_line":213,"end_character":71},"updated":"2023-01-26 23:48:35.000000000","message":"Same comment as below. \n\nA project reader should only be allowed to get a security service that belongs in their project","commit_id":"35d6ab42e2f49448cbd52e88e9830e061f72d4fb"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"e9769dd6326593d6ad05f894f880ee31aad9726b","unresolved":true,"context_lines":[{"line_number":211,"context_line":"    @decorators.idempotent_id(\u002721c170da-a7e1-4347-888b-c234b3c9ec41\u0027)"},{"line_number":212,"context_line":"    def test_get_security_service(self):"},{"line_number":213,"context_line":"        super(TestProjectReaderTests, self).test_get_security_service()"},{"line_number":214,"context_line":""},{"line_number":215,"context_line":"    @tc.attr(base.TAG_POSITIVE, base.TAG_API)"},{"line_number":216,"context_line":"    @decorators.idempotent_id(\u0027f82c3cf1-1dd4-4b5c-ac5d-670c7c3f5664\u0027)"},{"line_number":217,"context_line":"    def test_list_security_services(self):"},{"line_number":218,"context_line":"        super(TestProjectReaderTests, self).test_list_security_services()"},{"line_number":219,"context_line":""},{"line_number":220,"context_line":"    @tc.attr(base.TAG_POSITIVE, base.TAG_API)"},{"line_number":221,"context_line":"    @decorators.idempotent_id(\u0027d4c22d52-760e-40d4-bfe8-003160d247d0\u0027)"}],"source_content_type":"text/x-python","patch_set":8,"id":"33c93391_6f9b58ac","line":218,"range":{"start_line":214,"start_character":0,"end_line":218,"end_character":73},"updated":"2023-01-26 23:48:35.000000000","message":"Wouldn\u0027t this be a bug if a project reader is allowed to see all the security services with \"all_tenants\"? \n\nA project scoped reader should only be able to list security services belonging to the project","commit_id":"35d6ab42e2f49448cbd52e88e9830e061f72d4fb"}]}