)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"9584736a1e8667f308e3deba8b8925981e7bbca5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"bb2ee581_9e5020de","updated":"2022-07-19 13:19:52.000000000","message":"LGTM","commit_id":"b26e1f17631abf36e0805d12701ed2b6e1c94dce"},{"author":{"_account_id":13671,"name":"Vladislav Belogrudov","email":"v.belogrudov@yadro.com","username":"vb"},"change_message_id":"25607989f59782933d15c8709359d1a1811ec709","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"3a30b2d4_dcc00e58","updated":"2022-07-22 07:16:59.000000000","message":"run-Yadro Tatlin Unified CI","commit_id":"b26e1f17631abf36e0805d12701ed2b6e1c94dce"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"6f389a462cc16d1aa2eed41a114e937d93a85691","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"16f24a3c_97b3435a","updated":"2022-08-19 15:57:42.000000000","message":"LGTM\n\nPassed mypy locally\n\n  mypy: commands succeeded\n  congratulations :)\n","commit_id":"349513bb8aff713b36a6caf26e24a38ea02e167c"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"52fc7685b5866272dadffe35f9b932006e256a41","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"5ad89270_d863920d","updated":"2022-08-19 14:29:13.000000000","message":"Looks good to me, just a minor missing. ","commit_id":"349513bb8aff713b36a6caf26e24a38ea02e167c"},{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"96337a1be1f19494fe50e5a2556e47197618a5ef","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"1418f1e8_2e62e8f0","updated":"2022-08-19 15:51:41.000000000","message":"Looks good.  The one question that Sofia notes can probably be handled in a follow up patch?","commit_id":"349513bb8aff713b36a6caf26e24a38ea02e167c"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"0fbe3dc46ba34b75026918076725c0dc85026de4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"3ed619f5_f9a86361","updated":"2022-08-25 12:21:46.000000000","message":"LGTM","commit_id":"4c64879c39a172aca38156b25077b340b0ebf06e"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"548031c875a96321d4c1dd32790f48d04265761f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"6654f3b7_a9106c26","updated":"2022-08-23 18:22:26.000000000","message":"recheck\n\ntempest-full-py3\ntempest.api.compute.servers.test_server_actions.ServerActionsTestJSON.test_resize_volume_backed_server_confirm\nInstance could not be found","commit_id":"4c64879c39a172aca38156b25077b340b0ebf06e"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"d531a81c033ddbe30723bc237b1d45eb79295363","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"1d22fb04_affe308d","updated":"2022-08-23 15:17:49.000000000","message":"still LGTM after the revision","commit_id":"4c64879c39a172aca38156b25077b340b0ebf06e"}],"os_brick/local_dev/lvm.py":[{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"52fc7685b5866272dadffe35f9b932006e256a41","unresolved":true,"context_lines":[{"line_number":240,"context_line":"                version \u003d version_list[2]"},{"line_number":241,"context_line":"                version_filter \u003d r\"(\\d+)\\.(\\d+)\\.(\\d+).*\""},{"line_number":242,"context_line":"                r \u003d re.search(version_filter, version)"},{"line_number":243,"context_line":"                if r is None:"},{"line_number":244,"context_line":"                    raise exception.BrickException("},{"line_number":245,"context_line":"                        message\u003d\u0027Cannot parse LVM version\u0027)"},{"line_number":246,"context_line":"                version_tuple \u003d tuple(map(int, r.group(1, 2, 3)))"},{"line_number":247,"context_line":"                return version_tuple"},{"line_number":248,"context_line":"        raise"}],"source_content_type":"text/x-python","patch_set":7,"id":"9f5db9b6_8e0e6280","line":245,"range":{"start_line":243,"start_character":0,"end_line":245,"end_character":59},"updated":"2022-08-19 14:29:13.000000000","message":"guess this extends mypy scope but looks OK!","commit_id":"349513bb8aff713b36a6caf26e24a38ea02e167c"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"ede18bfff10de23d653e174936490fd3f6d5b66d","unresolved":false,"context_lines":[{"line_number":240,"context_line":"                version \u003d version_list[2]"},{"line_number":241,"context_line":"                version_filter \u003d r\"(\\d+)\\.(\\d+)\\.(\\d+).*\""},{"line_number":242,"context_line":"                r \u003d re.search(version_filter, version)"},{"line_number":243,"context_line":"                if r is None:"},{"line_number":244,"context_line":"                    raise exception.BrickException("},{"line_number":245,"context_line":"                        message\u003d\u0027Cannot parse LVM version\u0027)"},{"line_number":246,"context_line":"                version_tuple \u003d tuple(map(int, r.group(1, 2, 3)))"},{"line_number":247,"context_line":"                return version_tuple"},{"line_number":248,"context_line":"        raise"}],"source_content_type":"text/x-python","patch_set":7,"id":"56824a8f_3ee537c4","line":245,"range":{"start_line":243,"start_character":0,"end_line":245,"end_character":59},"in_reply_to":"78d70f26_0c89ab30","updated":"2022-08-22 19:08:55.000000000","message":"Nope, it doesn\u0027t.  This is for mypy.  Mypy can deduce after this that r is not None.\n\nRemove this change and you will see that mypy fails.","commit_id":"349513bb8aff713b36a6caf26e24a38ea02e167c"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"4a25f653382cbd1a3e9e02b6ab95be8468005b97","unresolved":false,"context_lines":[{"line_number":240,"context_line":"                version \u003d version_list[2]"},{"line_number":241,"context_line":"                version_filter \u003d r\"(\\d+)\\.(\\d+)\\.(\\d+).*\""},{"line_number":242,"context_line":"                r \u003d re.search(version_filter, version)"},{"line_number":243,"context_line":"                if r is None:"},{"line_number":244,"context_line":"                    raise exception.BrickException("},{"line_number":245,"context_line":"                        message\u003d\u0027Cannot parse LVM version\u0027)"},{"line_number":246,"context_line":"                version_tuple \u003d tuple(map(int, r.group(1, 2, 3)))"},{"line_number":247,"context_line":"                return version_tuple"},{"line_number":248,"context_line":"        raise"}],"source_content_type":"text/x-python","patch_set":7,"id":"78d70f26_0c89ab30","line":245,"range":{"start_line":243,"start_character":0,"end_line":245,"end_character":59},"in_reply_to":"9f5db9b6_8e0e6280","updated":"2022-08-19 14:29:32.000000000","message":"Done","commit_id":"349513bb8aff713b36a6caf26e24a38ea02e167c"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"6f389a462cc16d1aa2eed41a114e937d93a85691","unresolved":true,"context_lines":[{"line_number":373,"context_line":""},{"line_number":374,"context_line":"    @staticmethod"},{"line_number":375,"context_line":"    def get_all_physical_volumes(root_helper: str,"},{"line_number":376,"context_line":"                                 vg_name: Optional[str] \u003d None) -\u003e list:"},{"line_number":377,"context_line":"        \"\"\"Static method to get all PVs on a system."},{"line_number":378,"context_line":""},{"line_number":379,"context_line":"        :param root_helper: root_helper to use for execute"}],"source_content_type":"text/x-python","patch_set":7,"id":"aa618556_b1e3104d","line":376,"range":{"start_line":376,"start_character":67,"end_line":376,"end_character":71},"updated":"2022-08-19 15:57:42.000000000","message":"list[dict[str, Any]]","commit_id":"349513bb8aff713b36a6caf26e24a38ea02e167c"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"3dce1292aa22b5e7acf94e44d4d87d5aa1a09d19","unresolved":false,"context_lines":[{"line_number":373,"context_line":""},{"line_number":374,"context_line":"    @staticmethod"},{"line_number":375,"context_line":"    def get_all_physical_volumes(root_helper: str,"},{"line_number":376,"context_line":"                                 vg_name: Optional[str] \u003d None) -\u003e list:"},{"line_number":377,"context_line":"        \"\"\"Static method to get all PVs on a system."},{"line_number":378,"context_line":""},{"line_number":379,"context_line":"        :param root_helper: root_helper to use for execute"}],"source_content_type":"text/x-python","patch_set":7,"id":"609c99be_16b086fc","line":376,"range":{"start_line":376,"start_character":67,"end_line":376,"end_character":71},"in_reply_to":"aa618556_b1e3104d","updated":"2022-08-22 19:26:36.000000000","message":"Done","commit_id":"349513bb8aff713b36a6caf26e24a38ea02e167c"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"6f389a462cc16d1aa2eed41a114e937d93a85691","unresolved":true,"context_lines":[{"line_number":404,"context_line":"                            \u0027available\u0027: float(fields[3])})"},{"line_number":405,"context_line":"        return pv_list"},{"line_number":406,"context_line":""},{"line_number":407,"context_line":"    def get_physical_volumes(self) -\u003e list:"},{"line_number":408,"context_line":"        \"\"\"Get all PVs associated with this instantiation (VG)."},{"line_number":409,"context_line":""},{"line_number":410,"context_line":"        :returns: List of Dictionaries with PV info"}],"source_content_type":"text/x-python","patch_set":7,"id":"1f6c435b_3d5f1191","line":407,"range":{"start_line":407,"start_character":38,"end_line":407,"end_character":42},"updated":"2022-08-19 15:57:42.000000000","message":"list[dict[str, Any]]","commit_id":"349513bb8aff713b36a6caf26e24a38ea02e167c"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"3dce1292aa22b5e7acf94e44d4d87d5aa1a09d19","unresolved":false,"context_lines":[{"line_number":404,"context_line":"                            \u0027available\u0027: float(fields[3])})"},{"line_number":405,"context_line":"        return pv_list"},{"line_number":406,"context_line":""},{"line_number":407,"context_line":"    def get_physical_volumes(self) -\u003e list:"},{"line_number":408,"context_line":"        \"\"\"Get all PVs associated with this instantiation (VG)."},{"line_number":409,"context_line":""},{"line_number":410,"context_line":"        :returns: List of Dictionaries with PV info"}],"source_content_type":"text/x-python","patch_set":7,"id":"a24c7fce_ed696aaf","line":407,"range":{"start_line":407,"start_character":38,"end_line":407,"end_character":42},"in_reply_to":"1f6c435b_3d5f1191","updated":"2022-08-22 19:26:36.000000000","message":"Done","commit_id":"349513bb8aff713b36a6caf26e24a38ea02e167c"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"52fc7685b5866272dadffe35f9b932006e256a41","unresolved":true,"context_lines":[{"line_number":578,"context_line":""},{"line_number":579,"context_line":"    def create_volume(self,"},{"line_number":580,"context_line":"                      name: str,"},{"line_number":581,"context_line":"                      size_str,"},{"line_number":582,"context_line":"                      lv_type: str \u003d \u0027default\u0027,"},{"line_number":583,"context_line":"                      mirror_count: int \u003d 0) -\u003e None:"},{"line_number":584,"context_line":"        \"\"\"Creates a logical volume on the object\u0027s VG."}],"source_content_type":"text/x-python","patch_set":7,"id":"846ee9d6_da11d0d2","line":581,"range":{"start_line":581,"start_character":22,"end_line":581,"end_character":30},"updated":"2022-08-19 14:29:13.000000000","message":"`size_str : str` ?","commit_id":"349513bb8aff713b36a6caf26e24a38ea02e167c"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"3dce1292aa22b5e7acf94e44d4d87d5aa1a09d19","unresolved":false,"context_lines":[{"line_number":578,"context_line":""},{"line_number":579,"context_line":"    def create_volume(self,"},{"line_number":580,"context_line":"                      name: str,"},{"line_number":581,"context_line":"                      size_str,"},{"line_number":582,"context_line":"                      lv_type: str \u003d \u0027default\u0027,"},{"line_number":583,"context_line":"                      mirror_count: int \u003d 0) -\u003e None:"},{"line_number":584,"context_line":"        \"\"\"Creates a logical volume on the object\u0027s VG."}],"source_content_type":"text/x-python","patch_set":7,"id":"ef1052f8_e3e22a27","line":581,"range":{"start_line":581,"start_character":22,"end_line":581,"end_character":30},"in_reply_to":"846ee9d6_da11d0d2","updated":"2022-08-22 19:26:36.000000000","message":"Done","commit_id":"349513bb8aff713b36a6caf26e24a38ea02e167c"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"52fc7685b5866272dadffe35f9b932006e256a41","unresolved":true,"context_lines":[{"line_number":599,"context_line":"                                        \u0027-L\u0027, size_str]"},{"line_number":600,"context_line":""},{"line_number":601,"context_line":"        if mirror_count \u003e 0:"},{"line_number":602,"context_line":"            cmd.extend([\u0027-m\u0027, str(mirror_count), \u0027--nosync\u0027,"},{"line_number":603,"context_line":"                        \u0027--mirrorlog\u0027, \u0027mirrored\u0027])"},{"line_number":604,"context_line":"            terras \u003d int(size_str[:-1]) / 1024.0"},{"line_number":605,"context_line":"            if terras \u003e\u003d 1.5:"}],"source_content_type":"text/x-python","patch_set":7,"id":"2d5e7707_aa7e40da","line":602,"range":{"start_line":602,"start_character":30,"end_line":602,"end_character":47},"updated":"2022-08-19 14:29:13.000000000","message":"OK 😊","commit_id":"349513bb8aff713b36a6caf26e24a38ea02e167c"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"4a25f653382cbd1a3e9e02b6ab95be8468005b97","unresolved":false,"context_lines":[{"line_number":599,"context_line":"                                        \u0027-L\u0027, size_str]"},{"line_number":600,"context_line":""},{"line_number":601,"context_line":"        if mirror_count \u003e 0:"},{"line_number":602,"context_line":"            cmd.extend([\u0027-m\u0027, str(mirror_count), \u0027--nosync\u0027,"},{"line_number":603,"context_line":"                        \u0027--mirrorlog\u0027, \u0027mirrored\u0027])"},{"line_number":604,"context_line":"            terras \u003d int(size_str[:-1]) / 1024.0"},{"line_number":605,"context_line":"            if terras \u003e\u003d 1.5:"}],"source_content_type":"text/x-python","patch_set":7,"id":"631aa2eb_1fa09d56","line":602,"range":{"start_line":602,"start_character":30,"end_line":602,"end_character":47},"in_reply_to":"2d5e7707_aa7e40da","updated":"2022-08-19 14:29:32.000000000","message":"Done","commit_id":"349513bb8aff713b36a6caf26e24a38ea02e167c"}]}