)]}'
{"dragonflow/controller/common/arp_responder.py":[{"author":{"_account_id":7805,"name":"Li Ma","email":"skywalker.nick@gmail.com","username":"Nick"},"change_message_id":"8793b76b82e4753f43fd90b75d31aa3c4063dacb","unresolved":false,"context_lines":[{"line_number":29,"context_line":"            of remove."},{"line_number":30,"context_line":"    \"\"\""},{"line_number":31,"context_line":"    def __init__(self, datapath, network_id, interface_ip,"},{"line_number":32,"context_line":"                 interface_mac\u003dNone, table_id\u003dconst.ARP_TABLE):"},{"line_number":33,"context_line":"        self.datapath \u003d datapath"},{"line_number":34,"context_line":"        self.network_id \u003d network_id"},{"line_number":35,"context_line":"        self.interface_ip \u003d interface_ip"}],"source_content_type":"text/x-python","patch_set":1,"id":"5aef4532_a60ff52c","line":32,"range":{"start_line":32,"start_character":36,"end_line":32,"end_character":61},"updated":"2016-03-09 07:13:04.000000000","message":"Why add table_id variable?\nWill the ArpResponder use other table, rather than ARP_TABLE?","commit_id":"22c3485a477a87f65cd7cf94213c7cd4c8458587"},{"author":{"_account_id":7805,"name":"Li Ma","email":"skywalker.nick@gmail.com","username":"Nick"},"change_message_id":"e10346c7906b5a97210b36c37cf472bdcb090561","unresolved":false,"context_lines":[{"line_number":29,"context_line":"            of remove."},{"line_number":30,"context_line":"    \"\"\""},{"line_number":31,"context_line":"    def __init__(self, datapath, network_id, interface_ip,"},{"line_number":32,"context_line":"                 interface_mac\u003dNone, table_id\u003dconst.ARP_TABLE):"},{"line_number":33,"context_line":"        self.datapath \u003d datapath"},{"line_number":34,"context_line":"        self.network_id \u003d network_id"},{"line_number":35,"context_line":"        self.interface_ip \u003d interface_ip"}],"source_content_type":"text/x-python","patch_set":1,"id":"5aef4532_b2ca0619","line":32,"range":{"start_line":32,"start_character":36,"end_line":32,"end_character":61},"in_reply_to":"5aef4532_56eb3513","updated":"2016-03-11 09:24:45.000000000","message":"Thanks for your explanation, Fei Rao.","commit_id":"22c3485a477a87f65cd7cf94213c7cd4c8458587"},{"author":{"_account_id":11364,"name":"Fei Rao","email":"milo.frao@gmail.com","username":"milo.frao"},"change_message_id":"6d10f6ed51356c0960cc621e276f7c71675c12df","unresolved":false,"context_lines":[{"line_number":29,"context_line":"            of remove."},{"line_number":30,"context_line":"    \"\"\""},{"line_number":31,"context_line":"    def __init__(self, datapath, network_id, interface_ip,"},{"line_number":32,"context_line":"                 interface_mac\u003dNone, table_id\u003dconst.ARP_TABLE):"},{"line_number":33,"context_line":"        self.datapath \u003d datapath"},{"line_number":34,"context_line":"        self.network_id \u003d network_id"},{"line_number":35,"context_line":"        self.interface_ip \u003d interface_ip"}],"source_content_type":"text/x-python","patch_set":1,"id":"5aef4532_56eb3513","line":32,"range":{"start_line":32,"start_character":36,"end_line":32,"end_character":61},"in_reply_to":"5aef4532_a60ff52c","updated":"2016-03-09 11:24:40.000000000","message":"Currently the ARP_TABLE is for vm\u0027s ip arp responder, every vm will install this flow rule. But for fip, only some vm has itself fip, if we also use the same table to process fip arp respond, it\u0027s not clear. it\u0027s better to use other relative table to do it. Using INGRESS_NAT table for fip arp responder is unambiguous.","commit_id":"22c3485a477a87f65cd7cf94213c7cd4c8458587"}]}