)]}'
{"neutron/cmd/ovn/neutron_ovn_db_sync_util.py":[{"author":{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"change_message_id":"0958193ce870aa9c73847cc84bfbe2df1e3dd93a","unresolved":false,"context_lines":[{"line_number":200,"context_line":"    manager.init()"},{"line_number":201,"context_line":"    core_plugin \u003d directory.get_plugin()"},{"line_number":202,"context_line":"    driver \u003d core_plugin.mechanism_manager.mech_drivers[\u0027ovn-sync\u0027]"},{"line_number":203,"context_line":"    core_plugin.mechanism_manager.mech_drivers[\u0027ovn\u0027] \u003d driver"},{"line_number":204,"context_line":"    ovn_driver \u003d driver.obj"},{"line_number":205,"context_line":"    ovn_driver._nb_ovn \u003d ovn_api"},{"line_number":206,"context_line":"    ovn_driver._sb_ovn \u003d ovn_sb_api"}],"source_content_type":"text/x-python","patch_set":1,"id":"1f493fa4_0fd85e7d","line":203,"updated":"2020-05-04 17:10:00.000000000","message":"Wasn\u0027t completely sure whether to do this here to keep change local to the sync util, or whether it would be better to do try/except in neutron.services.ovn_l3.plugin.OVNL3RouterPlugin._plugin_driver.","commit_id":"50883930bbb040e1a1d5ae92eba450f0cfe58e96"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"98e396aa1f880cb54f39ebdd0613c43d49c3d0c7","unresolved":false,"context_lines":[{"line_number":200,"context_line":"    manager.init()"},{"line_number":201,"context_line":"    core_plugin \u003d directory.get_plugin()"},{"line_number":202,"context_line":"    driver \u003d core_plugin.mechanism_manager.mech_drivers[\u0027ovn-sync\u0027]"},{"line_number":203,"context_line":"    core_plugin.mechanism_manager.mech_drivers[\u0027ovn\u0027] \u003d driver"},{"line_number":204,"context_line":"    ovn_driver \u003d driver.obj"},{"line_number":205,"context_line":"    ovn_driver._nb_ovn \u003d ovn_api"},{"line_number":206,"context_line":"    ovn_driver._sb_ovn \u003d ovn_sb_api"}],"source_content_type":"text/x-python","patch_set":1,"id":"1f493fa4_cd778c08","line":203,"in_reply_to":"1f493fa4_0fd85e7d","updated":"2020-05-05 13:36:49.000000000","message":"Thanks for the fix and suggestion.\n\nPersonally I would prefer to keep the fix contained in the tool itself as you proposed.\n\nAlso, I wonder why we didn\u0027t see any failures in the tests due to this ?","commit_id":"50883930bbb040e1a1d5ae92eba450f0cfe58e96"},{"author":{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"change_message_id":"a41015ff7a0e0a8ef97308d56c6e637e9b88dd2f","unresolved":false,"context_lines":[{"line_number":200,"context_line":"    manager.init()"},{"line_number":201,"context_line":"    core_plugin \u003d directory.get_plugin()"},{"line_number":202,"context_line":"    driver \u003d core_plugin.mechanism_manager.mech_drivers[\u0027ovn-sync\u0027]"},{"line_number":203,"context_line":"    core_plugin.mechanism_manager.mech_drivers[\u0027ovn\u0027] \u003d driver"},{"line_number":204,"context_line":"    ovn_driver \u003d driver.obj"},{"line_number":205,"context_line":"    ovn_driver._nb_ovn \u003d ovn_api"},{"line_number":206,"context_line":"    ovn_driver._sb_ovn \u003d ovn_sb_api"}],"source_content_type":"text/x-python","patch_set":1,"id":"1f493fa4_f1efc2e6","line":203,"in_reply_to":"1f493fa4_cd778c08","updated":"2020-05-05 13:53:13.000000000","message":"I think we just test the maintenance worker thread and not running the sync util itself, but I could be wrong.","commit_id":"50883930bbb040e1a1d5ae92eba450f0cfe58e96"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"ff541a4de9d56c2f88273fe2f9a9cf338fd9e228","unresolved":false,"context_lines":[{"line_number":200,"context_line":"    manager.init()"},{"line_number":201,"context_line":"    core_plugin \u003d directory.get_plugin()"},{"line_number":202,"context_line":"    driver \u003d core_plugin.mechanism_manager.mech_drivers[\u0027ovn-sync\u0027]"},{"line_number":203,"context_line":"    core_plugin.mechanism_manager.mech_drivers[\u0027ovn\u0027] \u003d driver"},{"line_number":204,"context_line":"    ovn_driver \u003d driver.obj"},{"line_number":205,"context_line":"    ovn_driver._nb_ovn \u003d ovn_api"},{"line_number":206,"context_line":"    ovn_driver._sb_ovn \u003d ovn_sb_api"}],"source_content_type":"text/x-python","patch_set":1,"id":"1f493fa4_9acc9362","line":203,"in_reply_to":"1f493fa4_f1efc2e6","updated":"2020-05-05 15:22:32.000000000","message":"Maybe it would be good to leave a comment here. When we come in a year from now, we maybe think \"why do we set the \u0027ovn\u0027 driver\"?","commit_id":"50883930bbb040e1a1d5ae92eba450f0cfe58e96"}]}