)]}'
{"networking_ovn/ml2/mech_driver.py":[{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"80c95035148de2486b91574bbfed6a4572c64639","unresolved":false,"context_lines":[{"line_number":283,"context_line":"        \"\"\""},{"line_number":284,"context_line":"        network \u003d context.current"},{"line_number":285,"context_line":"        original_network \u003d context.original"},{"line_number":286,"context_line":"        self._ovn_client.update_network(network, original_network)"},{"line_number":287,"context_line":""},{"line_number":288,"context_line":"    def delete_network_postcommit(self, context):"},{"line_number":289,"context_line":"        \"\"\"Delete a network."}],"source_content_type":"text/x-python","patch_set":1,"id":"1f485f77_ac5f020d","line":286,"updated":"2017-11-09 14:22:26.000000000","message":"I agree with this patch but just to point out here:\n\nThis check is redundant, it\u0027s already being performed by the update_network() in OVNClient [0]\n\nThe QOS driver will also ignore the update if it\u0027s not enabled or if anything changes [1]\n\n[0] https://github.com/openstack/networking-ovn/blob/20fdec3e7ff6228b6184db4c8c9e4f173294bfd7/networking_ovn/common/ovn_client.py#L873\n\n[1] https://github.com/openstack/networking-ovn/blob/20fdec3e7ff6228b6184db4c8c9e4f173294bfd7/networking_ovn/ml2/qos_driver.py#L163-L164","commit_id":"d73f5c19f4e901403597709845a335f4ce82e234"}]}