)]}'
{"neutron/plugins/ml2/drivers/ovn/mech_driver/mech_driver.py":[{"author":{"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},"change_message_id":"6a57ecd98b847911ed1221a6b6080c42ccea87f5","unresolved":true,"context_lines":[{"line_number":270,"context_line":"                impl_idl_ovn.OvsdbNbOvnIdl, idl) as pre_ovn_nb_api:"},{"line_number":271,"context_line":"            try:"},{"line_number":272,"context_line":"                create_default_drop_port_group(pre_ovn_nb_api)"},{"line_number":273,"context_line":"            except KeyError:"},{"line_number":274,"context_line":"                # Due to a bug in python-ovs, we can send transactions before"},{"line_number":275,"context_line":"                # the initial OVSDB is populated in memory. This can break"},{"line_number":276,"context_line":"                # the AddCommand post_commit method which tries to return a"}],"source_content_type":"text/x-python","patch_set":1,"id":"afc0a0aa_5e2f1490","line":273,"range":{"start_line":273,"start_character":12,"end_line":273,"end_character":27},"updated":"2021-09-01 19:18:12.000000000","message":"It looks OK to me as this is for this specific short lived connection","commit_id":"76ee64f4a8095ba207872d5de9613f93f208fdd4"},{"author":{"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},"change_message_id":"6a57ecd98b847911ed1221a6b6080c42ccea87f5","unresolved":true,"context_lines":[{"line_number":277,"context_line":"                # row looked up by the newly commited row\u0027s uuid. Since we"},{"line_number":278,"context_line":"                # don\u0027t care about the return value from the PgAddCommand, we"},{"line_number":279,"context_line":"                # can just catch the KeyError and continue. This can be"},{"line_number":280,"context_line":"                # removed when the python-ovs bug is resolved."},{"line_number":281,"context_line":"                pass"},{"line_number":282,"context_line":"            except RuntimeError as re:"},{"line_number":283,"context_line":"                # If we don\u0027t get the lock, and the port group didn\u0027t exist"}],"source_content_type":"text/x-python","patch_set":1,"id":"eb455993_3ae1f9be","line":280,"range":{"start_line":280,"start_character":26,"end_line":280,"end_character":61},"updated":"2021-09-01 19:18:12.000000000","message":"Do we have a handy link to that bug that we can put here in the comments? (to track it more easily in code)","commit_id":"76ee64f4a8095ba207872d5de9613f93f208fdd4"},{"author":{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"change_message_id":"6bb07e0e6528f564d3e6e51d43a0a1ef98fc5bb2","unresolved":true,"context_lines":[{"line_number":277,"context_line":"                # row looked up by the newly commited row\u0027s uuid. Since we"},{"line_number":278,"context_line":"                # don\u0027t care about the return value from the PgAddCommand, we"},{"line_number":279,"context_line":"                # can just catch the KeyError and continue. This can be"},{"line_number":280,"context_line":"                # removed when the python-ovs bug is resolved."},{"line_number":281,"context_line":"                pass"},{"line_number":282,"context_line":"            except RuntimeError as re:"},{"line_number":283,"context_line":"                # If we don\u0027t get the lock, and the port group didn\u0027t exist"}],"source_content_type":"text/x-python","patch_set":1,"id":"c19e4b10_3aa34636","line":280,"range":{"start_line":280,"start_character":26,"end_line":280,"end_character":61},"in_reply_to":"eb455993_3ae1f9be","updated":"2021-09-01 20:04:20.000000000","message":"I have since created this patch: https://patchwork.ozlabs.org/project/openvswitch/patch/20210901161526.237479-1-twilson@redhat.com/\n\nbut the line-length of the URL is \u003e 79 chars. :D There are some related issues at bugzilla.redhat.com I could probably find to track it.","commit_id":"76ee64f4a8095ba207872d5de9613f93f208fdd4"}]}