)]}'
{"neutron/plugins/ml2/drivers/linuxbridge/agent/arp_protect.py":[{"author":{"_account_id":5948,"name":"Oleg Bondarev","email":"obondarev@mirantis.com","username":"obondarev"},"change_message_id":"eb8f98bc7a5be397d971bae5dfe32ed676796c5e","unresolved":false,"context_lines":[{"line_number":88,"context_line":"                  chain_name(vif), \u0027-p\u0027, \u0027ARP\u0027], table\u003dtable)"},{"line_number":89,"context_line":"    for vif in vifs:"},{"line_number":90,"context_line":"        if chain_exists(chain_name(vif), current_rules):"},{"line_number":91,"context_line":"            ebtables([\u0027-F\u0027, chain_name(vif)], table\u003dtable)"},{"line_number":92,"context_line":"            ebtables([\u0027-X\u0027, chain_name(vif)], table\u003dtable)"},{"line_number":93,"context_line":"    _delete_mac_spoofing_protection(vifs, current_rules, table\u003dtable,"},{"line_number":94,"context_line":"                                    chain\u003dchain)"},{"line_number":95,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"9f560f44_0d46a124","line":92,"range":{"start_line":91,"start_character":12,"end_line":92,"end_character":58},"updated":"2020-07-28 05:41:57.000000000","message":"since needed in more than 1 place, I\u0027d go with a separate method for this","commit_id":"5c05c5a9f8f3e4d199b819b31160f1879edf68aa"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"1cfc5246848f9fcbf2dec64c5cc8ca7ae6f7c001","unresolved":false,"context_lines":[{"line_number":88,"context_line":"                  chain_name(vif), \u0027-p\u0027, \u0027ARP\u0027], table\u003dtable)"},{"line_number":89,"context_line":"    for vif in vifs:"},{"line_number":90,"context_line":"        if chain_exists(chain_name(vif), current_rules):"},{"line_number":91,"context_line":"            ebtables([\u0027-F\u0027, chain_name(vif)], table\u003dtable)"},{"line_number":92,"context_line":"            ebtables([\u0027-X\u0027, chain_name(vif)], table\u003dtable)"},{"line_number":93,"context_line":"    _delete_mac_spoofing_protection(vifs, current_rules, table\u003dtable,"},{"line_number":94,"context_line":"                                    chain\u003dchain)"},{"line_number":95,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"9f560f44_1c21b0b5","line":92,"range":{"start_line":91,"start_character":12,"end_line":92,"end_character":58},"in_reply_to":"9f560f44_0d46a124","updated":"2020-07-28 09:21:20.000000000","message":"I agree. L90-L92 and L217-L219 can be moved to separate method and reused in both places.","commit_id":"5c05c5a9f8f3e4d199b819b31160f1879edf68aa"}]}