)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"39323c64efc7f37a7df828ea32cc9f593b980cff","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     hailun.huang \u003cts_1749@163.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2021-12-20 17:47:26 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix bug about reconnect subnet not learn mac"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"If router_A connect subnet by ip \u003cx.x.x.x\u003e and"},{"line_number":10,"context_line":" there are packets through it when always_learn_from_arp_request"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"076b875c_c479208a","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":44},"updated":"2021-12-20 10:18:39.000000000","message":"There is not need to say this is a fix. Describe the patch. For example: \"Delete MAC binding for LRP when the port is deleted\"","commit_id":"e176ca0384be900c5c1932e19ebe2ae9e1f4e512"},{"author":{"_account_id":34125,"name":"huanghailun","display_name":"huanghailun","email":"hailun.huang@easystack.cn","username":"lnun"},"change_message_id":"1a3786924bd156bd7707f8ac6516be6d1a6a03cb","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     hailun.huang \u003cts_1749@163.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2021-12-20 17:47:26 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix bug about reconnect subnet not learn mac"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"If router_A connect subnet by ip \u003cx.x.x.x\u003e and"},{"line_number":10,"context_line":" there are packets through it when always_learn_from_arp_request"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"786a32a4_ba0d3007","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":44},"in_reply_to":"076b875c_c479208a","updated":"2021-12-20 11:10:08.000000000","message":"Done","commit_id":"e176ca0384be900c5c1932e19ebe2ae9e1f4e512"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"39323c64efc7f37a7df828ea32cc9f593b980cff","unresolved":true,"context_lines":[{"line_number":7,"context_line":"Fix bug about reconnect subnet not learn mac"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"If router_A connect subnet by ip \u003cx.x.x.x\u003e and"},{"line_number":10,"context_line":" there are packets through it when always_learn_from_arp_request"},{"line_number":11,"context_line":" is set to False(patch[1]), the mac_binding table of ovn south"},{"line_number":12,"context_line":" db will learn mac address of \u003cx.x.x.x\u003e."},{"line_number":13,"context_line":"If disconnect the subnet, the mac of \u003cx.x.x.x\u003e still"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3a58b4a8_9ad4345c","line":10,"range":{"start_line":10,"start_character":0,"end_line":10,"end_character":1},"updated":"2021-12-20 10:18:39.000000000","message":"Why these spaces at the beginning?","commit_id":"e176ca0384be900c5c1932e19ebe2ae9e1f4e512"},{"author":{"_account_id":34125,"name":"huanghailun","display_name":"huanghailun","email":"hailun.huang@easystack.cn","username":"lnun"},"change_message_id":"1a3786924bd156bd7707f8ac6516be6d1a6a03cb","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Fix bug about reconnect subnet not learn mac"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"If router_A connect subnet by ip \u003cx.x.x.x\u003e and"},{"line_number":10,"context_line":" there are packets through it when always_learn_from_arp_request"},{"line_number":11,"context_line":" is set to False(patch[1]), the mac_binding table of ovn south"},{"line_number":12,"context_line":" db will learn mac address of \u003cx.x.x.x\u003e."},{"line_number":13,"context_line":"If disconnect the subnet, the mac of \u003cx.x.x.x\u003e still"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"6b04b555_94c5afb6","line":10,"range":{"start_line":10,"start_character":0,"end_line":10,"end_character":1},"in_reply_to":"3a58b4a8_9ad4345c","updated":"2021-12-20 11:10:08.000000000","message":"Done","commit_id":"e176ca0384be900c5c1932e19ebe2ae9e1f4e512"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"39323c64efc7f37a7df828ea32cc9f593b980cff","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"9932151f_0f32470d","updated":"2021-12-20 10:18:39.000000000","message":"Hello hhl, as commented by Miguel in the LP bug, can you provide some steps to reproduce this issue? ","commit_id":"e176ca0384be900c5c1932e19ebe2ae9e1f4e512"},{"author":{"_account_id":34125,"name":"huanghailun","display_name":"huanghailun","email":"hailun.huang@easystack.cn","username":"lnun"},"change_message_id":"1a3786924bd156bd7707f8ac6516be6d1a6a03cb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"9eb6d094_55a5e745","in_reply_to":"9932151f_0f32470d","updated":"2021-12-20 11:10:08.000000000","message":"Ok, I will add it to the bug comments.","commit_id":"e176ca0384be900c5c1932e19ebe2ae9e1f4e512"},{"author":{"_account_id":28056,"name":"Liushy","email":"liuxie_11@163.com","username":"liuxie_11"},"change_message_id":"3870e13c48c12cf5dd4a8b95bef82b538bff2868","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"687a6e17_d0e67191","updated":"2021-12-20 11:56:42.000000000","message":"Nice patch!!","commit_id":"3386133c4d195d501b2d44bbc96757f6bdca34b5"},{"author":{"_account_id":34125,"name":"huanghailun","display_name":"huanghailun","email":"hailun.huang@easystack.cn","username":"lnun"},"change_message_id":"7975024e451b5df6a995b04d468224a0280bb233","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"18fc7af7_6a03c66a","updated":"2021-12-21 03:22:19.000000000","message":"recheck","commit_id":"3386133c4d195d501b2d44bbc96757f6bdca34b5"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"c16cbb3d8294cc22ae4bbece3582f6f127669a7c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"447de7ce_bed24d61","updated":"2022-01-07 07:47:26.000000000","message":"Looks good, just a couple of nits","commit_id":"2007f7c8bc3a76722ad3ca7fd3e4be1451de6611"},{"author":{"_account_id":34125,"name":"huanghailun","display_name":"huanghailun","email":"hailun.huang@easystack.cn","username":"lnun"},"change_message_id":"d327ff61e76c1d715703392b35ae5677d28bdb1c","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":4,"id":"1d84b736_80d49a21","updated":"2022-01-05 02:39:18.000000000","message":"recheck","commit_id":"2007f7c8bc3a76722ad3ca7fd3e4be1451de6611"},{"author":{"_account_id":34125,"name":"huanghailun","display_name":"huanghailun","email":"hailun.huang@easystack.cn","username":"lnun"},"change_message_id":"015f217e63f16437b1735664fab4b8fd41d3ab81","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"094431bd_a043eec8","updated":"2022-01-11 09:48:05.000000000","message":"recheck","commit_id":"f2e8a619c5ce7a4b4713f40c6da68da1a3e4410c"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"9a8a44005ee67e07d3b70dc41fb050112e1dab13","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"ad8cb4f1_06944b29","updated":"2022-01-11 13:52:26.000000000","message":"recheck - bug 1956344 which shouldn\u0027t be a problem anymore","commit_id":"f2e8a619c5ce7a4b4713f40c6da68da1a3e4410c"}],"neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/ovn_client.py":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"39323c64efc7f37a7df828ea32cc9f593b980cff","unresolved":true,"context_lines":[{"line_number":1564,"context_line":"                context, port, ovn_const.TYPE_ROUTER_PORTS)"},{"line_number":1565,"context_line":""},{"line_number":1566,"context_line":"    @tenacity.retry(wait\u003dtenacity.wait_random(min\u003d2, max\u003d3),"},{"line_number":1567,"context_line":"                    stop\u003dtenacity.stop_after_attempt(100))"},{"line_number":1568,"context_line":"    def delete_mac_binding_entries_by_mac(self, mac):"},{"line_number":1569,"context_line":"        \"\"\"Delete all MAC_Binding entries associated to this IP/mac address\"\"\""},{"line_number":1570,"context_line":"        cmd \u003d [\u0027ovsdb-client\u0027, \u0027transact\u0027, ovn_conf.get_ovn_sb_connection(),"}],"source_content_type":"text/x-python","patch_set":1,"id":"4c8f4f13_aae8b3f5","line":1567,"range":{"start_line":1567,"start_character":53,"end_line":1567,"end_character":56},"updated":"2021-12-20 10:18:39.000000000","message":"We could be waiting up to 300 seconds for this? No way...","commit_id":"e176ca0384be900c5c1932e19ebe2ae9e1f4e512"},{"author":{"_account_id":34125,"name":"huanghailun","display_name":"huanghailun","email":"hailun.huang@easystack.cn","username":"lnun"},"change_message_id":"1a3786924bd156bd7707f8ac6516be6d1a6a03cb","unresolved":false,"context_lines":[{"line_number":1564,"context_line":"                context, port, ovn_const.TYPE_ROUTER_PORTS)"},{"line_number":1565,"context_line":""},{"line_number":1566,"context_line":"    @tenacity.retry(wait\u003dtenacity.wait_random(min\u003d2, max\u003d3),"},{"line_number":1567,"context_line":"                    stop\u003dtenacity.stop_after_attempt(100))"},{"line_number":1568,"context_line":"    def delete_mac_binding_entries_by_mac(self, mac):"},{"line_number":1569,"context_line":"        \"\"\"Delete all MAC_Binding entries associated to this IP/mac address\"\"\""},{"line_number":1570,"context_line":"        cmd \u003d [\u0027ovsdb-client\u0027, \u0027transact\u0027, ovn_conf.get_ovn_sb_connection(),"}],"source_content_type":"text/x-python","patch_set":1,"id":"d8eec5dc_fda63672","line":1567,"range":{"start_line":1567,"start_character":53,"end_line":1567,"end_character":56},"in_reply_to":"4c8f4f13_aae8b3f5","updated":"2021-12-20 11:10:08.000000000","message":"Done","commit_id":"e176ca0384be900c5c1932e19ebe2ae9e1f4e512"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"a48ee8efd991ed27b3432e3541f81d0ee72dacdc","unresolved":true,"context_lines":[{"line_number":1567,"context_line":"                    stop\u003dtenacity.stop_after_attempt(3))"},{"line_number":1568,"context_line":"    def delete_mac_binding_entries_by_mac(self, mac):"},{"line_number":1569,"context_line":"        \"\"\"Delete all MAC_Binding entries associated to this IP/mac address\"\"\""},{"line_number":1570,"context_line":"        cmd \u003d [\u0027ovsdb-client\u0027, \u0027transact\u0027, ovn_conf.get_ovn_sb_connection(),"},{"line_number":1571,"context_line":"               \u0027--timeout\u0027, str(ovn_conf.get_ovn_ovsdb_timeout())]"},{"line_number":1572,"context_line":"        if ovn_conf.get_ovn_sb_private_key():"},{"line_number":1573,"context_line":"            cmd +\u003d [\u0027-p\u0027, ovn_conf.get_ovn_sb_private_key(), \u0027-c\u0027,"},{"line_number":1574,"context_line":"                    ovn_conf.get_ovn_sb_certificate(), \u0027-C\u0027,"},{"line_number":1575,"context_line":"                    ovn_conf.get_ovn_sb_ca_cert()]"},{"line_number":1576,"context_line":"        cmd +\u003d [\u0027[\"OVN_Southbound\", {\"op\": \"delete\", \"table\": \"MAC_Binding\", \u0027"},{"line_number":1577,"context_line":"                \u0027\"where\": [[\"mac\", \"\u003d\u003d\", \"%s\"]]}]\u0027 % mac]"},{"line_number":1578,"context_line":"        return processutils.execute(*cmd,"},{"line_number":1579,"context_line":"                                    log_errors\u003dprocessutils.LOG_FINAL_ERROR)"},{"line_number":1580,"context_line":""},{"line_number":1581,"context_line":"    def _delete_lrouter_port(self, context, port_id, router_id\u003dNone, txn\u003dNone):"},{"line_number":1582,"context_line":"        \"\"\"Delete a logical router port.\"\"\""}],"source_content_type":"text/x-python","patch_set":4,"id":"ee709d94_7c0088ce","line":1579,"range":{"start_line":1570,"start_character":8,"end_line":1579,"end_character":76},"updated":"2022-01-05 11:06:21.000000000","message":"this shouldn\u0027t be CLI, but though ovsdb as I see:\nhttps://opendev.org/openstack/neutron/src/branch/master/neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/impl_idl_ovn.py#L811","commit_id":"2007f7c8bc3a76722ad3ca7fd3e4be1451de6611"},{"author":{"_account_id":34125,"name":"huanghailun","display_name":"huanghailun","email":"hailun.huang@easystack.cn","username":"lnun"},"change_message_id":"bf01755c4bad284600b97af3e00b1cec7f693088","unresolved":false,"context_lines":[{"line_number":1567,"context_line":"                    stop\u003dtenacity.stop_after_attempt(3))"},{"line_number":1568,"context_line":"    def delete_mac_binding_entries_by_mac(self, mac):"},{"line_number":1569,"context_line":"        \"\"\"Delete all MAC_Binding entries associated to this IP/mac address\"\"\""},{"line_number":1570,"context_line":"        cmd \u003d [\u0027ovsdb-client\u0027, \u0027transact\u0027, ovn_conf.get_ovn_sb_connection(),"},{"line_number":1571,"context_line":"               \u0027--timeout\u0027, str(ovn_conf.get_ovn_ovsdb_timeout())]"},{"line_number":1572,"context_line":"        if ovn_conf.get_ovn_sb_private_key():"},{"line_number":1573,"context_line":"            cmd +\u003d [\u0027-p\u0027, ovn_conf.get_ovn_sb_private_key(), \u0027-c\u0027,"},{"line_number":1574,"context_line":"                    ovn_conf.get_ovn_sb_certificate(), \u0027-C\u0027,"},{"line_number":1575,"context_line":"                    ovn_conf.get_ovn_sb_ca_cert()]"},{"line_number":1576,"context_line":"        cmd +\u003d [\u0027[\"OVN_Southbound\", {\"op\": \"delete\", \"table\": \"MAC_Binding\", \u0027"},{"line_number":1577,"context_line":"                \u0027\"where\": [[\"mac\", \"\u003d\u003d\", \"%s\"]]}]\u0027 % mac]"},{"line_number":1578,"context_line":"        return processutils.execute(*cmd,"},{"line_number":1579,"context_line":"                                    log_errors\u003dprocessutils.LOG_FINAL_ERROR)"},{"line_number":1580,"context_line":""},{"line_number":1581,"context_line":"    def _delete_lrouter_port(self, context, port_id, router_id\u003dNone, txn\u003dNone):"},{"line_number":1582,"context_line":"        \"\"\"Delete a logical router port.\"\"\""}],"source_content_type":"text/x-python","patch_set":4,"id":"081f1859_cde61c5c","line":1579,"range":{"start_line":1570,"start_character":8,"end_line":1579,"end_character":76},"in_reply_to":"19a914ae_1fafc2cc","updated":"2022-01-07 09:27:18.000000000","message":"Good, I\u0027ll add the patch link here.","commit_id":"2007f7c8bc3a76722ad3ca7fd3e4be1451de6611"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"c16cbb3d8294cc22ae4bbece3582f6f127669a7c","unresolved":true,"context_lines":[{"line_number":1567,"context_line":"                    stop\u003dtenacity.stop_after_attempt(3))"},{"line_number":1568,"context_line":"    def delete_mac_binding_entries_by_mac(self, mac):"},{"line_number":1569,"context_line":"        \"\"\"Delete all MAC_Binding entries associated to this IP/mac address\"\"\""},{"line_number":1570,"context_line":"        cmd \u003d [\u0027ovsdb-client\u0027, \u0027transact\u0027, ovn_conf.get_ovn_sb_connection(),"},{"line_number":1571,"context_line":"               \u0027--timeout\u0027, str(ovn_conf.get_ovn_ovsdb_timeout())]"},{"line_number":1572,"context_line":"        if ovn_conf.get_ovn_sb_private_key():"},{"line_number":1573,"context_line":"            cmd +\u003d [\u0027-p\u0027, ovn_conf.get_ovn_sb_private_key(), \u0027-c\u0027,"},{"line_number":1574,"context_line":"                    ovn_conf.get_ovn_sb_certificate(), \u0027-C\u0027,"},{"line_number":1575,"context_line":"                    ovn_conf.get_ovn_sb_ca_cert()]"},{"line_number":1576,"context_line":"        cmd +\u003d [\u0027[\"OVN_Southbound\", {\"op\": \"delete\", \"table\": \"MAC_Binding\", \u0027"},{"line_number":1577,"context_line":"                \u0027\"where\": [[\"mac\", \"\u003d\u003d\", \"%s\"]]}]\u0027 % mac]"},{"line_number":1578,"context_line":"        return processutils.execute(*cmd,"},{"line_number":1579,"context_line":"                                    log_errors\u003dprocessutils.LOG_FINAL_ERROR)"},{"line_number":1580,"context_line":""},{"line_number":1581,"context_line":"    def _delete_lrouter_port(self, context, port_id, router_id\u003dNone, txn\u003dNone):"},{"line_number":1582,"context_line":"        \"\"\"Delete a logical router port.\"\"\""}],"source_content_type":"text/x-python","patch_set":4,"id":"19a914ae_1fafc2cc","line":1579,"range":{"start_line":1570,"start_character":8,"end_line":1579,"end_character":76},"in_reply_to":"404f8837_aaf4770b","updated":"2022-01-07 07:47:26.000000000","message":"perhaps worth to add a note about it","commit_id":"2007f7c8bc3a76722ad3ca7fd3e4be1451de6611"},{"author":{"_account_id":34125,"name":"huanghailun","display_name":"huanghailun","email":"hailun.huang@easystack.cn","username":"lnun"},"change_message_id":"335f2e7da78bf6b50aaf247f9a0e72551e978b07","unresolved":false,"context_lines":[{"line_number":1567,"context_line":"                    stop\u003dtenacity.stop_after_attempt(3))"},{"line_number":1568,"context_line":"    def delete_mac_binding_entries_by_mac(self, mac):"},{"line_number":1569,"context_line":"        \"\"\"Delete all MAC_Binding entries associated to this IP/mac address\"\"\""},{"line_number":1570,"context_line":"        cmd \u003d [\u0027ovsdb-client\u0027, \u0027transact\u0027, ovn_conf.get_ovn_sb_connection(),"},{"line_number":1571,"context_line":"               \u0027--timeout\u0027, str(ovn_conf.get_ovn_ovsdb_timeout())]"},{"line_number":1572,"context_line":"        if ovn_conf.get_ovn_sb_private_key():"},{"line_number":1573,"context_line":"            cmd +\u003d [\u0027-p\u0027, ovn_conf.get_ovn_sb_private_key(), \u0027-c\u0027,"},{"line_number":1574,"context_line":"                    ovn_conf.get_ovn_sb_certificate(), \u0027-C\u0027,"},{"line_number":1575,"context_line":"                    ovn_conf.get_ovn_sb_ca_cert()]"},{"line_number":1576,"context_line":"        cmd +\u003d [\u0027[\"OVN_Southbound\", {\"op\": \"delete\", \"table\": \"MAC_Binding\", \u0027"},{"line_number":1577,"context_line":"                \u0027\"where\": [[\"mac\", \"\u003d\u003d\", \"%s\"]]}]\u0027 % mac]"},{"line_number":1578,"context_line":"        return processutils.execute(*cmd,"},{"line_number":1579,"context_line":"                                    log_errors\u003dprocessutils.LOG_FINAL_ERROR)"},{"line_number":1580,"context_line":""},{"line_number":1581,"context_line":"    def _delete_lrouter_port(self, context, port_id, router_id\u003dNone, txn\u003dNone):"},{"line_number":1582,"context_line":"        \"\"\"Delete a logical router port.\"\"\""}],"source_content_type":"text/x-python","patch_set":4,"id":"404f8837_aaf4770b","line":1579,"range":{"start_line":1570,"start_character":8,"end_line":1579,"end_character":76},"in_reply_to":"ee709d94_7c0088ce","updated":"2022-01-05 12:30:54.000000000","message":"this is refer to the modification of this patch:\nhttps://review.opendev.org/c/openstack/neutron/+/812805","commit_id":"2007f7c8bc3a76722ad3ca7fd3e4be1451de6611"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"c16cbb3d8294cc22ae4bbece3582f6f127669a7c","unresolved":true,"context_lines":[{"line_number":1628,"context_line":"                if port_removed:"},{"line_number":1629,"context_line":"                    self._delete_lrouter_port(context, port_id, router_id,"},{"line_number":1630,"context_line":"                                              txn\u003dtxn)"},{"line_number":1631,"context_line":"                if port_mac:"},{"line_number":1632,"context_line":"                    self.delete_mac_binding_entries_by_mac(port_mac)"},{"line_number":1633,"context_line":"                return"},{"line_number":1634,"context_line":""},{"line_number":1635,"context_line":"            if not subnet_ids:"}],"source_content_type":"text/x-python","patch_set":4,"id":"fa73413d_ea832c46","line":1632,"range":{"start_line":1631,"start_character":0,"end_line":1632,"end_character":68},"updated":"2022-01-07 07:47:26.000000000","message":"should we delete the mac entry before the port to avoid leaks in case something fails in between?","commit_id":"2007f7c8bc3a76722ad3ca7fd3e4be1451de6611"},{"author":{"_account_id":34125,"name":"huanghailun","display_name":"huanghailun","email":"hailun.huang@easystack.cn","username":"lnun"},"change_message_id":"bf01755c4bad284600b97af3e00b1cec7f693088","unresolved":false,"context_lines":[{"line_number":1628,"context_line":"                if port_removed:"},{"line_number":1629,"context_line":"                    self._delete_lrouter_port(context, port_id, router_id,"},{"line_number":1630,"context_line":"                                              txn\u003dtxn)"},{"line_number":1631,"context_line":"                if port_mac:"},{"line_number":1632,"context_line":"                    self.delete_mac_binding_entries_by_mac(port_mac)"},{"line_number":1633,"context_line":"                return"},{"line_number":1634,"context_line":""},{"line_number":1635,"context_line":"            if not subnet_ids:"}],"source_content_type":"text/x-python","patch_set":4,"id":"1025d696_9491afb6","line":1632,"range":{"start_line":1631,"start_character":0,"end_line":1632,"end_character":68},"in_reply_to":"fa73413d_ea832c46","updated":"2022-01-07 09:27:18.000000000","message":"This is to prevent learning the MAC of the port again after deleting the Mac and before deleting the port.\nAnd if the port deletion fails, there will be no problem deleting the MAC address.","commit_id":"2007f7c8bc3a76722ad3ca7fd3e4be1451de6611"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"57127fa76c2de8c4d7bb2061a85520587791afd2","unresolved":true,"context_lines":[{"line_number":1572,"context_line":"        is refer to patch:"},{"line_number":1573,"context_line":"        https://review.opendev.org/c/openstack/neutron/+/812805"},{"line_number":1574,"context_line":"        \"\"\""},{"line_number":1575,"context_line":"        cmd \u003d [\u0027ovsdb-client\u0027, \u0027transact\u0027, ovn_conf.get_ovn_sb_connection(),"},{"line_number":1576,"context_line":"               \u0027--timeout\u0027, str(ovn_conf.get_ovn_ovsdb_timeout())]"},{"line_number":1577,"context_line":"        if ovn_conf.get_ovn_sb_private_key():"},{"line_number":1578,"context_line":"            cmd +\u003d [\u0027-p\u0027, ovn_conf.get_ovn_sb_private_key(), \u0027-c\u0027,"},{"line_number":1579,"context_line":"                    ovn_conf.get_ovn_sb_certificate(), \u0027-C\u0027,"}],"source_content_type":"text/x-python","patch_set":5,"id":"b7f68884_0d272d69","line":1576,"range":{"start_line":1575,"start_character":8,"end_line":1576,"end_character":66},"updated":"2022-01-07 10:30:07.000000000","message":"If more \"ovsdb-client\" commands are needed, we\u0027ll need to create a utility method, somewhere in neutron.common.ovn","commit_id":"f2e8a619c5ce7a4b4713f40c6da68da1a3e4410c"}]}