)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"3ed581a0d5466c32c2ff54c18d2c027b5dac1c59","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"8ad0e206_fffc86ba","updated":"2025-11-17 14:17:04.000000000","message":"I\u0027m merging this because this has been blocking backport of https://review.opendev.org/c/openstack/oslo.limit/+/961329 ... We can discuss deprecation of *URL separately.","commit_id":"b8452d7bff17318fb841952ca13ebe48a932d0c1"}],"oslo_limit/limit.py":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"6c3c65bdd9faefd154dc3f0601d5876c6976370a","unresolved":true,"context_lines":[{"line_number":326,"context_line":"            region_id \u003d None"},{"line_number":327,"context_line":""},{"line_number":328,"context_line":"        interface \u003d CONF.oslo_limit.endpoint_interface"},{"line_number":329,"context_line":"        if interface.endswith(\u0027URL\u0027):"},{"line_number":330,"context_line":"            interface \u003d interface[:-3]"},{"line_number":331,"context_line":"        try:"},{"line_number":332,"context_line":"            endpoints \u003d self.connection.endpoints("}],"source_content_type":"text/x-python","patch_set":2,"id":"ec3893d3_4cc19c0f","line":329,"updated":"2025-09-17 16:55:55.000000000","message":"Can we log a warning so that we can move away from this down the line? People shouldn\u0027t still be using this Keystone v2 terminology in their deployments. We probably want a note about why we\u0027re doing this also.","commit_id":"b8452d7bff17318fb841952ca13ebe48a932d0c1"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"8e38bc6b8c791c4f4a6828aa1822288a77b6fc6a","unresolved":true,"context_lines":[{"line_number":326,"context_line":"            region_id \u003d None"},{"line_number":327,"context_line":""},{"line_number":328,"context_line":"        interface \u003d CONF.oslo_limit.endpoint_interface"},{"line_number":329,"context_line":"        if interface.endswith(\u0027URL\u0027):"},{"line_number":330,"context_line":"            interface \u003d interface[:-3]"},{"line_number":331,"context_line":"        try:"},{"line_number":332,"context_line":"            endpoints \u003d self.connection.endpoints("}],"source_content_type":"text/x-python","patch_set":2,"id":"d6cffcd6_e7060401","line":329,"in_reply_to":"ec3893d3_4cc19c0f","updated":"2025-09-17 17:03:04.000000000","message":"I aim to backport this along with the previous fix to stable branches so am hesitant to include that part (which we may not backport). Are you ok if I address it in a follow-up ?\n\nAlso there are too many places where we do this kind of trick (due to implicit conversion in keystoneauth). So removing the whole *URL support may become a bit bigger topic.","commit_id":"b8452d7bff17318fb841952ca13ebe48a932d0c1"}]}