)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"dcdb44e69dcfc348891a298a76d21486f7622670","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"9739e4a0_7ada059f","updated":"2024-01-13 14:34:26.000000000","message":"1) Please present this patch but for master branch.\n2) The backports, as commented in the change, should include this condition but not the master branch.\n3) Please use the LP bug opened: https://bugs.launchpad.net/neutron/+bug/2049265","commit_id":"ed08cac71a673b38a998150241150e48a4bc9b4c"},{"author":{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"change_message_id":"dc98f16ceaa0f607855d4e5c77438dda06f23c44","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"53324b8c_9d2e56b4","in_reply_to":"9739e4a0_7ada059f","updated":"2024-01-13 17:45:06.000000000","message":"1 \u0026 2) It shouldn\u0027t need to be in the master branch because of https://review.opendev.org/c/openstack/neutron/+/884898/10/neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/ovsdb_monitor.py#672\n3) Done.","commit_id":"ed08cac71a673b38a998150241150e48a4bc9b4c"},{"author":{"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},"change_message_id":"1246a850a8ed116b5a9e8e60a7df86a4a229341a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"c2ce9e42_293ea6ef","updated":"2024-01-15 11:11:19.000000000","message":"Looking good per Terry\u0027s comments for master","commit_id":"60eb15ed301751c3f6825f165fd36188e86dd642"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"938252566fc3659d1144c767e0857865f51b2c8a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"55e7b7a2_bdb3bfa3","updated":"2024-01-15 07:30:37.000000000","message":"We\u0027ll need backports up to Wallaby","commit_id":"60eb15ed301751c3f6825f165fd36188e86dd642"}],"neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/ovsdb_monitor.py":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"dcdb44e69dcfc348891a298a76d21486f7622670","unresolved":true,"context_lines":[{"line_number":670,"context_line":"class BaseOvnSbIdl(Ml2OvnIdlBase):"},{"line_number":671,"context_line":"    @classmethod"},{"line_number":672,"context_line":"    def from_server(cls, connection_string, helper):"},{"line_number":673,"context_line":"        if \u0027Chassis_Private\u0027 in helper.schema_json[\u0027tables\u0027]:"},{"line_number":674,"context_line":"            helper.register_table(\u0027Chassis_Private\u0027)"},{"line_number":675,"context_line":"        helper.register_table(\u0027Chassis\u0027)"},{"line_number":676,"context_line":"        helper.register_table(\u0027Encap\u0027)"},{"line_number":677,"context_line":"        helper.register_table(\u0027Port_Binding\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"ed57fb43_65226bcf","line":674,"range":{"start_line":673,"start_character":9,"end_line":674,"end_character":52},"updated":"2024-01-13 14:34:26.000000000","message":"Since Caracal (this version), we consider that \"Chassis_Private\" is always present.\n\nThis condition will be needed in the backports.","commit_id":"ed08cac71a673b38a998150241150e48a4bc9b4c"},{"author":{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"change_message_id":"dc98f16ceaa0f607855d4e5c77438dda06f23c44","unresolved":false,"context_lines":[{"line_number":670,"context_line":"class BaseOvnSbIdl(Ml2OvnIdlBase):"},{"line_number":671,"context_line":"    @classmethod"},{"line_number":672,"context_line":"    def from_server(cls, connection_string, helper):"},{"line_number":673,"context_line":"        if \u0027Chassis_Private\u0027 in helper.schema_json[\u0027tables\u0027]:"},{"line_number":674,"context_line":"            helper.register_table(\u0027Chassis_Private\u0027)"},{"line_number":675,"context_line":"        helper.register_table(\u0027Chassis\u0027)"},{"line_number":676,"context_line":"        helper.register_table(\u0027Encap\u0027)"},{"line_number":677,"context_line":"        helper.register_table(\u0027Port_Binding\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"060cdc96_12573bae","line":674,"range":{"start_line":673,"start_character":9,"end_line":674,"end_character":52},"in_reply_to":"ed57fb43_65226bcf","updated":"2024-01-13 17:45:06.000000000","message":"Caracal/master doesn\u0027t need this patch at all, so we\u0027ll just have this in backports as-is.","commit_id":"ed08cac71a673b38a998150241150e48a4bc9b4c"}]}