)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"b589f0c18352cd7c0cc39a8d6b2a4a953ee7164e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"29ecbed6_185e399c","updated":"2022-09-13 08:00:02.000000000","message":"If merged, we\u0027ll need to backport this revert up to Train (networking-ovn)","commit_id":"30d1a40c508386905b12c43be1891ce503a8b634"}],"neutron/plugins/ml2/drivers/ovn/mech_driver/mech_driver.py":[{"author":{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"change_message_id":"30910fa9b5a3dfa51f83ffb423a5a4fec6e54100","unresolved":true,"context_lines":[{"line_number":289,"context_line":"            if not target:"},{"line_number":290,"context_line":"                continue"},{"line_number":291,"context_line":""},{"line_number":292,"context_line":"            idl \u003d ovsdb_monitor.BaseOvnIdl.from_server(connection, schema)"},{"line_number":293,"context_line":"            with ovsdb_monitor.short_living_ovsdb_api(klass, idl) as idl_api:"},{"line_number":294,"context_line":"                conn \u003d idlutils.row_by_value(idl_api, \u0027Connection\u0027, \u0027target\u0027,"},{"line_number":295,"context_line":"                                             target, None)"}],"source_content_type":"text/x-python","patch_set":1,"id":"ad99d5d5_ad7aef85","side":"PARENT","line":292,"updated":"2022-09-10 21:35:45.000000000","message":"I don\u0027t disagree that it would be better for the installer to handle this, but on the off chance that someone relies on this, we could also just pass in the Connection table only I stead of taking the default behavior of registering all tables.","commit_id":"28f3017a90ecec6208aef9696cd7947972ec17d8"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"646226cc061e4db474f824d151e1df633a007b2e","unresolved":true,"context_lines":[{"line_number":289,"context_line":"            if not target:"},{"line_number":290,"context_line":"                continue"},{"line_number":291,"context_line":""},{"line_number":292,"context_line":"            idl \u003d ovsdb_monitor.BaseOvnIdl.from_server(connection, schema)"},{"line_number":293,"context_line":"            with ovsdb_monitor.short_living_ovsdb_api(klass, idl) as idl_api:"},{"line_number":294,"context_line":"                conn \u003d idlutils.row_by_value(idl_api, \u0027Connection\u0027, \u0027target\u0027,"},{"line_number":295,"context_line":"                                             target, None)"}],"source_content_type":"text/x-python","patch_set":1,"id":"b9b04005_81c70edc","side":"PARENT","line":292,"in_reply_to":"ad99d5d5_ad7aef85","updated":"2022-09-10 21:40:22.000000000","message":"That\u0027s a good point and I thought about it too. But then I decided to send out the revert because it\u0027s not Neutron\u0027s responsibility to configure services it relies on. e.g. we don\u0027t do and shouldn\u0027t do any configuration changes of rabbit or mysql.","commit_id":"28f3017a90ecec6208aef9696cd7947972ec17d8"}]}