)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"9e9f6c67fa5d5b9e94ec49ed2d99f75ee6edc5ed","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"b84a0e3f_a6364147","updated":"2022-04-11 11:48:28.000000000","message":"I\u0027m ","commit_id":"4f75c6a616d3cb549153fcc496926358dfc9178a"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"63a82e403ef60e3312a77933ff95c796ad5434dd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"9cd627b1_d5cdcd11","updated":"2022-04-05 12:58:12.000000000","message":"recheck - unrelated functional tests error with netlink \"NetlinkDumpInterrupted\"","commit_id":"4f75c6a616d3cb549153fcc496926358dfc9178a"}],"neutron/plugins/ml2/drivers/ovn/mech_driver/mech_driver.py":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"9e9f6c67fa5d5b9e94ec49ed2d99f75ee6edc5ed","unresolved":true,"context_lines":[{"line_number":777,"context_line":"                    LOG.debug(\"Revision number of the port %s has changed \""},{"line_number":778,"context_line":"                              \"probably during live migration. Retrying \""},{"line_number":779,"context_line":"                              \"update port in OVN.\", port)"},{"line_number":780,"context_line":"                    db_port \u003d self._plugin.get_port(plugin_context,"},{"line_number":781,"context_line":"                                                    port[\u0027id\u0027])"},{"line_number":782,"context_line":"                    port[\u0027revision_number\u0027] \u003d db_port[\u0027revision_number\u0027]"},{"line_number":783,"context_line":"                    self._ovn_update_port(plugin_context, port, original_port,"},{"line_number":784,"context_line":"                                          retry_on_revision_mismatch\u003dFalse)"},{"line_number":785,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"71dbdd5a_81590c9e","line":782,"range":{"start_line":780,"start_character":20,"end_line":782,"end_character":72},"updated":"2022-04-11 11:48:28.000000000","message":"At this point I was going to propose to use the OVN DB port revision number. But you are right, we should not hack this and we should use the Neutron DB info.","commit_id":"4f75c6a616d3cb549153fcc496926358dfc9178a"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"7b31633fd9efbaecf51e9facb9b6deeeec085e29","unresolved":true,"context_lines":[{"line_number":777,"context_line":"                    LOG.debug(\"Revision number of the port %s has changed \""},{"line_number":778,"context_line":"                              \"probably during live migration. Retrying \""},{"line_number":779,"context_line":"                              \"update port in OVN.\", port)"},{"line_number":780,"context_line":"                    db_port \u003d self._plugin.get_port(plugin_context,"},{"line_number":781,"context_line":"                                                    port[\u0027id\u0027])"},{"line_number":782,"context_line":"                    port[\u0027revision_number\u0027] \u003d db_port[\u0027revision_number\u0027]"},{"line_number":783,"context_line":"                    self._ovn_update_port(plugin_context, port, original_port,"},{"line_number":784,"context_line":"                                          retry_on_revision_mismatch\u003dFalse)"},{"line_number":785,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"412e3c0a_eebe571b","line":782,"range":{"start_line":780,"start_character":20,"end_line":782,"end_character":72},"in_reply_to":"71dbdd5a_81590c9e","updated":"2022-04-11 12:24:36.000000000","message":"I agree, let\u0027s not mess with those revisions as they are used for synchronization between the two dbs","commit_id":"4f75c6a616d3cb549153fcc496926358dfc9178a"}],"neutron/tests/unit/plugins/ml2/drivers/ovn/mech_driver/test_mech_driver.py":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"9e9f6c67fa5d5b9e94ec49ed2d99f75ee6edc5ed","unresolved":true,"context_lines":[{"line_number":2001,"context_line":"                       \u0027_is_port_provisioning_required\u0027, lambda *_: True)"},{"line_number":2002,"context_line":"    @mock.patch.object(mech_driver.OVNMechanismDriver, \u0027_notify_dhcp_updated\u0027)"},{"line_number":2003,"context_line":"    @mock.patch.object(ovn_client.OVNClient, \u0027update_port\u0027)"},{"line_number":2004,"context_line":"    def test_update_port_postcommit_live_migration_revision_mismatch_once("},{"line_number":2005,"context_line":"            self, mock_update_port, mock_notify_dhcp):"},{"line_number":2006,"context_line":"        self.plugin.update_port_status \u003d mock.Mock()"},{"line_number":2007,"context_line":"        self.plugin.get_port \u003d mock.Mock(return_value\u003dmock.MagicMock())"}],"source_content_type":"text/x-python","patch_set":1,"id":"275ea983_55bb750b","line":2004,"range":{"start_line":2004,"start_character":8,"end_line":2004,"end_character":73},"updated":"2022-04-11 11:48:28.000000000","message":"nit: I think this test is enough and the previous one is not needed.","commit_id":"4f75c6a616d3cb549153fcc496926358dfc9178a"}]}