)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"d065442e4adaba208a7de45541d6e2574c755a4f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"3e7cdd45_fac8d2ec","updated":"2022-01-31 16:57:42.000000000","message":"Abandon in favor of https://review.opendev.org/c/openstack/neutron/+/827156","commit_id":"2d77f98d81b5b51d28f9e5a493d1ba10a7956f19"},{"author":{"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},"change_message_id":"ef9eb10ad46d93f64a22d3b2124921c030709a59","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"c90bb68d_2d8b20e9","updated":"2022-01-10 15:57:10.000000000","message":"There is backportability question in xena https://review.opendev.org/c/openstack/neutron/+/823267 but I have one additional comment for this one (and probably older branches backports)","commit_id":"2d77f98d81b5b51d28f9e5a493d1ba10a7956f19"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"eb9937831bcb50159615d0a0e3a0f7cc5c3a4783","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"055ad720_10968150","updated":"2022-02-17 09:07:05.000000000","message":"-1 for visibility, Bernard\u0027s question","commit_id":"f2ddb6741ead160773076f1092b8bd1487736eae"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"716fbd5c0f93f0d9e5e33a088d0efd6a1ec35071","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"6ef049d7_f355dabb","updated":"2022-02-17 07:17:47.000000000","message":"recheck","commit_id":"f2ddb6741ead160773076f1092b8bd1487736eae"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"649ab4d9c036c971e31b76556692820f5f50b723","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"b619d0dc_c8ed91cb","updated":"2022-02-15 14:49:29.000000000","message":"recheck","commit_id":"f2ddb6741ead160773076f1092b8bd1487736eae"}],"neutron/tests/functional/plugins/ml2/drivers/ovn/mech_driver/ovsdb/test_ovsdb_monitor.py":[{"author":{"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},"change_message_id":"ef9eb10ad46d93f64a22d3b2124921c030709a59","unresolved":true,"context_lines":[{"line_number":205,"context_line":"            try:"},{"line_number":206,"context_line":"                self.sb_api.db_set(\u0027Port_Binding\u0027, port_uuid,"},{"line_number":207,"context_line":"                                   (\u0027up\u0027, True)).execute(check_error\u003dTrue)"},{"line_number":208,"context_line":"            except KeyError:"},{"line_number":209,"context_line":"                self.sb_api.db_set(\u0027Port_Binding\u0027, port_uuid,"},{"line_number":210,"context_line":"                                   (\u0027chassis\u0027, \u0027host-1\u0027)).execute("},{"line_number":211,"context_line":"                                       check_error\u003dTrue)"}],"source_content_type":"text/x-python","patch_set":3,"id":"2476ab75_c2fec4b5","line":208,"range":{"start_line":208,"start_character":11,"end_line":208,"end_character":28},"updated":"2022-01-10 15:57:10.000000000","message":"Needs a Conflicts line in commit message for that change, also why is this not needed in newer branches? Is it because of the used OVN version?","commit_id":"2d77f98d81b5b51d28f9e5a493d1ba10a7956f19"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"f0dff756541791cbb053248934f050452ca22a21","unresolved":true,"context_lines":[{"line_number":205,"context_line":"            try:"},{"line_number":206,"context_line":"                self.sb_api.db_set(\u0027Port_Binding\u0027, port_uuid,"},{"line_number":207,"context_line":"                                   (\u0027up\u0027, True)).execute(check_error\u003dTrue)"},{"line_number":208,"context_line":"            except KeyError:"},{"line_number":209,"context_line":"                self.sb_api.db_set(\u0027Port_Binding\u0027, port_uuid,"},{"line_number":210,"context_line":"                                   (\u0027chassis\u0027, \u0027host-1\u0027)).execute("},{"line_number":211,"context_line":"                                       check_error\u003dTrue)"}],"source_content_type":"text/x-python","patch_set":3,"id":"280759d5_f23ba285","line":208,"range":{"start_line":208,"start_character":11,"end_line":208,"end_character":28},"in_reply_to":"2476ab75_c2fec4b5","updated":"2022-02-17 09:22:21.000000000","message":"there were no conflict here, it was just the test failing due to older version of ovn not having that field. So I had to add this extra checkings. I discussed this with bcafarel over IRC, and agreed to tackle that on the commit message by adding: \n\"As the \"up\" field may not exists in older versions, therere is a need to\nadapt the functional test in case that field does not exists (KeyError),\nand therefore falling back to previous behaviour, i.e., just setting the\nchassis.\"","commit_id":"2d77f98d81b5b51d28f9e5a493d1ba10a7956f19"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"9a2d266d8af628ef750b7af5475e512f73787247","unresolved":true,"context_lines":[{"line_number":205,"context_line":"            try:"},{"line_number":206,"context_line":"                self.sb_api.db_set(\u0027Port_Binding\u0027, port_uuid,"},{"line_number":207,"context_line":"                                   (\u0027up\u0027, True)).execute(check_error\u003dTrue)"},{"line_number":208,"context_line":"            except KeyError:"},{"line_number":209,"context_line":"                self.sb_api.db_set(\u0027Port_Binding\u0027, port_uuid,"},{"line_number":210,"context_line":"                                   (\u0027chassis\u0027, \u0027host-1\u0027)).execute("},{"line_number":211,"context_line":"                                       check_error\u003dTrue)"}],"source_content_type":"text/x-python","patch_set":3,"id":"e36a4e76_11576122","line":208,"range":{"start_line":208,"start_character":11,"end_line":208,"end_character":28},"in_reply_to":"280759d5_f23ba285","updated":"2022-02-17 09:26:13.000000000","message":"ack","commit_id":"2d77f98d81b5b51d28f9e5a493d1ba10a7956f19"}]}