)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"9c5f58219ab882f8d30bf16d1fa032d848921c7c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"bd294f72_7d0d83d7","updated":"2022-05-09 17:04:00.000000000","message":"LGTM thank you for this cleanup; a question inline.. ","commit_id":"77fc718b523bd2a2133543204c6396fda0e48164"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"eacd78df605fe7bfe4a6fef3fd5a788606822aba","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"58c5b0b2_ab20570e","updated":"2022-05-10 18:02:24.000000000","message":"LGTM, thanks Liron","commit_id":"ad511b6ac05c2d3026512adba516042effa3bd3c"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"3baf1e5bd043337b8ef0b35d22e4bf3ca8afc553","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"293a1e68_482246fe","updated":"2022-05-12 11:18:18.000000000","message":"Late +2 - This is looking good, Liron! Thank you","commit_id":"ad511b6ac05c2d3026512adba516042effa3bd3c"},{"author":{"_account_id":30025,"name":"Vida Haririan","display_name":"Vida Haririan","email":"vhariria@redhat.com","username":"vhari"},"change_message_id":"0f59186528ab0119c3306da000ddf77b4c327744","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"14024e35_dc3da147","updated":"2022-05-11 15:13:29.000000000","message":"Looks good, thanks Liron!","commit_id":"ad511b6ac05c2d3026512adba516042effa3bd3c"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"2217e8fb211ea63d6e2b1581dfa27b5e3c4acf9b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"de1b5301_bfdef6cf","updated":"2022-05-09 18:26:07.000000000","message":"Thanks for the comments Goutham !","commit_id":"ad511b6ac05c2d3026512adba516042effa3bd3c"}],"manila_tempest_tests/tests/api/test_access_rules_metadata.py":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"9c5f58219ab882f8d30bf16d1fa032d848921c7c","unresolved":true,"context_lines":[{"line_number":88,"context_line":"        data \u003d {\"key1\": \"v\" * 255, \"k\" * 255: \"value2\"}"},{"line_number":89,"context_line":"        # set metadata"},{"line_number":90,"context_line":"        access \u003d self.allow_access("},{"line_number":91,"context_line":"            share_id\u003dself.share[\"id\"], access_type\u003dself.access_type,"},{"line_number":92,"context_line":"            access_to\u003dself.access_to[self.access_type].pop(),"},{"line_number":93,"context_line":"            access_level\u003d\u0027rw\u0027, metadata\u003ddata)"},{"line_number":94,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"a0b0a0f5_58189cd8","line":91,"range":{"start_line":91,"start_character":12,"end_line":91,"end_character":21},"updated":"2022-05-09 17:04:00.000000000","message":"not a kwarg","commit_id":"77fc718b523bd2a2133543204c6396fda0e48164"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"2217e8fb211ea63d6e2b1581dfa27b5e3c4acf9b","unresolved":false,"context_lines":[{"line_number":88,"context_line":"        data \u003d {\"key1\": \"v\" * 255, \"k\" * 255: \"value2\"}"},{"line_number":89,"context_line":"        # set metadata"},{"line_number":90,"context_line":"        access \u003d self.allow_access("},{"line_number":91,"context_line":"            share_id\u003dself.share[\"id\"], access_type\u003dself.access_type,"},{"line_number":92,"context_line":"            access_to\u003dself.access_to[self.access_type].pop(),"},{"line_number":93,"context_line":"            access_level\u003d\u0027rw\u0027, metadata\u003ddata)"},{"line_number":94,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"23a18541_8e09e61c","line":91,"range":{"start_line":91,"start_character":12,"end_line":91,"end_character":21},"in_reply_to":"a0b0a0f5_58189cd8","updated":"2022-05-09 18:26:07.000000000","message":"Done","commit_id":"77fc718b523bd2a2133543204c6396fda0e48164"}],"manila_tempest_tests/tests/api/test_rules.py":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"9c5f58219ab882f8d30bf16d1fa032d848921c7c","unresolved":true,"context_lines":[{"line_number":136,"context_line":"        for key in (\u0027deleted\u0027, \u0027deleted_at\u0027, \u0027instance_mappings\u0027):"},{"line_number":137,"context_line":"            self.assertNotIn(key, rule.keys())"},{"line_number":138,"context_line":""},{"line_number":139,"context_line":"        # rules must start out in \u0027new\u0027 until 2.28 \u0026 \u0027queued_to_apply\u0027 after"},{"line_number":140,"context_line":"        if utils.is_microversion_le(version, \"2.27\"):"},{"line_number":141,"context_line":"            self.assertEqual(\"new\", rule[\u0027state\u0027])"},{"line_number":142,"context_line":"        else:"},{"line_number":143,"context_line":"            self.assertEqual(\"queued_to_apply\", rule[\u0027state\u0027])"},{"line_number":144,"context_line":""},{"line_number":145,"context_line":"        if utils.is_microversion_eq(version, \u00271.0\u0027):"},{"line_number":146,"context_line":"            waiters.wait_for_resource_status("}],"source_content_type":"text/x-python","patch_set":3,"id":"e47a82f9_83a94239","side":"PARENT","line":143,"range":{"start_line":139,"start_character":0,"end_line":143,"end_character":62},"updated":"2022-05-09 17:04:00.000000000","message":"We\u0027re losing this API validation.. do we want to add a test just for this?","commit_id":"c0e182601b83bf7aa708a12336a96c37d75da530"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"2217e8fb211ea63d6e2b1581dfa27b5e3c4acf9b","unresolved":true,"context_lines":[{"line_number":136,"context_line":"        for key in (\u0027deleted\u0027, \u0027deleted_at\u0027, \u0027instance_mappings\u0027):"},{"line_number":137,"context_line":"            self.assertNotIn(key, rule.keys())"},{"line_number":138,"context_line":""},{"line_number":139,"context_line":"        # rules must start out in \u0027new\u0027 until 2.28 \u0026 \u0027queued_to_apply\u0027 after"},{"line_number":140,"context_line":"        if utils.is_microversion_le(version, \"2.27\"):"},{"line_number":141,"context_line":"            self.assertEqual(\"new\", rule[\u0027state\u0027])"},{"line_number":142,"context_line":"        else:"},{"line_number":143,"context_line":"            self.assertEqual(\"queued_to_apply\", rule[\u0027state\u0027])"},{"line_number":144,"context_line":""},{"line_number":145,"context_line":"        if utils.is_microversion_eq(version, \u00271.0\u0027):"},{"line_number":146,"context_line":"            waiters.wait_for_resource_status("}],"source_content_type":"text/x-python","patch_set":3,"id":"76c7ea56_a0dbbc41","side":"PARENT","line":143,"range":{"start_line":139,"start_character":0,"end_line":143,"end_character":62},"in_reply_to":"e47a82f9_83a94239","updated":"2022-05-09 18:26:07.000000000","message":"Oh, my mistake, it shouldn\u0027t be deleted.","commit_id":"c0e182601b83bf7aa708a12336a96c37d75da530"}]}