)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":23766,"name":"Dima Kuznetsov","email":"dima.kuznetsov@toganetworks.com","username":"dimak"},"change_message_id":"26fae61450fb63a5578b8578b04051f2de470b8d","unresolved":false,"context_lines":[{"line_number":9,"context_line":"OvsPort holds its lport ID in iface_id. This change turns that field"},{"line_number":10,"context_line":"into a reference to lport, using the ID as key."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"The field (iface_id) is also renamed to iface to reflect the change in"},{"line_number":13,"context_line":"semantics."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Related-Bug: #1690775"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3f1d235d_b5246fb0","line":12,"range":{"start_line":12,"start_character":40,"end_line":12,"end_character":45},"updated":"2017-07-04 17:20:59.000000000","message":"Why not lport?","commit_id":"e3a8206271d2312f8831db9a260469e86330f1bf"}],"dragonflow/controller/apps/classifier.py":[{"author":{"_account_id":23766,"name":"Dima Kuznetsov","email":"dima.kuznetsov@toganetworks.com","username":"dimak"},"change_message_id":"1c11c829f7156e975eca84319b7ff30f630342ce","unresolved":false,"context_lines":[{"line_number":49,"context_line":"            self._ovs_port_deleted(ovs_port)"},{"line_number":50,"context_line":"        if not ofport or ofport \u003d\u003d -1:"},{"line_number":51,"context_line":"            return  # Not ready yet, or error"},{"line_number":52,"context_line":"        lport \u003d self.nb_api.get(lport_ref)"},{"line_number":53,"context_line":"        self._ofport_unique_key_map[ovs_port.id] \u003d (ofport, lport.unique_key)"},{"line_number":54,"context_line":"        LOG.info(\"Add local ovs port %(ovs_port)s, logical port \""},{"line_number":55,"context_line":"                 \"%(lport)s for classification\","}],"source_content_type":"text/x-python","patch_set":3,"id":"9f436f4f_56df2668","line":52,"range":{"start_line":52,"start_character":8,"end_line":52,"end_character":42},"updated":"2017-08-16 12:02:41.000000000","message":"Maybe get it from db_store if present? After all, we need just the port key, it is not going to change.","commit_id":"c3a84d2d5df2a73ca033b470729d970abc629c54"},{"author":{"_account_id":20229,"name":"Omer Anson","email":"omer.anson@toganetworks.com","username":"oanson"},"change_message_id":"70cc9165c39628b6f789bc1f282ab8c4331f488a","unresolved":false,"context_lines":[{"line_number":49,"context_line":"            self._ovs_port_deleted(ovs_port)"},{"line_number":50,"context_line":"        if not ofport or ofport \u003d\u003d -1:"},{"line_number":51,"context_line":"            return  # Not ready yet, or error"},{"line_number":52,"context_line":"        lport \u003d self.nb_api.get(lport_ref)"},{"line_number":53,"context_line":"        self._ofport_unique_key_map[ovs_port.id] \u003d (ofport, lport.unique_key)"},{"line_number":54,"context_line":"        LOG.info(\"Add local ovs port %(ovs_port)s, logical port \""},{"line_number":55,"context_line":"                 \"%(lport)s for classification\","}],"source_content_type":"text/x-python","patch_set":3,"id":"9f436f4f_f6ecdabb","line":52,"range":{"start_line":52,"start_character":8,"end_line":52,"end_character":42},"in_reply_to":"9f436f4f_56df2668","updated":"2017-08-16 12:23:03.000000000","message":"That would change the functionality here. It might be correct, but should be done in another patch.\n\nRefactor patches should *only* refactor. Otherwise you have changes hidden in commits you can\u0027t find.","commit_id":"c3a84d2d5df2a73ca033b470729d970abc629c54"},{"author":{"_account_id":23766,"name":"Dima Kuznetsov","email":"dima.kuznetsov@toganetworks.com","username":"dimak"},"change_message_id":"27855d2a616d1d31794cbc4893176e71c42c0f99","unresolved":false,"context_lines":[{"line_number":49,"context_line":"            self._ovs_port_deleted(ovs_port)"},{"line_number":50,"context_line":"        if not ofport or ofport \u003d\u003d -1:"},{"line_number":51,"context_line":"            return  # Not ready yet, or error"},{"line_number":52,"context_line":"        lport \u003d self.nb_api.get(lport_ref)"},{"line_number":53,"context_line":"        self._ofport_unique_key_map[ovs_port.id] \u003d (ofport, lport.unique_key)"},{"line_number":54,"context_line":"        LOG.info(\"Add local ovs port %(ovs_port)s, logical port \""},{"line_number":55,"context_line":"                 \"%(lport)s for classification\","}],"source_content_type":"text/x-python","patch_set":3,"id":"9f436f4f_d500a304","line":52,"range":{"start_line":52,"start_character":8,"end_line":52,"end_character":42},"in_reply_to":"9f436f4f_f6ecdabb","updated":"2017-08-16 18:17:06.000000000","message":"Agreed","commit_id":"c3a84d2d5df2a73ca033b470729d970abc629c54"}]}
