)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":37089,"name":"Chris Buggy","display_name":"cbuggy","email":"cbuggy@redhat.com","username":"cbuggy"},"change_message_id":"5e5c9adab98a8a3785a5660ed9ce3b131e97d3ee","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"2ec46483_25e56a03","updated":"2024-11-27 17:08:04.000000000","message":"Two questions on a possible indentation error.\nOne Nit on comments that can be ignored.\nGood work on adding the tests it should bring up our coverage.\nI\u0027ll change to +1 once addressed.","commit_id":"385e153e83b776f355d4db3f32ac8b65a57ecff9"},{"author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"change_message_id":"98bca62dd5a10a7409627b48f3fd3cb9cd74ebc5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"900c02d2_8ad0b498","updated":"2024-11-21 16:28:20.000000000","message":"any change this has backport potential into 2024.1?","commit_id":"385e153e83b776f355d4db3f32ac8b65a57ecff9"},{"author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"change_message_id":"b3dbae07eb9bc1fc190485dc290a66cf36130e90","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"80c72f27_85ea6f00","updated":"2024-11-21 16:27:53.000000000","message":"this would be great to have.","commit_id":"385e153e83b776f355d4db3f32ac8b65a57ecff9"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"4c266b2bbafa8287dfdfb1feac6083ccd7f1e0e2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"1c3530d9_ea631595","updated":"2025-01-14 14:31:14.000000000","message":"please fix pep8 issues 😊","commit_id":"5d5b3c9d3c35f039061fffac93a74b7b2936ab95"},{"author":{"_account_id":12404,"name":"Rico Lin","email":"ricolin@ricolky.com","username":"rico.lin"},"change_message_id":"411f737592781de90bf1c230b0b0743da3e33a96","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"82edffac_d598a753","updated":"2025-01-09 07:03:24.000000000","message":"recheck","commit_id":"5d5b3c9d3c35f039061fffac93a74b7b2936ab95"},{"author":{"_account_id":34451,"name":"Fernando Royo","email":"froyo@redhat.com","username":"froyo"},"change_message_id":"0a9e5e88b31c6a36d84206d91c8d0c63b65ca590","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"95443046_9e679a84","updated":"2024-11-29 09:30:21.000000000","message":"recheck ovn-octavia-provider-tempest-release unrelated","commit_id":"5d5b3c9d3c35f039061fffac93a74b7b2936ab95"},{"author":{"_account_id":34451,"name":"Fernando Royo","email":"froyo@redhat.com","username":"froyo"},"change_message_id":"0cae82bd79774b098a617ef988e6219aa631862b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"bbbb644e_a81aef66","updated":"2024-11-28 17:15:02.000000000","message":"recheck ovn-octavia-provider-tempest-release unrelated","commit_id":"5d5b3c9d3c35f039061fffac93a74b7b2936ab95"},{"author":{"_account_id":34451,"name":"Fernando Royo","email":"froyo@redhat.com","username":"froyo"},"change_message_id":"aa909e2cd37d6e16bf1b779107fe7435e9e16793","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":23,"id":"514f8c3a_41cf919d","updated":"2025-02-18 20:38:54.000000000","message":"recheck after merging https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/941928","commit_id":"615bb010e261480c4a90945c4e2ead4f5788aef6"},{"author":{"_account_id":34451,"name":"Fernando Royo","email":"froyo@redhat.com","username":"froyo"},"change_message_id":"10a73074623babcca24451b9691fce7aef8f7a85","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":23,"id":"48bc05a5_51e5689a","updated":"2025-02-19 11:27:39.000000000","message":"recheck after tempest fix merged","commit_id":"615bb010e261480c4a90945c4e2ead4f5788aef6"}],"ovn_octavia_provider/helper.py":[{"author":{"_account_id":37089,"name":"Chris Buggy","display_name":"cbuggy","email":"cbuggy@redhat.com","username":"cbuggy"},"change_message_id":"5e5c9adab98a8a3785a5660ed9ce3b131e97d3ee","unresolved":true,"context_lines":[{"line_number":3239,"context_line":"                            self.ovn_nbdb_api.db_create("},{"line_number":3240,"context_line":"                                \u0027Load_Balancer_Health_Check\u0027,"},{"line_number":3241,"context_line":"                                **kwargs))"},{"line_number":3242,"context_line":"                        if recreate:"},{"line_number":3243,"context_line":"                            txn.add("},{"line_number":3244,"context_line":"                                self.ovn_nbdb_api.db_set("},{"line_number":3245,"context_line":"                                    \u0027Load_Balancer_Health_Check\u0027,"}],"source_content_type":"text/x-python","patch_set":11,"id":"bdddf000_91412431","line":3242,"updated":"2024-11-27 17:08:04.000000000","message":"Is this an indention error as it is already checked above?","commit_id":"385e153e83b776f355d4db3f32ac8b65a57ecff9"},{"author":{"_account_id":34451,"name":"Fernando Royo","email":"froyo@redhat.com","username":"froyo"},"change_message_id":"cff02b8cf4133ae18377a8a20eb8bacda9116430","unresolved":false,"context_lines":[{"line_number":3239,"context_line":"                            self.ovn_nbdb_api.db_create("},{"line_number":3240,"context_line":"                                \u0027Load_Balancer_Health_Check\u0027,"},{"line_number":3241,"context_line":"                                **kwargs))"},{"line_number":3242,"context_line":"                        if recreate:"},{"line_number":3243,"context_line":"                            txn.add("},{"line_number":3244,"context_line":"                                self.ovn_nbdb_api.db_set("},{"line_number":3245,"context_line":"                                    \u0027Load_Balancer_Health_Check\u0027,"}],"source_content_type":"text/x-python","patch_set":11,"id":"bd4ca638_447394bf","line":3242,"in_reply_to":"bdddf000_91412431","updated":"2024-11-28 14:57:01.000000000","message":"Done, good catch!","commit_id":"385e153e83b776f355d4db3f32ac8b65a57ecff9"},{"author":{"_account_id":37089,"name":"Chris Buggy","display_name":"cbuggy","email":"cbuggy@redhat.com","username":"cbuggy"},"change_message_id":"5e5c9adab98a8a3785a5660ed9ce3b131e97d3ee","unresolved":true,"context_lines":[{"line_number":3298,"context_line":"                                self.ovn_nbdb_api.db_create("},{"line_number":3299,"context_line":"                                    \u0027Load_Balancer_Health_Check\u0027,"},{"line_number":3300,"context_line":"                                    **fip_kwargs))"},{"line_number":3301,"context_line":"                            if recreate:"},{"line_number":3302,"context_line":"                                txn.add("},{"line_number":3303,"context_line":"                                    self.ovn_nbdb_api.db_set("},{"line_number":3304,"context_line":"                                        \u0027Load_Balancer_Health_Check\u0027,"}],"source_content_type":"text/x-python","patch_set":11,"id":"212bc559_8b6eeb36","line":3301,"updated":"2024-11-27 17:08:04.000000000","message":"Same line 3242","commit_id":"385e153e83b776f355d4db3f32ac8b65a57ecff9"},{"author":{"_account_id":34451,"name":"Fernando Royo","email":"froyo@redhat.com","username":"froyo"},"change_message_id":"cff02b8cf4133ae18377a8a20eb8bacda9116430","unresolved":false,"context_lines":[{"line_number":3298,"context_line":"                                self.ovn_nbdb_api.db_create("},{"line_number":3299,"context_line":"                                    \u0027Load_Balancer_Health_Check\u0027,"},{"line_number":3300,"context_line":"                                    **fip_kwargs))"},{"line_number":3301,"context_line":"                            if recreate:"},{"line_number":3302,"context_line":"                                txn.add("},{"line_number":3303,"context_line":"                                    self.ovn_nbdb_api.db_set("},{"line_number":3304,"context_line":"                                        \u0027Load_Balancer_Health_Check\u0027,"}],"source_content_type":"text/x-python","patch_set":11,"id":"9144c532_bfb4b36e","line":3301,"in_reply_to":"212bc559_8b6eeb36","updated":"2024-11-28 14:57:01.000000000","message":"Done","commit_id":"385e153e83b776f355d4db3f32ac8b65a57ecff9"},{"author":{"_account_id":37089,"name":"Chris Buggy","display_name":"cbuggy","email":"cbuggy@redhat.com","username":"cbuggy"},"change_message_id":"5e5c9adab98a8a3785a5660ed9ce3b131e97d3ee","unresolved":true,"context_lines":[{"line_number":3543,"context_line":"        \"\"\"Sync Health Monitor object with an OVN LoadBalancer"},{"line_number":3544,"context_line":""},{"line_number":3545,"context_line":"        The method performs the following steps:"},{"line_number":3546,"context_line":"        1. Create Health Monitor in OVN NB in case we don\u0027t found"},{"line_number":3547,"context_line":"        load_balancer_health_checks entries associated"},{"line_number":3548,"context_line":"        2. If we found load_balancer_health_checks entries associated"},{"line_number":3549,"context_line":"            2.1. Update member status affected on OVN loadbalancer external_ids"}],"source_content_type":"text/x-python","patch_set":11,"id":"ccd3b96e_f40b99e6","line":3546,"updated":"2024-11-27 17:08:04.000000000","message":"Nit: found should be find in the comments","commit_id":"385e153e83b776f355d4db3f32ac8b65a57ecff9"},{"author":{"_account_id":34451,"name":"Fernando Royo","email":"froyo@redhat.com","username":"froyo"},"change_message_id":"cff02b8cf4133ae18377a8a20eb8bacda9116430","unresolved":false,"context_lines":[{"line_number":3543,"context_line":"        \"\"\"Sync Health Monitor object with an OVN LoadBalancer"},{"line_number":3544,"context_line":""},{"line_number":3545,"context_line":"        The method performs the following steps:"},{"line_number":3546,"context_line":"        1. Create Health Monitor in OVN NB in case we don\u0027t found"},{"line_number":3547,"context_line":"        load_balancer_health_checks entries associated"},{"line_number":3548,"context_line":"        2. If we found load_balancer_health_checks entries associated"},{"line_number":3549,"context_line":"            2.1. Update member status affected on OVN loadbalancer external_ids"}],"source_content_type":"text/x-python","patch_set":11,"id":"8973f7b1_f7211c7d","line":3546,"in_reply_to":"ccd3b96e_f40b99e6","updated":"2024-11-28 14:57:01.000000000","message":"Done","commit_id":"385e153e83b776f355d4db3f32ac8b65a57ecff9"}]}