)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"9e81e6451c675b9579bf7882e8bf35754fd103ff","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"d1c7152f_7c7e0556","updated":"2025-11-07 19:21:04.000000000","message":"Thanks for working on this Riane, please see comments inline","commit_id":"e0eca2e2c83de55fc60577972ddb6d5ef812f648"},{"author":{"_account_id":38038,"name":"Riane TOrres","display_name":"Riane Torres","email":"riane.torres@gmail.com","username":"realrtorres"},"change_message_id":"5fe8302f5359af7408f63cf39fbcb1daf854f100","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"0e0ae440_8ec1bbbb","updated":"2025-11-14 00:09:23.000000000","message":"Resolved comments","commit_id":"3e975c5ff82d0e1090d9b81503703c811b87af36"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"2eeabcbd557bce11b8fa5429ba5eb7cb4ed32622","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"205dc25d_7e8047d8","updated":"2025-11-14 22:25:23.000000000","message":"Fixed up a minor issue in one of the tests","commit_id":"449e1a8ebf05145002c7f3be28286dd02af279ed"},{"author":{"_account_id":38038,"name":"Riane TOrres","display_name":"Riane Torres","email":"riane.torres@gmail.com","username":"realrtorres"},"change_message_id":"9ec22202a42bff4373e7af89342342dcbad7fa8e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"6c9934fe_84c2f671","updated":"2025-11-20 21:03:17.000000000","message":"Noticed my build failure failed last time on Friday 11/14, for some reason PyCharm either reverted my changes or never made them in order to change the pep8 styling. I had to run the build again.","commit_id":"e5a56f2827bf755dd96cf5e628204ed0517e9713"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"9819b42123f2723f4fded72776d947162c159ec6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"9a0b5b9a_24e7bf48","updated":"2025-11-20 22:57:21.000000000","message":"Thanks Riane, LGTM!","commit_id":"e5a56f2827bf755dd96cf5e628204ed0517e9713"}],"manilaclient/tests/functional/osc/test_share_instances.py":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"9e81e6451c675b9579bf7882e8bf35754fd103ff","unresolved":true,"context_lines":[{"line_number":23,"context_line":"        self.share \u003d self.create_share()"},{"line_number":24,"context_line":"        # Get share instances for testing"},{"line_number":25,"context_line":"        self.share_instances \u003d self.listing_result(\u0027share instance\u0027, \u0027list\u0027)"},{"line_number":26,"context_line":"        if self.share_instances:"},{"line_number":27,"context_line":"            self.share_instance \u003d self.share_instances[0]"},{"line_number":28,"context_line":"            self.non_existent_id \u003d \u0027non-existent-id-123\u0027"},{"line_number":29,"context_line":"            self.invalid_id \u003d \u0027invalid-format\u0027"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"    def test_share_instance_list(self):"},{"line_number":32,"context_line":"        # Test basic listing"}],"source_content_type":"text/x-python","patch_set":2,"id":"063c9fd0_9b288604","line":29,"range":{"start_line":26,"start_character":0,"end_line":29,"end_character":46},"updated":"2025-11-07 19:21:04.000000000","message":"```suggestion\n        self.share_instance \u003d (\n            self.share_instances[0] if self.share_instances else None\n        )\n        self.non_existent_id \u003d \u002776a7dd8e-7564-41d4-a184-06dc20e13d7a\u0027\n        self.invalid_id \u003d \u0027invalid-format\u0027\n```","commit_id":"e0eca2e2c83de55fc60577972ddb6d5ef812f648"},{"author":{"_account_id":38038,"name":"Riane TOrres","display_name":"Riane Torres","email":"riane.torres@gmail.com","username":"realrtorres"},"change_message_id":"5fe8302f5359af7408f63cf39fbcb1daf854f100","unresolved":false,"context_lines":[{"line_number":23,"context_line":"        self.share \u003d self.create_share()"},{"line_number":24,"context_line":"        # Get share instances for testing"},{"line_number":25,"context_line":"        self.share_instances \u003d self.listing_result(\u0027share instance\u0027, \u0027list\u0027)"},{"line_number":26,"context_line":"        if self.share_instances:"},{"line_number":27,"context_line":"            self.share_instance \u003d self.share_instances[0]"},{"line_number":28,"context_line":"            self.non_existent_id \u003d \u0027non-existent-id-123\u0027"},{"line_number":29,"context_line":"            self.invalid_id \u003d \u0027invalid-format\u0027"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"    def test_share_instance_list(self):"},{"line_number":32,"context_line":"        # Test basic listing"}],"source_content_type":"text/x-python","patch_set":2,"id":"c1b488be_751d5983","line":29,"range":{"start_line":26,"start_character":0,"end_line":29,"end_character":46},"in_reply_to":"063c9fd0_9b288604","updated":"2025-11-14 00:09:23.000000000","message":"Done","commit_id":"e0eca2e2c83de55fc60577972ddb6d5ef812f648"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"9e81e6451c675b9579bf7882e8bf35754fd103ff","unresolved":true,"context_lines":[{"line_number":58,"context_line":"            exceptions.CommandFailed,"},{"line_number":59,"context_line":"            self.openstack,"},{"line_number":60,"context_line":"            f\u0027share instance set {self.non_existent_id} \u0027"},{"line_number":61,"context_line":"            \u0027unexistent_share_instance_id --status available\u0027"},{"line_number":62,"context_line":"        )"},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"    def test_share_instance_set_invalid_status(self):"}],"source_content_type":"text/x-python","patch_set":2,"id":"9c316218_07a13168","line":61,"range":{"start_line":61,"start_character":13,"end_line":61,"end_character":41},"updated":"2025-11-07 19:21:04.000000000","message":"This parameter isn\u0027t expected.. you already have the ID added above","commit_id":"e0eca2e2c83de55fc60577972ddb6d5ef812f648"},{"author":{"_account_id":38038,"name":"Riane TOrres","display_name":"Riane Torres","email":"riane.torres@gmail.com","username":"realrtorres"},"change_message_id":"5fe8302f5359af7408f63cf39fbcb1daf854f100","unresolved":false,"context_lines":[{"line_number":58,"context_line":"            exceptions.CommandFailed,"},{"line_number":59,"context_line":"            self.openstack,"},{"line_number":60,"context_line":"            f\u0027share instance set {self.non_existent_id} \u0027"},{"line_number":61,"context_line":"            \u0027unexistent_share_instance_id --status available\u0027"},{"line_number":62,"context_line":"        )"},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"    def test_share_instance_set_invalid_status(self):"}],"source_content_type":"text/x-python","patch_set":2,"id":"4d280497_85193cd3","line":61,"range":{"start_line":61,"start_character":13,"end_line":61,"end_character":41},"in_reply_to":"9c316218_07a13168","updated":"2025-11-14 00:09:23.000000000","message":"Done","commit_id":"e0eca2e2c83de55fc60577972ddb6d5ef812f648"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"9e81e6451c675b9579bf7882e8bf35754fd103ff","unresolved":true,"context_lines":[{"line_number":66,"context_line":"            exceptions.CommandFailed,"},{"line_number":67,"context_line":"            self.openstack,"},{"line_number":68,"context_line":"            f\u0027share instance set {self.invalid_id} \u0027"},{"line_number":69,"context_line":"            \u0027invalid_share_instance_id --status available\u0027"},{"line_number":70,"context_line":"        )"},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"    def test_share_instance_show_non_existent(self):"}],"source_content_type":"text/x-python","patch_set":2,"id":"c0bffcc3_fb00a938","line":69,"range":{"start_line":69,"start_character":13,"end_line":69,"end_character":38},"updated":"2025-11-07 19:21:04.000000000","message":"same comment as above","commit_id":"e0eca2e2c83de55fc60577972ddb6d5ef812f648"},{"author":{"_account_id":38038,"name":"Riane TOrres","display_name":"Riane Torres","email":"riane.torres@gmail.com","username":"realrtorres"},"change_message_id":"5fe8302f5359af7408f63cf39fbcb1daf854f100","unresolved":false,"context_lines":[{"line_number":66,"context_line":"            exceptions.CommandFailed,"},{"line_number":67,"context_line":"            self.openstack,"},{"line_number":68,"context_line":"            f\u0027share instance set {self.invalid_id} \u0027"},{"line_number":69,"context_line":"            \u0027invalid_share_instance_id --status available\u0027"},{"line_number":70,"context_line":"        )"},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"    def test_share_instance_show_non_existent(self):"}],"source_content_type":"text/x-python","patch_set":2,"id":"9fbb57b9_ea38b814","line":69,"range":{"start_line":69,"start_character":13,"end_line":69,"end_character":38},"in_reply_to":"c0bffcc3_fb00a938","updated":"2025-11-14 00:09:23.000000000","message":"Done","commit_id":"e0eca2e2c83de55fc60577972ddb6d5ef812f648"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"9e81e6451c675b9579bf7882e8bf35754fd103ff","unresolved":true,"context_lines":[{"line_number":73,"context_line":"        self.assertRaises("},{"line_number":74,"context_line":"            exceptions.CommandFailed,"},{"line_number":75,"context_line":"            self.openstack,"},{"line_number":76,"context_line":"            f\u0027share instance show {self.share[\"id\"]} {self.non_existent_id}\u0027)"},{"line_number":77,"context_line":""},{"line_number":78,"context_line":"    def test_share_instance_show_invalid_id_format(self):"},{"line_number":79,"context_line":"        self.assertRaises("}],"source_content_type":"text/x-python","patch_set":2,"id":"dbce450f_0e39fe04","line":76,"range":{"start_line":76,"start_character":34,"end_line":76,"end_character":53},"updated":"2025-11-07 19:21:04.000000000","message":"this parameter isn\u0027t expected","commit_id":"e0eca2e2c83de55fc60577972ddb6d5ef812f648"},{"author":{"_account_id":38038,"name":"Riane TOrres","display_name":"Riane Torres","email":"riane.torres@gmail.com","username":"realrtorres"},"change_message_id":"5fe8302f5359af7408f63cf39fbcb1daf854f100","unresolved":false,"context_lines":[{"line_number":73,"context_line":"        self.assertRaises("},{"line_number":74,"context_line":"            exceptions.CommandFailed,"},{"line_number":75,"context_line":"            self.openstack,"},{"line_number":76,"context_line":"            f\u0027share instance show {self.share[\"id\"]} {self.non_existent_id}\u0027)"},{"line_number":77,"context_line":""},{"line_number":78,"context_line":"    def test_share_instance_show_invalid_id_format(self):"},{"line_number":79,"context_line":"        self.assertRaises("}],"source_content_type":"text/x-python","patch_set":2,"id":"afc67ead_f035edda","line":76,"range":{"start_line":76,"start_character":34,"end_line":76,"end_character":53},"in_reply_to":"dbce450f_0e39fe04","updated":"2025-11-14 00:09:23.000000000","message":"Done","commit_id":"e0eca2e2c83de55fc60577972ddb6d5ef812f648"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"9e81e6451c675b9579bf7882e8bf35754fd103ff","unresolved":true,"context_lines":[{"line_number":79,"context_line":"        self.assertRaises("},{"line_number":80,"context_line":"            exceptions.CommandFailed,"},{"line_number":81,"context_line":"            self.openstack,"},{"line_number":82,"context_line":"            f\u0027share instance show {self.share[\"id\"]} {self.invalid_id}\u0027)"},{"line_number":83,"context_line":""},{"line_number":84,"context_line":"    def test_share_instance_list_invalid_filter_share_id_format(self):"},{"line_number":85,"context_line":"        self.assertRaises("}],"source_content_type":"text/x-python","patch_set":2,"id":"b545a64b_0f1bb876","line":82,"range":{"start_line":82,"start_character":34,"end_line":82,"end_character":53},"updated":"2025-11-07 19:21:04.000000000","message":"same","commit_id":"e0eca2e2c83de55fc60577972ddb6d5ef812f648"},{"author":{"_account_id":38038,"name":"Riane TOrres","display_name":"Riane Torres","email":"riane.torres@gmail.com","username":"realrtorres"},"change_message_id":"5fe8302f5359af7408f63cf39fbcb1daf854f100","unresolved":false,"context_lines":[{"line_number":79,"context_line":"        self.assertRaises("},{"line_number":80,"context_line":"            exceptions.CommandFailed,"},{"line_number":81,"context_line":"            self.openstack,"},{"line_number":82,"context_line":"            f\u0027share instance show {self.share[\"id\"]} {self.invalid_id}\u0027)"},{"line_number":83,"context_line":""},{"line_number":84,"context_line":"    def test_share_instance_list_invalid_filter_share_id_format(self):"},{"line_number":85,"context_line":"        self.assertRaises("}],"source_content_type":"text/x-python","patch_set":2,"id":"0b267357_e841529f","line":82,"range":{"start_line":82,"start_character":34,"end_line":82,"end_character":53},"in_reply_to":"b545a64b_0f1bb876","updated":"2025-11-14 00:09:23.000000000","message":"Done","commit_id":"e0eca2e2c83de55fc60577972ddb6d5ef812f648"}]}