)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"5a3f05dcd746c90aa1362c4bc3a3b0f71757aa0e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"bec27ff2_8dbb96f6","updated":"2024-04-26 14:40:20.000000000","message":"recheck\nLogs are gone, resuming the work","commit_id":"1c3f8ffd37a1d53e23a4332def71163ab36c851a"},{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"b72e3758a0ae6fc8300ca18ffbbf2f33f6a8da2c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"7a015b89_c97f3f81","updated":"2026-08-06 13:22:09.000000000","message":"zuul failing. Will revisit this patch once the zuul has passed.","commit_id":"a2c3c3951a1453f1ebd9df08b6f36cb512a716c3"}],"manila/data/helper.py":[{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"2d4484c529ecc6152ad3052b8a600a69d99af357","unresolved":true,"context_lines":[{"line_number":251,"context_line":""},{"line_number":252,"context_line":"    def mount_share_instance_or_backup(self, mount_info, mount_path):"},{"line_number":253,"context_line":"        mount_point \u003d mount_info.get(\u0027mount_point\u0027)"},{"line_number":254,"context_line":"        mount_template \u003d mount_info.get(\u0027mount\u0027)"},{"line_number":255,"context_line":"        share_instance_id \u003d mount_info.get(\u0027share_instance_id\u0027)"},{"line_number":256,"context_line":"        backup \u003d mount_info.get(\u0027backup\u0027)"},{"line_number":257,"context_line":"        restore \u003d mount_info.get(\u0027restore\u0027)"}],"source_content_type":"text/x-python","patch_set":6,"id":"538c5862_f9f78627","line":254,"range":{"start_line":254,"start_character":0,"end_line":254,"end_character":48},"updated":"2026-06-12 20:12:58.000000000","message":"same here, assuming that we will always get \u0027mount\u0027 as the value for mount_info.get(\u0027mount\u0027)","commit_id":"e584906f7b754ec41e6111b4aa9d3b3776523b84"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"7074b975b15dbdcd86eee5028c749f3a57d04b2b","unresolved":false,"context_lines":[{"line_number":251,"context_line":""},{"line_number":252,"context_line":"    def mount_share_instance_or_backup(self, mount_info, mount_path):"},{"line_number":253,"context_line":"        mount_point \u003d mount_info.get(\u0027mount_point\u0027)"},{"line_number":254,"context_line":"        mount_template \u003d mount_info.get(\u0027mount\u0027)"},{"line_number":255,"context_line":"        share_instance_id \u003d mount_info.get(\u0027share_instance_id\u0027)"},{"line_number":256,"context_line":"        backup \u003d mount_info.get(\u0027backup\u0027)"},{"line_number":257,"context_line":"        restore \u003d mount_info.get(\u0027restore\u0027)"}],"source_content_type":"text/x-python","patch_set":6,"id":"e69da5c1_e420d84b","line":254,"range":{"start_line":254,"start_character":0,"end_line":254,"end_character":48},"in_reply_to":"538c5862_f9f78627","updated":"2026-06-19 20:10:09.000000000","message":"actually, let\u0027s keep it as is","commit_id":"e584906f7b754ec41e6111b4aa9d3b3776523b84"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"2d4484c529ecc6152ad3052b8a600a69d99af357","unresolved":true,"context_lines":[{"line_number":301,"context_line":""},{"line_number":302,"context_line":"    def unmount_share_instance_or_backup(self, unmount_info, mount_path):"},{"line_number":303,"context_line":"        mount_point \u003d unmount_info.get(\u0027mount_point\u0027)"},{"line_number":304,"context_line":"        unmount_template \u003d unmount_info.get(\u0027unmount\u0027)"},{"line_number":305,"context_line":"        share_instance_id \u003d unmount_info.get(\u0027share_instance_id\u0027)"},{"line_number":306,"context_line":""},{"line_number":307,"context_line":"        if share_instance_id:"}],"source_content_type":"text/x-python","patch_set":6,"id":"404159de_08debc19","line":304,"range":{"start_line":304,"start_character":0,"end_line":304,"end_character":54},"updated":"2026-06-12 20:12:58.000000000","message":"actually unnecessary, I can just drop this as this is being ignored below","commit_id":"e584906f7b754ec41e6111b4aa9d3b3776523b84"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"7074b975b15dbdcd86eee5028c749f3a57d04b2b","unresolved":false,"context_lines":[{"line_number":301,"context_line":""},{"line_number":302,"context_line":"    def unmount_share_instance_or_backup(self, unmount_info, mount_path):"},{"line_number":303,"context_line":"        mount_point \u003d unmount_info.get(\u0027mount_point\u0027)"},{"line_number":304,"context_line":"        unmount_template \u003d unmount_info.get(\u0027unmount\u0027)"},{"line_number":305,"context_line":"        share_instance_id \u003d unmount_info.get(\u0027share_instance_id\u0027)"},{"line_number":306,"context_line":""},{"line_number":307,"context_line":"        if share_instance_id:"}],"source_content_type":"text/x-python","patch_set":6,"id":"4fe340f8_d9b1909b","line":304,"range":{"start_line":304,"start_character":0,"end_line":304,"end_character":54},"in_reply_to":"404159de_08debc19","updated":"2026-06-19 20:10:09.000000000","message":"Done","commit_id":"e584906f7b754ec41e6111b4aa9d3b3776523b84"}],"manila/privsep/os.py":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"f333614c048a970494ce663c5cc6f76a091fc87c","unresolved":true,"context_lines":[{"line_number":35,"context_line":"        extra_args.append(\u0027-p\u0027)"},{"line_number":36,"context_line":"    extra_args.append(dir_path)"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"    processutils.execute(\u0027mkdir\u0027, dir_path, *extra_args)"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"@manila.privsep.sys_admin_pctxt.entrypoint"}],"source_content_type":"text/x-python","patch_set":2,"id":"3a4c5e6a_a4303d14","line":38,"range":{"start_line":38,"start_character":34,"end_line":38,"end_character":42},"updated":"2025-01-09 14:31:18.000000000","message":"This causes duplicate dir_path in the command line.","commit_id":"b2190cd84951d91a7860e0f2c0837cf941bd3bda"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"a30562bd024049039ed2eaaf801239c24f7fa278","unresolved":false,"context_lines":[{"line_number":35,"context_line":"        extra_args.append(\u0027-p\u0027)"},{"line_number":36,"context_line":"    extra_args.append(dir_path)"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"    processutils.execute(\u0027mkdir\u0027, dir_path, *extra_args)"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"@manila.privsep.sys_admin_pctxt.entrypoint"}],"source_content_type":"text/x-python","patch_set":2,"id":"23d32a6d_c85f6cec","line":38,"range":{"start_line":38,"start_character":34,"end_line":38,"end_character":42},"in_reply_to":"3a4c5e6a_a4303d14","updated":"2026-04-28 20:53:34.000000000","message":"Done","commit_id":"b2190cd84951d91a7860e0f2c0837cf941bd3bda"}],"manila/tests/share/drivers/huawei/test_huawei_nas.py":[{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"eb8a7e95dd1cff9fd8b307817d6ddfcdc7775159","unresolved":true,"context_lines":[{"line_number":2187,"context_line":"        self.mock_object(utils,"},{"line_number":2188,"context_line":"                         \u0027execute\u0027,"},{"line_number":2189,"context_line":"                         mock.Mock(return_value\u003d(\"\", \"\")))"},{"line_number":2190,"context_line":"        # Mock Copy class for data copying"},{"line_number":2191,"context_line":"        mock_copy \u003d mock.Mock()"},{"line_number":2192,"context_line":"        mock_copy.get_progress.return_value \u003d {\u0027total_progress\u0027: 100}"},{"line_number":2193,"context_line":"        self.mock_object(data_utils, \u0027Copy\u0027,"}],"source_content_type":"text/x-python","patch_set":6,"id":"64add744_5e723016","line":2190,"range":{"start_line":2190,"start_character":0,"end_line":2190,"end_character":42},"updated":"2026-06-10 18:56:14.000000000","message":"remove comment","commit_id":"e584906f7b754ec41e6111b4aa9d3b3776523b84"}]}
