)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":34451,"name":"Fernando Royo","email":"froyo@redhat.com","username":"froyo"},"change_message_id":"737136a0b9910b6e0caf6d6bc1cdfc8c1c6bd077","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"1ecad357_62fccdca","updated":"2022-05-31 10:07:19.000000000","message":"recheck functional-master timeout not related","commit_id":"28d378868fc6ad005f82ef96e573766de68e4d37"}],"ovn_octavia_provider/helper.py":[{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"d62265d3bf857e724700476236ceea775e390393","unresolved":true,"context_lines":[{"line_number":937,"context_line":"                \u0027external_ids\u0027: external_ids}"},{"line_number":938,"context_line":"            if self._are_selection_fields_supported():"},{"line_number":939,"context_line":"                kwargs[\u0027selection_fields\u0027] \u003d self._get_selection_keys("},{"line_number":940,"context_line":"                    lb_algorithm)"},{"line_number":941,"context_line":""},{"line_number":942,"context_line":"            self.ovn_nbdb_api.db_create("},{"line_number":943,"context_line":"                \u0027Load_Balancer\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"c3376f8c_b42985ac","line":940,"updated":"2022-05-27 08:54:15.000000000","message":"do we really need all of that in the same try..except.. block? It\u0027s huge and very generic. Maybe You could have new try..except block here and do some better logging. Wdyt?","commit_id":"98ed549aaf44c38a914595c608a9ce12d1e24fac"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"9014ff7ee3db7104d9745197d1c130a00ef6061b","unresolved":false,"context_lines":[{"line_number":937,"context_line":"                \u0027external_ids\u0027: external_ids}"},{"line_number":938,"context_line":"            if self._are_selection_fields_supported():"},{"line_number":939,"context_line":"                kwargs[\u0027selection_fields\u0027] \u003d self._get_selection_keys("},{"line_number":940,"context_line":"                    lb_algorithm)"},{"line_number":941,"context_line":""},{"line_number":942,"context_line":"            self.ovn_nbdb_api.db_create("},{"line_number":943,"context_line":"                \u0027Load_Balancer\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"008f9fc1_7c68f05e","line":940,"in_reply_to":"6c5c3983_dab9817a","updated":"2022-05-27 12:53:56.000000000","message":"Thx, it\u0027s better now IMO :)","commit_id":"98ed549aaf44c38a914595c608a9ce12d1e24fac"},{"author":{"_account_id":34451,"name":"Fernando Royo","email":"froyo@redhat.com","username":"froyo"},"change_message_id":"dadfaa41c69949adf2f196fdb07072cb7ec2ee27","unresolved":true,"context_lines":[{"line_number":937,"context_line":"                \u0027external_ids\u0027: external_ids}"},{"line_number":938,"context_line":"            if self._are_selection_fields_supported():"},{"line_number":939,"context_line":"                kwargs[\u0027selection_fields\u0027] \u003d self._get_selection_keys("},{"line_number":940,"context_line":"                    lb_algorithm)"},{"line_number":941,"context_line":""},{"line_number":942,"context_line":"            self.ovn_nbdb_api.db_create("},{"line_number":943,"context_line":"                \u0027Load_Balancer\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"6c5c3983_dab9817a","line":940,"in_reply_to":"c3376f8c_b42985ac","updated":"2022-05-27 10:19:28.000000000","message":"yeah, it just added to reuse the delete_vip_port code in general exception block, but lets cover just the neutron_client section.","commit_id":"98ed549aaf44c38a914595c608a9ce12d1e24fac"}]}