)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":23804,"name":"Daniel Alvarez","email":"dalvarez@redhat.com","username":"dalvarez"},"change_message_id":"521132e6178a71ef600a010524fba5bc4f5faa14","unresolved":false,"context_lines":[{"line_number":7,"context_line":"ovn: Override clean_hash_ring for sync util"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"ML2 OVN Mechanism driver wipes all the nodes from ovn_hash_ring table on"},{"line_number":10,"context_line":"initialization. The DB sync untility inherits from the mechanism driver"},{"line_number":11,"context_line":"calling the cleanup whenever sync util is executed. This leads to nodes"},{"line_number":12,"context_line":"being deleted while neutron-server is typically running, causing serious"},{"line_number":13,"context_line":"issues to neutron-server until it is restarted."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"ff570b3c_c6cdd566","line":10,"range":{"start_line":10,"start_character":28,"end_line":10,"end_character":36},"updated":"2020-05-21 15:24:51.000000000","message":"nit: utility","commit_id":"f5c391077139b9ca61349b59e1e5270b42125cd2"}],"neutron/cmd/ovn/neutron_ovn_db_sync_util.py":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"83fe8a2d9738c90679cdf45d7c94f77fe85d1d69","unresolved":false,"context_lines":[{"line_number":61,"context_line":"        If this method was not overriden, cleanup would be performed when"},{"line_number":62,"context_line":"        calling the db sync and running neutron server would lose all the nodes"},{"line_number":63,"context_line":"        from the ring."},{"line_number":64,"context_line":"        \"\"\""},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"    # Since we are not using the ovn mechanism driver while syncing,"},{"line_number":67,"context_line":"    # we override the post and pre commit methods so that original ones are"}],"source_content_type":"text/x-python","patch_set":1,"id":"ff570b3c_7c557cf9","line":64,"updated":"2020-05-21 16:20:03.000000000","message":"I guess this doesn\u0027t need \u0027pass\u0027 here?","commit_id":"f5c391077139b9ca61349b59e1e5270b42125cd2"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"bb9c70172ceec1104a0198b17f1975793ab9aada","unresolved":false,"context_lines":[{"line_number":61,"context_line":"        If this method was not overriden, cleanup would be performed when"},{"line_number":62,"context_line":"        calling the db sync and running neutron server would lose all the nodes"},{"line_number":63,"context_line":"        from the ring."},{"line_number":64,"context_line":"        \"\"\""},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"    # Since we are not using the ovn mechanism driver while syncing,"},{"line_number":67,"context_line":"    # we override the post and pre commit methods so that original ones are"}],"source_content_type":"text/x-python","patch_set":1,"id":"ff570b3c_77b6358c","line":64,"in_reply_to":"ff570b3c_7c557cf9","updated":"2020-05-21 17:21:03.000000000","message":"It has a docstring, so the pass is not needed.","commit_id":"f5c391077139b9ca61349b59e1e5270b42125cd2"}]}