)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":30025,"name":"Vida Haririan","display_name":"Vida Haririan","email":"vhariria@redhat.com","username":"vhari"},"change_message_id":"d44947fa35195b3002c394408507a307cf288ecf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"3d35345d_dd95a555","updated":"2023-10-18 14:16:28.000000000","message":"Change looks good, thanks Liron","commit_id":"0a48eeacb380cbf5ccf355f999bf8545c35ffc20"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"ec1a67dae44f971742af48f12b497545814e070c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"eeec2dfb_739025b8","updated":"2023-10-19 04:15:13.000000000","message":"I think this won\u0027t merge until we have manila backports pending (see \"Depends On\").. we have a couple of choices here:\n\n1) Merge the backports that are open... we intend to backport fixes all teh way to stable/wallaby... so we can merge what\u0027s open, merge this change and then continue backports ... or\n\n2) Drop all the \"Depends-On\" change IDs from the commit message and re-workflow this.","commit_id":"0a48eeacb380cbf5ccf355f999bf8545c35ffc20"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"1dc3d307fdd4e95314f6792bce5cddeac57cb4b2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"e3131049_4ac57f35","updated":"2023-10-18 14:15:05.000000000","message":"LGTM, thanks","commit_id":"0a48eeacb380cbf5ccf355f999bf8545c35ffc20"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"b3d6ca3bf148c2faf81c6a4361d3a9a73b372b9b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"d32bc2c2_76331d9d","updated":"2023-10-17 18:28:42.000000000","message":"LGTM, thanks for fixing this bug","commit_id":"0a48eeacb380cbf5ccf355f999bf8545c35ffc20"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"df103d1689d5d0682a2762b52b69425cf43a8dff","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"25c868e9_716af5a0","updated":"2023-10-17 14:41:59.000000000","message":"The changes have merged in branches of openstack/manila that support running these SRBAC tests; so lets get this in.. Thanks Liron!\n\nhttps://review.opendev.org/q/topic:bug%252F2004230 \nhttps://review.opendev.org/q/topic:bug%252F1955627","commit_id":"0a48eeacb380cbf5ccf355f999bf8545c35ffc20"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"490ee5490a752bfb1dc3fd368ecd0c87c9b688b1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"246f5438_411052a7","updated":"2023-09-22 02:25:07.000000000","message":"recheck\n\nparent patch updated","commit_id":"0a48eeacb380cbf5ccf355f999bf8545c35ffc20"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"eea8328c0a4d5a261c591de49fb9aecc1d736331","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"e695efef_271fa0d1","updated":"2023-10-20 17:47:57.000000000","message":"I popped off the Depends-On; lets get this gate fix in.","commit_id":"2c06e81bfdbe72297b9fbc0c8d571c0ea51ab443"}],"manila_tempest_tests/tests/rbac/test_snapshots.py":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"d37e23d552494f882314336ecd2a52ed4dad54b9","unresolved":true,"context_lines":[{"line_number":317,"context_line":"        alt_snap \u003d self.create_snapshot("},{"line_number":318,"context_line":"            self.alt_project_share_v2_client, self.alt_share[\u0027id\u0027])"},{"line_number":319,"context_line":"        self.do_request("},{"line_number":320,"context_line":"            \u0027snapshot_reset_state\u0027, expected_status\u003dlib_exc.NotFound,"},{"line_number":321,"context_line":"            snapshot_id\u003dalt_snap[\u0027id\u0027], status\u003d\u0027error\u0027)"},{"line_number":322,"context_line":""},{"line_number":323,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"675b77d5_1bc6a4de","line":320,"range":{"start_line":320,"start_character":52,"end_line":320,"end_character":68},"updated":"2023-02-02 20:57:58.000000000","message":"hmmm, this should still be forbidden - the API isn\u0027t available to non-admins by default:\n\nOur RBAC enforcement happens in stages:\n1) Do you have access to the API\n2) Do you have access to the resource\n3) Do you have access to perform the API on the resource\n\nIf (1) fails, you get a 403 always, \nif (2) fails, you get a 404; \nif (3) fails, you get a 403. \n\nMore often than not, (2) will prevent \"existence\" detection of resources that don\u0027t belong to the user; except in case of \"public\" resources (e.g.: shares can be \"public\")","commit_id":"6ca2b0c6941f6bc56d77403b4b28311caae1be35"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"b15ec066604d34e2b5ef261381501f801937ee56","unresolved":false,"context_lines":[{"line_number":317,"context_line":"        alt_snap \u003d self.create_snapshot("},{"line_number":318,"context_line":"            self.alt_project_share_v2_client, self.alt_share[\u0027id\u0027])"},{"line_number":319,"context_line":"        self.do_request("},{"line_number":320,"context_line":"            \u0027snapshot_reset_state\u0027, expected_status\u003dlib_exc.NotFound,"},{"line_number":321,"context_line":"            snapshot_id\u003dalt_snap[\u0027id\u0027], status\u003d\u0027error\u0027)"},{"line_number":322,"context_line":""},{"line_number":323,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"1b7dda69_bea5e429","line":320,"range":{"start_line":320,"start_character":52,"end_line":320,"end_character":68},"in_reply_to":"5c86c5d2_d6685e6e","updated":"2023-09-25 23:32:13.000000000","message":"Done","commit_id":"6ca2b0c6941f6bc56d77403b4b28311caae1be35"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"f92bc7e370604d6b0f31feb09f2b2deab7a72432","unresolved":true,"context_lines":[{"line_number":317,"context_line":"        alt_snap \u003d self.create_snapshot("},{"line_number":318,"context_line":"            self.alt_project_share_v2_client, self.alt_share[\u0027id\u0027])"},{"line_number":319,"context_line":"        self.do_request("},{"line_number":320,"context_line":"            \u0027snapshot_reset_state\u0027, expected_status\u003dlib_exc.NotFound,"},{"line_number":321,"context_line":"            snapshot_id\u003dalt_snap[\u0027id\u0027], status\u003d\u0027error\u0027)"},{"line_number":322,"context_line":""},{"line_number":323,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"5c86c5d2_d6685e6e","line":320,"range":{"start_line":320,"start_character":52,"end_line":320,"end_character":68},"in_reply_to":"675b77d5_1bc6a4de","updated":"2023-02-02 21:11:05.000000000","message":"Thank you for the clarification!\nI wasn\u0027t sure about that.\nIt makes sense.\n\nI reported a bug on it on both upstream and downstream.\nI\u0027ll close them.","commit_id":"6ca2b0c6941f6bc56d77403b4b28311caae1be35"}]}