)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"79ceadd023e32dd057bf1114759e6106cc6e4575","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"2fa3bddd_fa61e71e","updated":"2022-02-09 17:23:01.000000000","message":"I think Don\u0027s change here was in error. It probably should be reverted.","commit_id":"727cf365b5bbceddebc18a10af3cfe883408137b"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"5546e3a2e2682c82b300c1c6a495084edba3009c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"94e0d64d_28613dc3","updated":"2022-02-09 17:28:06.000000000","message":"LGTM\nThis is a minor commit message issue.","commit_id":"727cf365b5bbceddebc18a10af3cfe883408137b"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"98d3f4c7e0c4ddde646ba469f7fbfac3d017e74a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"9fcad25c_7a6c8073","updated":"2022-04-14 22:52:51.000000000","message":"LGTM","commit_id":"b4806a83b3fa142d1fbd522add11f453199f2a77"},{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"983ecd11e4e1fe4e68d3279a82aac7f2b6abd598","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"ab3d5db6_bdfa15d9","updated":"2022-04-27 13:05:27.000000000","message":"recheck git update issue was resolved","commit_id":"b4806a83b3fa142d1fbd522add11f453199f2a77"}],"octavia/controller/worker/v2/controller_worker.py":[{"author":{"_account_id":34429,"name":"Tom Weininger","email":"dienste@weinimo.de","username":"tweining"},"change_message_id":"c3b0864af8cf26e197424dfe1b0c811b1c2f97f3","unresolved":true,"context_lines":[{"line_number":968,"context_line":"                             constants.LOADBALANCER_ID: lb_id,"},{"line_number":969,"context_line":"                             constants.VIP: vip_dict}"},{"line_number":970,"context_line":""},{"line_number":971,"context_line":"            self.run_flow("},{"line_number":972,"context_line":"                flow_utils.get_failover_amphora_flow,"},{"line_number":973,"context_line":"                amphora.to_dict(), lb_amp_count,"},{"line_number":974,"context_line":"                store\u003dstored_params)"},{"line_number":975,"context_line":""},{"line_number":976,"context_line":"            LOG.info(\"Successfully completed the failover for an amphora: %s\","},{"line_number":977,"context_line":"                     {\"id\": amphora_id,"}],"source_content_type":"text/x-python","patch_set":2,"id":"d6a8bba3_b6410e6a","line":974,"range":{"start_line":971,"start_character":12,"end_line":974,"end_character":36},"updated":"2022-02-17 11:12:44.000000000","message":"I wonder why you updated these particular run_flow() calls, but not the other ones that use the wait kwarg. Is it because the Taskflow engine may still use it?","commit_id":"727cf365b5bbceddebc18a10af3cfe883408137b"},{"author":{"_account_id":34429,"name":"Tom Weininger","email":"dienste@weinimo.de","username":"tweining"},"change_message_id":"ffda74cf8b10290bf7e27b54945310dddb96f08f","unresolved":true,"context_lines":[{"line_number":968,"context_line":"                             constants.LOADBALANCER_ID: lb_id,"},{"line_number":969,"context_line":"                             constants.VIP: vip_dict}"},{"line_number":970,"context_line":""},{"line_number":971,"context_line":"            self.run_flow("},{"line_number":972,"context_line":"                flow_utils.get_failover_amphora_flow,"},{"line_number":973,"context_line":"                amphora.to_dict(), lb_amp_count,"},{"line_number":974,"context_line":"                store\u003dstored_params)"},{"line_number":975,"context_line":""},{"line_number":976,"context_line":"            LOG.info(\"Successfully completed the failover for an amphora: %s\","},{"line_number":977,"context_line":"                     {\"id\": amphora_id,"}],"source_content_type":"text/x-python","patch_set":2,"id":"81fcd943_b43dc9cc","line":974,"range":{"start_line":971,"start_character":12,"end_line":974,"end_character":36},"in_reply_to":"6b4b1c12_610b51df","updated":"2022-02-17 12:13:55.000000000","message":"Yes, you\u0027re right. It\u0027s only in line 129, it seems.","commit_id":"727cf365b5bbceddebc18a10af3cfe883408137b"},{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"7a0387488ef9ea8ac5e0745123f6c2b4153b5e5a","unresolved":false,"context_lines":[{"line_number":968,"context_line":"                             constants.LOADBALANCER_ID: lb_id,"},{"line_number":969,"context_line":"                             constants.VIP: vip_dict}"},{"line_number":970,"context_line":""},{"line_number":971,"context_line":"            self.run_flow("},{"line_number":972,"context_line":"                flow_utils.get_failover_amphora_flow,"},{"line_number":973,"context_line":"                amphora.to_dict(), lb_amp_count,"},{"line_number":974,"context_line":"                store\u003dstored_params)"},{"line_number":975,"context_line":""},{"line_number":976,"context_line":"            LOG.info(\"Successfully completed the failover for an amphora: %s\","},{"line_number":977,"context_line":"                     {\"id\": amphora_id,"}],"source_content_type":"text/x-python","patch_set":2,"id":"ab70ad43_8fd07ff0","line":974,"range":{"start_line":971,"start_character":12,"end_line":974,"end_character":36},"in_reply_to":"81fcd943_b43dc9cc","updated":"2022-02-21 16:18:07.000000000","message":"Done","commit_id":"727cf365b5bbceddebc18a10af3cfe883408137b"},{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"9786aad5abe3491dfa41c9f19e06099c4ab0f822","unresolved":true,"context_lines":[{"line_number":968,"context_line":"                             constants.LOADBALANCER_ID: lb_id,"},{"line_number":969,"context_line":"                             constants.VIP: vip_dict}"},{"line_number":970,"context_line":""},{"line_number":971,"context_line":"            self.run_flow("},{"line_number":972,"context_line":"                flow_utils.get_failover_amphora_flow,"},{"line_number":973,"context_line":"                amphora.to_dict(), lb_amp_count,"},{"line_number":974,"context_line":"                store\u003dstored_params)"},{"line_number":975,"context_line":""},{"line_number":976,"context_line":"            LOG.info(\"Successfully completed the failover for an amphora: %s\","},{"line_number":977,"context_line":"                     {\"id\": amphora_id,"}],"source_content_type":"text/x-python","patch_set":2,"id":"6b4b1c12_610b51df","line":974,"range":{"start_line":971,"start_character":12,"end_line":974,"end_character":36},"in_reply_to":"d6a8bba3_b6410e6a","updated":"2022-02-17 11:44:54.000000000","message":"are you talking about line 129?\nThis is a really great question! I missed the call line 129 because this function no longer exists on master, so the patch doesn\u0027t update it.\nThanks for reporting it, I will update the patches on ussuri, victoria and wallaby (xena is not affected)","commit_id":"727cf365b5bbceddebc18a10af3cfe883408137b"}]}