)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"c3a98f5a51b07824aa40d85899e723f351361ac2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"3deaa625_525543ad","updated":"2026-06-18 07:14:07.000000000","message":"I think it will break the tempest tests for 3rd party providers (ovn-provider), octavia-tempest-plugin uses the return code to skip not implemented features","commit_id":"3711fc91a92280b8c58affaf5b1478222f085f3c"},{"author":{"_account_id":38562,"name":"Richard Cruise","email":"rcruise@redhat.com","username":"rcruise"},"change_message_id":"b8115bc898073ad81d049c03ac5018bab271d300","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"b10b514d_2a82035b","updated":"2026-06-18 08:17:31.000000000","message":"I\u0027m marking as -1 because this definitely needs a release note. Can you add one describing the API response changes?","commit_id":"3711fc91a92280b8c58affaf5b1478222f085f3c"},{"author":{"_account_id":34451,"name":"Fernando Royo","email":"froyo@redhat.com","username":"froyo"},"change_message_id":"911a773a9ac3ccb783b0057bbaf5d0947bd04d9b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"7770a991_c458d0d2","updated":"2026-06-18 13:56:17.000000000","message":"Yeah, 5XX error codes are used as a flag for NotImplemented responses: https://github.com/openstack/octavia-tempest-plugin/blob/bb4776fabba145a03ea0b0df379dcd63d1a26a90/octavia_tempest_plugin/services/load_balancer/v2/flavor_client.py#L52. They are used by the Tempest tests to skip failed tests, and all providers use them in some cases.","commit_id":"0fc79ece9815495f7b105ccad9faadf4d3c8d874"},{"author":{"_account_id":3031,"name":"Sam Morrison","email":"sorrison@gmail.com","username":"sorrison"},"change_message_id":"700449e99fc0c7617ed4e3e233f0689648de8ca6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"95adb44e_7b517e85","in_reply_to":"7770a991_c458d0d2","updated":"2026-06-19 02:52:54.000000000","message":"Yeah fair enough, check out https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/994029 to see if that is a good approach, I struggled to think of a nice solution here that is backwards compatible but I haven\u0027t hacked on tempest in a while so might be a better way","commit_id":"0fc79ece9815495f7b105ccad9faadf4d3c8d874"}],"octavia/common/exceptions.py":[{"author":{"_account_id":38562,"name":"Richard Cruise","email":"rcruise@redhat.com","username":"rcruise"},"change_message_id":"b8115bc898073ad81d049c03ac5018bab271d300","unresolved":true,"context_lines":[{"line_number":376,"context_line":""},{"line_number":377,"context_line":"class ProviderNotFound(APIException):"},{"line_number":378,"context_line":"    msg \u003d _(\"Provider \u0027%(prov)s\u0027 was not found.\")"},{"line_number":379,"context_line":"    code \u003d 501"},{"line_number":380,"context_line":""},{"line_number":381,"context_line":""},{"line_number":382,"context_line":"class ProviderDriverError(APIException):"}],"source_content_type":"text/x-python","patch_set":6,"id":"ac53e4a5_493c51fe","line":379,"updated":"2026-06-18 08:17:31.000000000","message":"While we\u0027re at it, perhaps this should also be a 4XX error?","commit_id":"3711fc91a92280b8c58affaf5b1478222f085f3c"}]}