)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"089a293947d5f2c42d45244d0384753ff3edf103","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"4798bd3d_c54d95bb","updated":"2022-11-17 11:39:19.000000000","message":"This patch makes sense.\n\nIf the CI approves it, I\u0027ll +2 it.","commit_id":"9a483f02bebfd0f1241e40fc98eb740496f78990"},{"author":{"_account_id":30314,"name":"Sebastian Lohff","email":"sebastian.lohff@sap.com","username":"seba"},"change_message_id":"709f33672bf156b518a2cf19a6837bb980a0f471","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"9bb8abf9_45810f57","updated":"2022-11-22 12:52:48.000000000","message":"recheck","commit_id":"9a483f02bebfd0f1241e40fc98eb740496f78990"},{"author":{"_account_id":30314,"name":"Sebastian Lohff","email":"sebastian.lohff@sap.com","username":"seba"},"change_message_id":"14abbeea49f8bbccf8f932c9b0562e7b913144b7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"ad08516c_6cc53662","updated":"2022-11-21 15:37:47.000000000","message":"recheck","commit_id":"9a483f02bebfd0f1241e40fc98eb740496f78990"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"398a7eac8f305a8f644f24c5170b80c1ef259a35","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"d9b38867_d03203a6","updated":"2022-11-23 08:13:34.000000000","message":"recheck\nWith https://review.opendev.org/c/openstack/neutron/+/865295 merged trunk migrate test is skipped temporaryly\nAnother hit of https://bugs.launchpad.net/neutron/+bug/1997341","commit_id":"9a483f02bebfd0f1241e40fc98eb740496f78990"},{"author":{"_account_id":30314,"name":"Sebastian Lohff","email":"sebastian.lohff@sap.com","username":"seba"},"change_message_id":"4eadec742eb6e0bd0c13bffa986c97291a32f432","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"dd93c3e7_17d187eb","updated":"2022-11-21 15:05:57.000000000","message":"recheck arm64 worked before","commit_id":"9a483f02bebfd0f1241e40fc98eb740496f78990"},{"author":{"_account_id":30314,"name":"Sebastian Lohff","email":"sebastian.lohff@sap.com","username":"seba"},"change_message_id":"52a869662d8b1fcdb45753ed2cbd34f203023e3f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"1666fae6_10c1fdb1","updated":"2022-11-17 13:16:31.000000000","message":"recheck git clone failed","commit_id":"9a483f02bebfd0f1241e40fc98eb740496f78990"},{"author":{"_account_id":30314,"name":"Sebastian Lohff","email":"sebastian.lohff@sap.com","username":"seba"},"change_message_id":"3e4a817e0c3f55ef98e1c83fc1a73bf677518234","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"dd6b8f02_16202bf2","updated":"2022-11-21 13:21:17.000000000","message":"recheck not sure if last failure was a fluke","commit_id":"9a483f02bebfd0f1241e40fc98eb740496f78990"}],"neutron/plugins/ml2/drivers/ovn/mech_driver/mech_driver.py":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"398a7eac8f305a8f644f24c5170b80c1ef259a35","unresolved":true,"context_lines":[{"line_number":879,"context_line":"        \"\"\""},{"line_number":880,"context_line":"        port \u003d copy.deepcopy(context.current)"},{"line_number":881,"context_line":"        port[\u0027network\u0027] \u003d context.network.current"},{"line_number":882,"context_line":"        # FIXME(lucasagomes): PortContext does not have a session, therefore"},{"line_number":883,"context_line":"        # we need to use the _plugin_context attribute."},{"line_number":884,"context_line":"        self._ovn_client.delete_port(context._plugin_context, port[\u0027id\u0027],"},{"line_number":885,"context_line":"                                     port_object\u003dport)"},{"line_number":886,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"818328d4_e6811ee5","side":"PARENT","line":883,"range":{"start_line":882,"start_character":8,"end_line":883,"end_character":55},"updated":"2022-11-23 08:13:34.000000000","message":"why did you delete this comment?","commit_id":"bf44e70db6219e7f3a45bd61b7dd14a31ae33bb0"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"6ba39d71b6f060fca1f962fc22f3bc15933ec8cb","unresolved":true,"context_lines":[{"line_number":879,"context_line":"        \"\"\""},{"line_number":880,"context_line":"        port \u003d copy.deepcopy(context.current)"},{"line_number":881,"context_line":"        port[\u0027network\u0027] \u003d context.network.current"},{"line_number":882,"context_line":"        # FIXME(lucasagomes): PortContext does not have a session, therefore"},{"line_number":883,"context_line":"        # we need to use the _plugin_context attribute."},{"line_number":884,"context_line":"        self._ovn_client.delete_port(context._plugin_context, port[\u0027id\u0027],"},{"line_number":885,"context_line":"                                     port_object\u003dport)"},{"line_number":886,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"20701483_4aa34e5e","side":"PARENT","line":883,"range":{"start_line":882,"start_character":8,"end_line":883,"end_character":55},"in_reply_to":"817bd9db_dac1b34f","updated":"2022-11-23 17:45:07.000000000","message":"Makes sense.","commit_id":"bf44e70db6219e7f3a45bd61b7dd14a31ae33bb0"},{"author":{"_account_id":30314,"name":"Sebastian Lohff","email":"sebastian.lohff@sap.com","username":"seba"},"change_message_id":"42b042100ec7ea75652f7aa4d6a13d14a1821eaa","unresolved":true,"context_lines":[{"line_number":879,"context_line":"        \"\"\""},{"line_number":880,"context_line":"        port \u003d copy.deepcopy(context.current)"},{"line_number":881,"context_line":"        port[\u0027network\u0027] \u003d context.network.current"},{"line_number":882,"context_line":"        # FIXME(lucasagomes): PortContext does not have a session, therefore"},{"line_number":883,"context_line":"        # we need to use the _plugin_context attribute."},{"line_number":884,"context_line":"        self._ovn_client.delete_port(context._plugin_context, port[\u0027id\u0027],"},{"line_number":885,"context_line":"                                     port_object\u003dport)"},{"line_number":886,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"817bd9db_dac1b34f","side":"PARENT","line":883,"range":{"start_line":882,"start_character":8,"end_line":883,"end_character":55},"in_reply_to":"818328d4_e6811ee5","updated":"2022-11-23 13:07:39.000000000","message":"I understood this FIXME comment as saying \"the PortContext given to delete_port_postcommit() (the method we are in) does not have a DB session, therefore we need to get the session by accessing a private attribute of PortContext to get one\", i.e. as a justification as why we access a private member here. As this is no longer the case (the plugin_context is properly exposed) we no longer need to justify using it.","commit_id":"bf44e70db6219e7f3a45bd61b7dd14a31ae33bb0"}]}