)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"c91910d718b46f2f02d9a765388740eeda93d4e6","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"ovsfw: Update SG rules even if OVSFW Port is not found"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"In ovs firewall driver update_port_filter() method was not"},{"line_number":10,"context_line":"trying to initialize port flows in case when OVSFWPortNotFound"},{"line_number":11,"context_line":"is raised."},{"line_number":12,"context_line":"Without that when e.g. instance is hard rebooted and of_port"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3fa0c359_cf4d119b","line":9,"updated":"2018-02-09 06:11:39.000000000","message":"ok would be nice to have a link here to the patch that broke it","commit_id":"5b7d92cc0a2df3cdc9bd3eaac75bc27762892c7b"}],"neutron/agent/linux/openvswitch_firewall/firewall.py":[{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"7cda43e8899021752226d1218ba9489a2b133ce0","unresolved":false,"context_lines":[{"line_number":539,"context_line":"                      \u0027err\u0027: not_found_error})"},{"line_number":540,"context_line":"            return"},{"line_number":541,"context_line":"        # TODO(jlibosva): Handle firewall blink"},{"line_number":542,"context_line":"        if old_of_port:"},{"line_number":543,"context_line":"            self.delete_all_port_flows(old_of_port)"},{"line_number":544,"context_line":"        self.initialize_port_flows(of_port)"},{"line_number":545,"context_line":"        self.add_flows_from_rules(of_port)"}],"source_content_type":"text/x-python","patch_set":4,"id":"3fa0c359_08cafa9f","line":542,"updated":"2018-02-09 14:38:15.000000000","message":"not sure if that\u0027s applicable, but almost the same code as here is present in prepare_port_filter. it almost like we could reuse the code in both places.","commit_id":"6815316904f2753dee05e863440014205ebdbbac"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"5e29fb711148751f1d8a28079592d0b950e55116","unresolved":false,"context_lines":[{"line_number":539,"context_line":"                      \u0027err\u0027: not_found_error})"},{"line_number":540,"context_line":"            return"},{"line_number":541,"context_line":"        # TODO(jlibosva): Handle firewall blink"},{"line_number":542,"context_line":"        if old_of_port:"},{"line_number":543,"context_line":"            self.delete_all_port_flows(old_of_port)"},{"line_number":544,"context_line":"        self.initialize_port_flows(of_port)"},{"line_number":545,"context_line":"        self.add_flows_from_rules(of_port)"}],"source_content_type":"text/x-python","patch_set":4,"id":"3fa0c359_19200348","line":542,"in_reply_to":"3fa0c359_08cafa9f","updated":"2018-02-09 21:59:19.000000000","message":"I changed this code to reuse same parts in both places","commit_id":"6815316904f2753dee05e863440014205ebdbbac"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"8b75200004b4446ef1dad48e6a6ff601e7f12531","unresolved":false,"context_lines":[{"line_number":532,"context_line":"        # allowed_address_pair MACs will be updated in"},{"line_number":533,"context_line":"        # self.get_or_create_ofport(port)"},{"line_number":534,"context_line":"        if old_of_port:"},{"line_number":535,"context_line":"            LOG.info(\"Initializing port %s that was already \""},{"line_number":536,"context_line":"                     \"initialized.\","},{"line_number":537,"context_line":"                     port[\u0027device\u0027])"},{"line_number":538,"context_line":"            self.delete_all_port_flows(old_of_port)"}],"source_content_type":"text/x-python","patch_set":5,"id":"3fa0c359_1d29b6e4","line":535,"updated":"2018-02-12 11:06:24.000000000","message":"Now we could get a false message that we\u0027re initializing a port while it received a proper update.","commit_id":"492a9d21f97d5e9fe4ed1915b2468b24def3c2c8"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"71cfe86da6d7a461e927b21e67328467b9bc779f","unresolved":false,"context_lines":[{"line_number":532,"context_line":"        # allowed_address_pair MACs will be updated in"},{"line_number":533,"context_line":"        # self.get_or_create_ofport(port)"},{"line_number":534,"context_line":"        if old_of_port:"},{"line_number":535,"context_line":"            LOG.info(\"Initializing port %s that was already \""},{"line_number":536,"context_line":"                     \"initialized.\","},{"line_number":537,"context_line":"                     port[\u0027device\u0027])"},{"line_number":538,"context_line":"            self.delete_all_port_flows(old_of_port)"}],"source_content_type":"text/x-python","patch_set":5,"id":"3fa0c359_7da1da45","line":535,"in_reply_to":"3fa0c359_1d29b6e4","updated":"2018-02-12 11:15:51.000000000","message":"right, I will change it right now","commit_id":"492a9d21f97d5e9fe4ed1915b2468b24def3c2c8"}]}
