)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":33240,"name":"Sunday Mgbogu","email":"digitalsimboja@gmail.com","username":"digitalsimboja"},"change_message_id":"a48a3ba91e156c6db907c85eb115eece88d7c930","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"087791c8_6bbe96da","updated":"2022-02-01 07:45:16.000000000","message":"recheck","commit_id":"1635f4ea6e27c435283a851c006b7e977ccdd4f3"},{"author":{"_account_id":33240,"name":"Sunday Mgbogu","email":"digitalsimboja@gmail.com","username":"digitalsimboja"},"change_message_id":"f0799830bd2861d7b0877fe39bfe2d93bcc08bb2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"5af19341_68844768","updated":"2022-02-07 08:07:52.000000000","message":"recheck","commit_id":"1635f4ea6e27c435283a851c006b7e977ccdd4f3"},{"author":{"_account_id":33240,"name":"Sunday Mgbogu","email":"digitalsimboja@gmail.com","username":"digitalsimboja"},"change_message_id":"9f41b65541b3c2cf2e6e28f8706cf25dab379b8f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"72ab5c43_1dff96fd","updated":"2022-02-03 08:43:54.000000000","message":"recheck","commit_id":"1635f4ea6e27c435283a851c006b7e977ccdd4f3"},{"author":{"_account_id":27032,"name":"Maysa de Macedo Souza","email":"maysa.macedo95@gmail.com","username":"maysa"},"change_message_id":"ca9e394f3dea39af5bcf7adeab45da6f4443d4d6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"67393acb_22569f0a","updated":"2022-02-13 00:46:19.000000000","message":"Looks good. There are just tiny remarks and it would be great to have tempest tests covering this scenario.","commit_id":"4f08d2aaacd4849b85a0e7cdc6367ee8c86bff6d"},{"author":{"_account_id":33240,"name":"Sunday Mgbogu","email":"digitalsimboja@gmail.com","username":"digitalsimboja"},"change_message_id":"b184212e722a26396c3da4cb5f9528a44186f35a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"eb3f6948_1a91884d","updated":"2022-02-08 08:28:32.000000000","message":"recheck","commit_id":"4f08d2aaacd4849b85a0e7cdc6367ee8c86bff6d"},{"author":{"_account_id":33240,"name":"Sunday Mgbogu","email":"digitalsimboja@gmail.com","username":"digitalsimboja"},"change_message_id":"2e904cfe8aa34f13c6ab07489188f5eff386d371","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"a6bbfdd2_fadd6f6d","updated":"2022-03-09 05:24:02.000000000","message":"Calling for reviews here","commit_id":"01e8212ede1ef89fb85b07dfdff4644b6f4d10bb"},{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"168a9db8710e90a56b902c3de4d5302d79857901","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"129e0bee_dae027da","updated":"2022-03-10 11:41:22.000000000","message":"Looks good!","commit_id":"01e8212ede1ef89fb85b07dfdff4644b6f4d10bb"},{"author":{"_account_id":33240,"name":"Sunday Mgbogu","email":"digitalsimboja@gmail.com","username":"digitalsimboja"},"change_message_id":"dc99c15bd99142540369f5de216188048ecefa83","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"d12876ed_d980bb5a","updated":"2022-02-13 22:34:46.000000000","message":"recheck","commit_id":"01e8212ede1ef89fb85b07dfdff4644b6f4d10bb"},{"author":{"_account_id":33240,"name":"Sunday Mgbogu","email":"digitalsimboja@gmail.com","username":"digitalsimboja"},"change_message_id":"16326a27c7ed4d69467cda679741a9a7236eda56","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"bc766c87_3bb66707","updated":"2022-03-16 07:59:25.000000000","message":"recheck","commit_id":"7e85dc0f5df8adc4085a783e555a6683eb039c61"},{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"93a79fa810ec1e781eb5a7a1151c42ce86e4889c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"6ef17391_666b011a","updated":"2022-03-17 10:28:16.000000000","message":"recheck\n\nCI should be more stable nowe as fix [1] got in.\n\n[1] https://review.opendev.org/c/openstack/kuryr-kubernetes/+/833994","commit_id":"7e85dc0f5df8adc4085a783e555a6683eb039c61"},{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"1add01f6b19810b88dadac98bf3143393564c3a5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"17606c01_568d5bb6","updated":"2022-03-17 13:45:18.000000000","message":"recheck\n\nUnrelated.","commit_id":"7e85dc0f5df8adc4085a783e555a6683eb039c61"}],"kuryr_kubernetes/controller/handlers/loadbalancer.py":[{"author":{"_account_id":27032,"name":"Maysa de Macedo Souza","email":"maysa.macedo95@gmail.com","username":"maysa"},"change_message_id":"ca9e394f3dea39af5bcf7adeab45da6f4443d4d6","unresolved":true,"context_lines":[{"line_number":207,"context_line":"                                          \u0027pklb\u0027: klb})"},{"line_number":208,"context_line":""},{"line_number":209,"context_line":"        resources_already_triggered \u003d []"},{"line_number":210,"context_line":"        # let\u0027s reconcile load balancers first, and then listeners"},{"line_number":211,"context_line":"        resource_types \u003d (\u0027loadbalancer\u0027, \u0027listener\u0027, \u0027pool\u0027)"},{"line_number":212,"context_line":"        for resource_type in resource_types:"},{"line_number":213,"context_line":"            filters \u003d {}"}],"source_content_type":"text/x-python","patch_set":2,"id":"bf2726eb_bcb52917","line":210,"updated":"2022-02-13 00:46:19.000000000","message":"missing to mention pools:\n\n\"let\u0027s reconcile load balancers first, listeners and then pools\"","commit_id":"4f08d2aaacd4849b85a0e7cdc6367ee8c86bff6d"},{"author":{"_account_id":27032,"name":"Maysa de Macedo Souza","email":"maysa.macedo95@gmail.com","username":"maysa"},"change_message_id":"ca9e394f3dea39af5bcf7adeab45da6f4443d4d6","unresolved":true,"context_lines":[{"line_number":238,"context_line":"                                \u0027Warning\u0027)"},{"line_number":239,"context_line":"            if data.get(\u0027pklb\u0027):"},{"line_number":240,"context_line":"                self.add_event(data[\u0027pklb\u0027], \u0027LoadBalancerRecreating\u0027,"},{"line_number":241,"context_line":"                               \u0027Pool of Load Balancer for the Service\u0027"},{"line_number":242,"context_line":"                               \u0027seems to not exist anymore. Recreating it.\u0027,"},{"line_number":243,"context_line":"                               \u0027Warning\u0027)"},{"line_number":244,"context_line":"            kubernetes.patch_crd(\u0027status\u0027, data[\u0027selflink\u0027], {})"},{"line_number":245,"context_line":"        except k_exc.K8sResourceNotFound:"}],"source_content_type":"text/x-python","patch_set":2,"id":"d5af3376_08af38e1","line":242,"range":{"start_line":241,"start_character":32,"end_line":242,"end_character":59},"updated":"2022-02-13 00:46:19.000000000","message":"Maybe it could be rephrased to:\n\n\"Load Balancer pool does not exist anymore. Recreating it.\"","commit_id":"4f08d2aaacd4849b85a0e7cdc6367ee8c86bff6d"},{"author":{"_account_id":27032,"name":"Maysa de Macedo Souza","email":"maysa.macedo95@gmail.com","username":"maysa"},"change_message_id":"8ef114f8f6b231c104942fc6a749c1638f2b6894","unresolved":true,"context_lines":[{"line_number":235,"context_line":"                self._add_event(data[\u0027lklb\u0027], \u0027Load Balancer listener does not\u0027"},{"line_number":236,"context_line":"                                \u0027 exist anymore. Recreating it.\u0027, \u0027Warning\u0027)"},{"line_number":237,"context_line":"            if data.get(\u0027pklb\u0027):"},{"line_number":238,"context_line":"                self.add_event(data[\u0027pklb\u0027], \u0027Load Balancer pool does not \u0027"},{"line_number":239,"context_line":"                               \u0027exist anymore. Recreating it.\u0027, \u0027Warning\u0027)"},{"line_number":240,"context_line":""},{"line_number":241,"context_line":"            kubernetes.patch_crd(\u0027status\u0027, data[\u0027selflink\u0027], {})"}],"source_content_type":"text/x-python","patch_set":4,"id":"3a12aa5f_dcae1cd5","line":238,"range":{"start_line":238,"start_character":21,"end_line":238,"end_character":30},"updated":"2022-03-15 18:06:34.000000000","message":"this would break the reconciliation process, there is no add_event method in this file. I\u0027m including the fix.","commit_id":"01e8212ede1ef89fb85b07dfdff4644b6f4d10bb"},{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"e0dcc04fb40516e2ab8546b725b146bcec2eae02","unresolved":true,"context_lines":[{"line_number":235,"context_line":"                self._add_event(data[\u0027lklb\u0027], \u0027Load Balancer listener does not\u0027"},{"line_number":236,"context_line":"                                \u0027 exist anymore. Recreating it.\u0027, \u0027Warning\u0027)"},{"line_number":237,"context_line":"            if data.get(\u0027pklb\u0027):"},{"line_number":238,"context_line":"                self.add_event(data[\u0027pklb\u0027], \u0027Load Balancer pool does not \u0027"},{"line_number":239,"context_line":"                               \u0027exist anymore. Recreating it.\u0027, \u0027Warning\u0027)"},{"line_number":240,"context_line":""},{"line_number":241,"context_line":"            kubernetes.patch_crd(\u0027status\u0027, data[\u0027selflink\u0027], {})"}],"source_content_type":"text/x-python","patch_set":4,"id":"1a1e064c_6b79cb53","line":238,"range":{"start_line":238,"start_character":21,"end_line":238,"end_character":30},"in_reply_to":"3a12aa5f_dcae1cd5","updated":"2022-03-17 11:39:40.000000000","message":"Wow, nice.","commit_id":"01e8212ede1ef89fb85b07dfdff4644b6f4d10bb"}]}