)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":24245,"name":"Harald Jensås","email":"hjensas@redhat.com","username":"harald.jensas"},"change_message_id":"d2dc173e6c1030c7bfc9b4944615a799700a5878","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"ed4f3b26_1181842b","updated":"2026-03-24 15:53:17.000000000","message":"recheck\n\nUbuntu package download issue.","commit_id":"13df8a996d258b83067c98bf7fbf7fe1c16525b0"},{"author":{"_account_id":24245,"name":"Harald Jensås","email":"hjensas@redhat.com","username":"harald.jensas"},"change_message_id":"a53b8c888daaec394fa01392bcd22ea3d74a8939","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"c0b17f82_98afc143","updated":"2026-03-24 11:13:29.000000000","message":"recheck\n\nnetworking-baremetal-multitenant-vlans POST_FAILURE - looks like debian mirror issue.","commit_id":"13df8a996d258b83067c98bf7fbf7fe1c16525b0"}],"networking_baremetal/agent/l2vni_trunk_manager.py":[{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"1fed7375629daf6f644680c4b9bb2a9627d3c428","unresolved":true,"context_lines":[{"line_number":806,"context_line":"                return False"},{"line_number":807,"context_line":""},{"line_number":808,"context_line":"            if \u0027Logical_Switch\u0027 not in self.ovn_nb_idl.tables:"},{"line_number":809,"context_line":"                return False"},{"line_number":810,"context_line":""},{"line_number":811,"context_line":"            lsp_table \u003d self.ovn_nb_idl.tables[\u0027Logical_Switch_Port\u0027]"},{"line_number":812,"context_line":"            ls_table \u003d self.ovn_nb_idl.tables[\u0027Logical_Switch\u0027]"}],"source_content_type":"text/x-python","patch_set":2,"id":"a90e1200_15b76262","line":809,"updated":"2026-03-24 03:04:59.000000000","message":"It feels like these should be constants.","commit_id":"13df8a996d258b83067c98bf7fbf7fe1c16525b0"},{"author":{"_account_id":24245,"name":"Harald Jensås","email":"hjensas@redhat.com","username":"harald.jensas"},"change_message_id":"042e0df493446922ea81bbb406fc2f908adeb7a3","unresolved":true,"context_lines":[{"line_number":806,"context_line":"                return False"},{"line_number":807,"context_line":""},{"line_number":808,"context_line":"            if \u0027Logical_Switch\u0027 not in self.ovn_nb_idl.tables:"},{"line_number":809,"context_line":"                return False"},{"line_number":810,"context_line":""},{"line_number":811,"context_line":"            lsp_table \u003d self.ovn_nb_idl.tables[\u0027Logical_Switch_Port\u0027]"},{"line_number":812,"context_line":"            ls_table \u003d self.ovn_nb_idl.tables[\u0027Logical_Switch\u0027]"}],"source_content_type":"text/x-python","patch_set":2,"id":"3ff8b7e2_de1c44f8","line":809,"in_reply_to":"a90e1200_15b76262","updated":"2026-03-24 11:12:34.000000000","message":"The \u0027Logical_Switch_Port\u0027 and \u0027Logical_Switch\u0027 ? I had the same feeling, I asked Claude to see if neutron already have constants that we could import. It then told me neutron code does not have constants for them, they use the table names as done here. So I decided to stay with the same pattern.","commit_id":"13df8a996d258b83067c98bf7fbf7fe1c16525b0"}]}