)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":13692,"name":"Roman Dobosz","email":"gryf73@gmail.com","username":"gryf"},"change_message_id":"d49027ae4119f418ef4603a2a4c3ab8baea66497","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"7d2c3029_73510bb3","updated":"2022-03-28 10:59:27.000000000","message":"recheck","commit_id":"549d6875cd690e7006758742d8f030724ff3f575"},{"author":{"_account_id":13692,"name":"Roman Dobosz","email":"gryf73@gmail.com","username":"gryf"},"change_message_id":"9c0e421799302537b60be33a1ce91e24b44175fe","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"03776389_b2387184","updated":"2022-03-30 06:32:13.000000000","message":"recheck","commit_id":"0a0fa8a5f77951c8d11bd434bd950c8d8e690971"},{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"a6c85573ecc0e830a41d280d88e92a882ed00a25","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"62a82256_4073bdc3","updated":"2022-04-08 15:55:07.000000000","message":"Do we still need this one?","commit_id":"ad3e66880b860efbbc5919d5dd137a040cef26b4"},{"author":{"_account_id":13692,"name":"Roman Dobosz","email":"gryf73@gmail.com","username":"gryf"},"change_message_id":"0aaa3ba9bc482642321585bfac807288399b5c9e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"97af8e65_09ca7182","in_reply_to":"62a82256_4073bdc3","updated":"2022-04-12 10:15:51.000000000","message":"Yes, we do. Without it tempests tests will crush on new netowrks/subnets names.","commit_id":"ad3e66880b860efbbc5919d5dd137a040cef26b4"},{"author":{"_account_id":13692,"name":"Roman Dobosz","email":"gryf73@gmail.com","username":"gryf"},"change_message_id":"b36ab4db846aca5b3669b8d1e57b22abd4f8b703","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"6216cd34_ba2ae757","updated":"2022-04-14 12:24:53.000000000","message":"recheck","commit_id":"331547d50a2cbdcb2c0c395b7b599b283515de1b"},{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"d250f89e18e159651b2990715c43525c8c415ec0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"901f0272_5a171af0","updated":"2022-04-14 16:15:59.000000000","message":"recheck","commit_id":"331547d50a2cbdcb2c0c395b7b599b283515de1b"},{"author":{"_account_id":13692,"name":"Roman Dobosz","email":"gryf73@gmail.com","username":"gryf"},"change_message_id":"a68bf111b29fe5b9456d79cdcc55dd3bf163654d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"bd411cbf_98fb0cdf","updated":"2022-04-15 06:28:37.000000000","message":"recheck","commit_id":"331547d50a2cbdcb2c0c395b7b599b283515de1b"},{"author":{"_account_id":27032,"name":"Maysa de Macedo Souza","email":"maysa.macedo95@gmail.com","username":"maysa"},"change_message_id":"b2fd66e675a69c3cd169a89f5fea9a557dae78e6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"be49f944_51d0bd4d","updated":"2022-04-18 17:50:03.000000000","message":"recheck","commit_id":"331547d50a2cbdcb2c0c395b7b599b283515de1b"}],"kuryr_tempest_plugin/tests/scenario/test_namespace.py":[{"author":{"_account_id":27032,"name":"Maysa de Macedo Souza","email":"maysa.macedo95@gmail.com","username":"maysa"},"change_message_id":"a6a8f60f8650864af6c5e44184f3f0789653cf82","unresolved":true,"context_lines":[{"line_number":62,"context_line":""},{"line_number":63,"context_line":"        seen_subnets \u003d self.os_admin.subnets_client.list_subnets()"},{"line_number":64,"context_line":"        seen_subnet_names \u003d [n[\u0027name\u0027] for n in seen_subnets[\u0027subnets\u0027]]"},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"        # TODO(gryf): Remove old nameing convention and the condition after"},{"line_number":67,"context_line":"        # 835078 merge, keeping subnet_name containing only ns name."},{"line_number":68,"context_line":"        subnet_name \u003d (f\"{namespace_name}\")"},{"line_number":69,"context_line":"        if subnet_name not in seen_subnet_names:"},{"line_number":70,"context_line":"            subnet_name \u003d \u0027ns/\u0027 + namespace_name + \u0027-subnet\u0027"}],"source_content_type":"text/x-python","patch_set":3,"id":"703ccfed_016be2d1","line":67,"range":{"start_line":65,"start_character":0,"end_line":67,"end_character":68},"updated":"2022-04-12 10:26:18.000000000","message":"I think you can remove this TODO given that we need to keep compatibility with all the branches. Note that tempest has only master.","commit_id":"ad3e66880b860efbbc5919d5dd137a040cef26b4"},{"author":{"_account_id":13692,"name":"Roman Dobosz","email":"gryf73@gmail.com","username":"gryf"},"change_message_id":"8e3d2108ec94b33374cf6fbce70591ec42bb4439","unresolved":false,"context_lines":[{"line_number":62,"context_line":""},{"line_number":63,"context_line":"        seen_subnets \u003d self.os_admin.subnets_client.list_subnets()"},{"line_number":64,"context_line":"        seen_subnet_names \u003d [n[\u0027name\u0027] for n in seen_subnets[\u0027subnets\u0027]]"},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"        # TODO(gryf): Remove old nameing convention and the condition after"},{"line_number":67,"context_line":"        # 835078 merge, keeping subnet_name containing only ns name."},{"line_number":68,"context_line":"        subnet_name \u003d (f\"{namespace_name}\")"},{"line_number":69,"context_line":"        if subnet_name not in seen_subnet_names:"},{"line_number":70,"context_line":"            subnet_name \u003d \u0027ns/\u0027 + namespace_name + \u0027-subnet\u0027"}],"source_content_type":"text/x-python","patch_set":3,"id":"fe40589a_ba4eeb3d","line":67,"range":{"start_line":65,"start_character":0,"end_line":67,"end_character":68},"in_reply_to":"703ccfed_016be2d1","updated":"2022-04-12 12:59:15.000000000","message":"Ack","commit_id":"ad3e66880b860efbbc5919d5dd137a040cef26b4"},{"author":{"_account_id":27032,"name":"Maysa de Macedo Souza","email":"maysa.macedo95@gmail.com","username":"maysa"},"change_message_id":"ae5be9a5367b8fb8d28f2c9cb7cc14f02065bfcd","unresolved":true,"context_lines":[{"line_number":65,"context_line":""},{"line_number":66,"context_line":"        # TODO(gryf): Remove old nameing convention and the condition after"},{"line_number":67,"context_line":"        # 835078 merge, keeping subnet_name containing only ns name."},{"line_number":68,"context_line":"        subnet_name \u003d (f\"{namespace_name}\")"},{"line_number":69,"context_line":"        if subnet_name not in seen_subnet_names:"},{"line_number":70,"context_line":"            subnet_name \u003d \u0027ns/\u0027 + namespace_name + \u0027-subnet\u0027"},{"line_number":71,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"f630d5f2_b2fd5010","line":68,"range":{"start_line":68,"start_character":8,"end_line":68,"end_character":43},"updated":"2022-04-07 10:14:08.000000000","message":"is this needed? I\u0027m fine with living it here for now as there will be another iteration here when 835078 merges","commit_id":"ad3e66880b860efbbc5919d5dd137a040cef26b4"},{"author":{"_account_id":27032,"name":"Maysa de Macedo Souza","email":"maysa.macedo95@gmail.com","username":"maysa"},"change_message_id":"a6a8f60f8650864af6c5e44184f3f0789653cf82","unresolved":false,"context_lines":[{"line_number":65,"context_line":""},{"line_number":66,"context_line":"        # TODO(gryf): Remove old nameing convention and the condition after"},{"line_number":67,"context_line":"        # 835078 merge, keeping subnet_name containing only ns name."},{"line_number":68,"context_line":"        subnet_name \u003d (f\"{namespace_name}\")"},{"line_number":69,"context_line":"        if subnet_name not in seen_subnet_names:"},{"line_number":70,"context_line":"            subnet_name \u003d \u0027ns/\u0027 + namespace_name + \u0027-subnet\u0027"},{"line_number":71,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"847b100d_cc2b1391","line":68,"range":{"start_line":68,"start_character":8,"end_line":68,"end_character":43},"in_reply_to":"027b7419_5dc7b221","updated":"2022-04-12 10:26:18.000000000","message":"would you like to update that?","commit_id":"ad3e66880b860efbbc5919d5dd137a040cef26b4"},{"author":{"_account_id":13692,"name":"Roman Dobosz","email":"gryf73@gmail.com","username":"gryf"},"change_message_id":"8e3d2108ec94b33374cf6fbce70591ec42bb4439","unresolved":false,"context_lines":[{"line_number":65,"context_line":""},{"line_number":66,"context_line":"        # TODO(gryf): Remove old nameing convention and the condition after"},{"line_number":67,"context_line":"        # 835078 merge, keeping subnet_name containing only ns name."},{"line_number":68,"context_line":"        subnet_name \u003d (f\"{namespace_name}\")"},{"line_number":69,"context_line":"        if subnet_name not in seen_subnet_names:"},{"line_number":70,"context_line":"            subnet_name \u003d \u0027ns/\u0027 + namespace_name + \u0027-subnet\u0027"},{"line_number":71,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"3be6c3ea_5fb2fcf5","line":68,"range":{"start_line":68,"start_character":8,"end_line":68,"end_character":43},"in_reply_to":"847b100d_cc2b1391","updated":"2022-04-12 12:59:15.000000000","message":"I do, since need to get rid of the comments","commit_id":"ad3e66880b860efbbc5919d5dd137a040cef26b4"},{"author":{"_account_id":13692,"name":"Roman Dobosz","email":"gryf73@gmail.com","username":"gryf"},"change_message_id":"bb903048539a25f316f7b343f3450898a8aa0354","unresolved":false,"context_lines":[{"line_number":65,"context_line":""},{"line_number":66,"context_line":"        # TODO(gryf): Remove old nameing convention and the condition after"},{"line_number":67,"context_line":"        # 835078 merge, keeping subnet_name containing only ns name."},{"line_number":68,"context_line":"        subnet_name \u003d (f\"{namespace_name}\")"},{"line_number":69,"context_line":"        if subnet_name not in seen_subnet_names:"},{"line_number":70,"context_line":"            subnet_name \u003d \u0027ns/\u0027 + namespace_name + \u0027-subnet\u0027"},{"line_number":71,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"027b7419_5dc7b221","line":68,"range":{"start_line":68,"start_character":8,"end_line":68,"end_character":43},"in_reply_to":"f630d5f2_b2fd5010","updated":"2022-04-07 13:39:27.000000000","message":"we need that so that my patches won\u0027t fail on this tests, where newly created subnets would have different name than ns/namespace_name-subnet. Although I just realized, subnet name could be just:\n\n  subnet_name \u003d namespace_name","commit_id":"ad3e66880b860efbbc5919d5dd137a040cef26b4"},{"author":{"_account_id":27032,"name":"Maysa de Macedo Souza","email":"maysa.macedo95@gmail.com","username":"maysa"},"change_message_id":"a6a8f60f8650864af6c5e44184f3f0789653cf82","unresolved":true,"context_lines":[{"line_number":152,"context_line":"        self.assertIn(consts.POD_OUTPUT,"},{"line_number":153,"context_line":"                      self.exec_command_in_pod(pod_nsdefault_name, cmd))"},{"line_number":154,"context_line":""},{"line_number":155,"context_line":"        # TODO(gryf): Remove this call and conditionals below, and just keep"},{"line_number":156,"context_line":"        # name of the subnet returned by _get_and_check_ns_resources function"},{"line_number":157,"context_line":"        seen_subnets \u003d self.os_admin.subnets_client.list_subnets()"},{"line_number":158,"context_line":"        if subnet_ns1_name not in seen_subnets:"}],"source_content_type":"text/x-python","patch_set":3,"id":"b293742e_2a36f64b","line":155,"updated":"2022-04-12 10:26:18.000000000","message":"ditto","commit_id":"ad3e66880b860efbbc5919d5dd137a040cef26b4"},{"author":{"_account_id":13692,"name":"Roman Dobosz","email":"gryf73@gmail.com","username":"gryf"},"change_message_id":"8e3d2108ec94b33374cf6fbce70591ec42bb4439","unresolved":false,"context_lines":[{"line_number":152,"context_line":"        self.assertIn(consts.POD_OUTPUT,"},{"line_number":153,"context_line":"                      self.exec_command_in_pod(pod_nsdefault_name, cmd))"},{"line_number":154,"context_line":""},{"line_number":155,"context_line":"        # TODO(gryf): Remove this call and conditionals below, and just keep"},{"line_number":156,"context_line":"        # name of the subnet returned by _get_and_check_ns_resources function"},{"line_number":157,"context_line":"        seen_subnets \u003d self.os_admin.subnets_client.list_subnets()"},{"line_number":158,"context_line":"        if subnet_ns1_name not in seen_subnets:"}],"source_content_type":"text/x-python","patch_set":3,"id":"dbbd2d9f_23d83f5e","line":155,"in_reply_to":"b293742e_2a36f64b","updated":"2022-04-12 12:59:15.000000000","message":"Ack","commit_id":"ad3e66880b860efbbc5919d5dd137a040cef26b4"},{"author":{"_account_id":27032,"name":"Maysa de Macedo Souza","email":"maysa.macedo95@gmail.com","username":"maysa"},"change_message_id":"a6a8f60f8650864af6c5e44184f3f0789653cf82","unresolved":true,"context_lines":[{"line_number":256,"context_line":"                      self.exec_command_in_pod(pod_ns1_name, cmd, ns1_name))"},{"line_number":257,"context_line":""},{"line_number":258,"context_line":"        # Check resources are deleted"},{"line_number":259,"context_line":"        # TODO(gryf): Remove this call and conditionals below, and just keep"},{"line_number":260,"context_line":"        # name of the subnet returned by _get_and_check_ns_resources function"},{"line_number":261,"context_line":"        seen_subnets \u003d self.os_admin.subnets_client.list_subnets()"},{"line_number":262,"context_line":"        if subnet_ns1_name not in seen_subnets:"}],"source_content_type":"text/x-python","patch_set":3,"id":"5e455dca_3cd0990c","line":259,"updated":"2022-04-12 10:26:18.000000000","message":"ditto","commit_id":"ad3e66880b860efbbc5919d5dd137a040cef26b4"},{"author":{"_account_id":13692,"name":"Roman Dobosz","email":"gryf73@gmail.com","username":"gryf"},"change_message_id":"8e3d2108ec94b33374cf6fbce70591ec42bb4439","unresolved":false,"context_lines":[{"line_number":256,"context_line":"                      self.exec_command_in_pod(pod_ns1_name, cmd, ns1_name))"},{"line_number":257,"context_line":""},{"line_number":258,"context_line":"        # Check resources are deleted"},{"line_number":259,"context_line":"        # TODO(gryf): Remove this call and conditionals below, and just keep"},{"line_number":260,"context_line":"        # name of the subnet returned by _get_and_check_ns_resources function"},{"line_number":261,"context_line":"        seen_subnets \u003d self.os_admin.subnets_client.list_subnets()"},{"line_number":262,"context_line":"        if subnet_ns1_name not in seen_subnets:"}],"source_content_type":"text/x-python","patch_set":3,"id":"b6baf16a_929ef8be","line":259,"in_reply_to":"5e455dca_3cd0990c","updated":"2022-04-12 12:59:15.000000000","message":"Ack","commit_id":"ad3e66880b860efbbc5919d5dd137a040cef26b4"}],"kuryr_tempest_plugin/tests/scenario/test_port_pool.py":[{"author":{"_account_id":27032,"name":"Maysa de Macedo Souza","email":"maysa.macedo95@gmail.com","username":"maysa"},"change_message_id":"a6a8f60f8650864af6c5e44184f3f0789653cf82","unresolved":true,"context_lines":[{"line_number":61,"context_line":"        if subnet_id:"},{"line_number":62,"context_line":"            subnet_id \u003d subnet_id[0]"},{"line_number":63,"context_line":"        else:"},{"line_number":64,"context_line":"            # TODO(gryf): Remove this block after 835078 merge"},{"line_number":65,"context_line":"            subnet_name \u003d f\u0027ns/{namespace_name}-subnet\u0027"},{"line_number":66,"context_line":"            subnet_id \u003d [n[\u0027id\u0027] for n in subnets_list[\u0027subnets\u0027]"},{"line_number":67,"context_line":"                         if n[\u0027name\u0027] \u003d\u003d subnet_name][0]"}],"source_content_type":"text/x-python","patch_set":3,"id":"288cb140_3a3d9a78","line":64,"range":{"start_line":64,"start_character":12,"end_line":64,"end_character":62},"updated":"2022-04-12 10:26:18.000000000","message":"ditto","commit_id":"ad3e66880b860efbbc5919d5dd137a040cef26b4"},{"author":{"_account_id":13692,"name":"Roman Dobosz","email":"gryf73@gmail.com","username":"gryf"},"change_message_id":"8e3d2108ec94b33374cf6fbce70591ec42bb4439","unresolved":false,"context_lines":[{"line_number":61,"context_line":"        if subnet_id:"},{"line_number":62,"context_line":"            subnet_id \u003d subnet_id[0]"},{"line_number":63,"context_line":"        else:"},{"line_number":64,"context_line":"            # TODO(gryf): Remove this block after 835078 merge"},{"line_number":65,"context_line":"            subnet_name \u003d f\u0027ns/{namespace_name}-subnet\u0027"},{"line_number":66,"context_line":"            subnet_id \u003d [n[\u0027id\u0027] for n in subnets_list[\u0027subnets\u0027]"},{"line_number":67,"context_line":"                         if n[\u0027name\u0027] \u003d\u003d subnet_name][0]"}],"source_content_type":"text/x-python","patch_set":3,"id":"77eb14a7_8d671b7d","line":64,"range":{"start_line":64,"start_character":12,"end_line":64,"end_character":62},"in_reply_to":"288cb140_3a3d9a78","updated":"2022-04-12 12:59:15.000000000","message":"Ack","commit_id":"ad3e66880b860efbbc5919d5dd137a040cef26b4"}]}