)]}'
{"neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/ovn_client.py":[{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"fa01256a4a03b87ea3cfefe57ebe1bec3127e54e","unresolved":false,"context_lines":[{"line_number":565,"context_line":""},{"line_number":566,"context_line":"        # NOTE(mjozefcz): try to pack everything in one txn, if it was"},{"line_number":567,"context_line":"        # passed in kwargs."},{"line_number":568,"context_line":"        if txn:"},{"line_number":569,"context_line":"            ret \u003d _update()"},{"line_number":570,"context_line":"        else:"},{"line_number":571,"context_line":"            with self._nb_idl.transaction(check_error\u003dTrue) as txn:"},{"line_number":572,"context_line":"                ret \u003d _update()"},{"line_number":573,"context_line":""},{"line_number":574,"context_line":"        if ret.result \u003d\u003d ovn_const.TXN_COMMITTED:"},{"line_number":575,"context_line":"            db_rev.bump_revision(context, port, ovn_const.TYPE_PORTS)"}],"source_content_type":"text/x-python","patch_set":15,"id":"9f560f44_e54a38aa","line":572,"range":{"start_line":568,"start_character":0,"end_line":572,"end_character":31},"updated":"2020-07-30 09:49:39.000000000","message":"self._transaction(commands, txn\u003dtxn)","commit_id":"7926f1561bf8b4954d16e3858802c626e7f0b67d"},{"author":{"_account_id":24791,"name":"Maciej Jozefczyk","email":"jeicam.pl@gmail.com","username":"maciej.jozefczyk"},"change_message_id":"4cb2e4ee449af3eb6d79239bf579c17730dc5f7f","unresolved":false,"context_lines":[{"line_number":565,"context_line":""},{"line_number":566,"context_line":"        # NOTE(mjozefcz): try to pack everything in one txn, if it was"},{"line_number":567,"context_line":"        # passed in kwargs."},{"line_number":568,"context_line":"        if txn:"},{"line_number":569,"context_line":"            ret \u003d _update()"},{"line_number":570,"context_line":"        else:"},{"line_number":571,"context_line":"            with self._nb_idl.transaction(check_error\u003dTrue) as txn:"},{"line_number":572,"context_line":"                ret \u003d _update()"},{"line_number":573,"context_line":""},{"line_number":574,"context_line":"        if ret.result \u003d\u003d ovn_const.TXN_COMMITTED:"},{"line_number":575,"context_line":"            db_rev.bump_revision(context, port, ovn_const.TYPE_PORTS)"}],"source_content_type":"text/x-python","patch_set":15,"id":"9f560f44_a0f30ee7","line":572,"range":{"start_line":568,"start_character":0,"end_line":572,"end_character":31},"in_reply_to":"9f560f44_e54a38aa","updated":"2020-07-30 10:33:49.000000000","message":"Not really. That will need a list of commands, but the list is not directly generated in port_update().\nFunctions like:\n\nself._set_unset_virtual_port_type(\n    context, txn, port, addr_pairs_diff.added)\n\nneeds txn passed. So that complicates it and we would need a refactor :(","commit_id":"7926f1561bf8b4954d16e3858802c626e7f0b67d"}]}