)]}'
{"neutron/services/portforwarding/callbacks.py":[{"author":{"_account_id":5948,"name":"Oleg Bondarev","email":"obondarev@mirantis.com","username":"obondarev"},"change_message_id":"d719a8002e5dbbd43c8f9e06e460e409e93c6d7c","unresolved":true,"context_lines":[{"line_number":20,"context_line":"        self.current_pf \u003d current_pf"},{"line_number":21,"context_line":"        self.original_pf \u003d original_pf"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"    def __eq__(self, other):"},{"line_number":24,"context_line":"        return (isinstance(other, self.__class__) and"},{"line_number":25,"context_line":"                self.__dict__ \u003d\u003d other.__dict__)"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"    def __ne__(self, other):"},{"line_number":28,"context_line":"        return not self.__eq__(other)"}],"source_content_type":"text/x-python","patch_set":5,"id":"362abb07_e25b8476","side":"PARENT","line":28,"range":{"start_line":23,"start_character":4,"end_line":28,"end_character":37},"updated":"2021-08-13 08:12:21.000000000","message":"Please check if payloads are compared to each other somewhere. DBEventPayload may not support comparison.","commit_id":"578f74ca8782badfa6251f438c3cfd3e8fd10009"},{"author":{"_account_id":32667,"name":"Mamatisa Nurmatov","email":"nurmatov.mamatisa@huawei.com","username":"isabek"},"change_message_id":"d4561021d03740e79be27c1f4b070e69020c93c0","unresolved":true,"context_lines":[{"line_number":20,"context_line":"        self.current_pf \u003d current_pf"},{"line_number":21,"context_line":"        self.original_pf \u003d original_pf"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"    def __eq__(self, other):"},{"line_number":24,"context_line":"        return (isinstance(other, self.__class__) and"},{"line_number":25,"context_line":"                self.__dict__ \u003d\u003d other.__dict__)"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"    def __ne__(self, other):"},{"line_number":28,"context_line":"        return not self.__eq__(other)"}],"source_content_type":"text/x-python","patch_set":5,"id":"c05b3791_183f06bf","side":"PARENT","line":28,"range":{"start_line":23,"start_character":4,"end_line":28,"end_character":37},"in_reply_to":"362abb07_e25b8476","updated":"2021-08-13 09:31:56.000000000","message":"I have checked code, there is no comparison","commit_id":"578f74ca8782badfa6251f438c3cfd3e8fd10009"}],"neutron/services/portforwarding/drivers/ovn/driver.py":[{"author":{"_account_id":5948,"name":"Oleg Bondarev","email":"obondarev@mirantis.com","username":"obondarev"},"change_message_id":"d483fb167835ead12443ec03b8c75afb8e0c56f6","unresolved":true,"context_lines":[{"line_number":174,"context_line":"        ovn_nb \u003d self._l3_plugin._ovn"},{"line_number":175,"context_line":"        with ovn_nb.transaction(check_error\u003dTrue) as ovn_txn:"},{"line_number":176,"context_line":"            if event_type \u003d\u003d events.AFTER_CREATE:"},{"line_number":177,"context_line":"                for pf_payload in payload:"},{"line_number":178,"context_line":"                    self._handler.port_forwarding_created(ovn_txn, ovn_nb,"},{"line_number":179,"context_line":"                        pf_payload.latest_state)"},{"line_number":180,"context_line":"                    self._l3_plugin.update_floatingip_status("}],"source_content_type":"text/x-python","patch_set":4,"id":"2e1d62d0_49dfd9b4","line":177,"range":{"start_line":177,"start_character":16,"end_line":177,"end_character":41},"updated":"2021-08-12 13:52:01.000000000","message":"payload is not a list now, so this \u0027for\u0027 loop should be removed","commit_id":"a3096bb06755eb8c4b3ce5398655cf6cb3f7f25e"},{"author":{"_account_id":32667,"name":"Mamatisa Nurmatov","email":"nurmatov.mamatisa@huawei.com","username":"isabek"},"change_message_id":"e20bbeb7645e54fc09441f1749dfead995fdeced","unresolved":false,"context_lines":[{"line_number":174,"context_line":"        ovn_nb \u003d self._l3_plugin._ovn"},{"line_number":175,"context_line":"        with ovn_nb.transaction(check_error\u003dTrue) as ovn_txn:"},{"line_number":176,"context_line":"            if event_type \u003d\u003d events.AFTER_CREATE:"},{"line_number":177,"context_line":"                for pf_payload in payload:"},{"line_number":178,"context_line":"                    self._handler.port_forwarding_created(ovn_txn, ovn_nb,"},{"line_number":179,"context_line":"                        pf_payload.latest_state)"},{"line_number":180,"context_line":"                    self._l3_plugin.update_floatingip_status("}],"source_content_type":"text/x-python","patch_set":4,"id":"6dd68d23_df791fb2","line":177,"range":{"start_line":177,"start_character":16,"end_line":177,"end_character":41},"in_reply_to":"2e1d62d0_49dfd9b4","updated":"2021-08-12 14:57:17.000000000","message":"Done","commit_id":"a3096bb06755eb8c4b3ce5398655cf6cb3f7f25e"}]}
