)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"change_message_id":"785c3b5b499f2b67cb9c99747a51f59815d9b9cf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"db784c28_33e1614b","updated":"2024-11-08 15:42:53.000000000","message":"Thank @sowmya.kamavaram@rackspace.com for putting this together. The fix is working great in my lab.","commit_id":"be507d0e99d1ede9a19158a1d937488a45945c7b"}],"heat/engine/clients/os/keystone/heat_keystoneclient.py":[{"author":{"_account_id":8833,"name":"Rabi Mishra","email":"ramishra@redhat.com","username":"rabi"},"change_message_id":"9478ac4abe9b82ad9d6e9976f9c6fdf8067a7e92","unresolved":true,"context_lines":[{"line_number":600,"context_line":"                    auth_ref \u003d self.context.auth_plugin.get_access("},{"line_number":601,"context_line":"                        self.session)"},{"line_number":602,"context_line":"                    if hasattr(auth_ref, \"service_catalog\"):"},{"line_number":603,"context_line":"                        unversioned_sc_auth_uri \u003d ("},{"line_number":604,"context_line":"                            auth_ref.service_catalog.get_urls("},{"line_number":605,"context_line":"                                service_type\u003d\u0027identity\u0027,"},{"line_number":606,"context_line":"                                interface\u003dks_endpoint_type))"},{"line_number":607,"context_line":"                        if len(unversioned_sc_auth_uri) \u003e 0:"},{"line_number":608,"context_line":"                            sc_auth_uri \u003d ("},{"line_number":609,"context_line":"                                unversioned_sc_auth_uri[0] + \"/v3\")"},{"line_number":610,"context_line":"                            return sc_auth_uri"},{"line_number":611,"context_line":"                except ks_exception.Unauthorized:"},{"line_number":612,"context_line":"                    LOG.error(\"Keystone client authentication failed\")"},{"line_number":613,"context_line":"        return fallback_endpoint"}],"source_content_type":"text/x-python","patch_set":1,"id":"15653dcc_70e496c2","side":"PARENT","line":610,"range":{"start_line":603,"start_character":0,"end_line":610,"end_character":46},"updated":"2024-11-13 13:51:28.000000000","message":"How was this working all this time? I think the assumption was catalog always contains unversioned endpoints. Maybe something changed when we dropped v2 support in keystone?\n\nAs per https://docs.openstack.org/keystone/latest/contributor/http-api.html#i-am-a-deployer it\u0027s only ideal.","commit_id":"dbc298b4ccc8f776791745496ff303387ad88f24"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"8f5f36993f975b212dd1e18a3929522986b9cbd3","unresolved":true,"context_lines":[{"line_number":600,"context_line":"                    auth_ref \u003d self.context.auth_plugin.get_access("},{"line_number":601,"context_line":"                        self.session)"},{"line_number":602,"context_line":"                    if hasattr(auth_ref, \"service_catalog\"):"},{"line_number":603,"context_line":"                        unversioned_sc_auth_uri \u003d ("},{"line_number":604,"context_line":"                            auth_ref.service_catalog.get_urls("},{"line_number":605,"context_line":"                                service_type\u003d\u0027identity\u0027,"},{"line_number":606,"context_line":"                                interface\u003dks_endpoint_type))"},{"line_number":607,"context_line":"                        if len(unversioned_sc_auth_uri) \u003e 0:"},{"line_number":608,"context_line":"                            sc_auth_uri \u003d ("},{"line_number":609,"context_line":"                                unversioned_sc_auth_uri[0] + \"/v3\")"},{"line_number":610,"context_line":"                            return sc_auth_uri"},{"line_number":611,"context_line":"                except ks_exception.Unauthorized:"},{"line_number":612,"context_line":"                    LOG.error(\"Keystone client authentication failed\")"},{"line_number":613,"context_line":"        return fallback_endpoint"}],"source_content_type":"text/x-python","patch_set":1,"id":"db3e73e9_d1e3dda5","side":"PARENT","line":610,"range":{"start_line":603,"start_character":0,"end_line":610,"end_character":46},"in_reply_to":"15653dcc_70e496c2","updated":"2024-11-13 13:56:40.000000000","message":"IIUC endpoint detection is supposed to work with/without version url. I don\u0027t think it\u0027s popular to include version in keystone endpoint url in catalog but it\u0027s still a valid way technically.","commit_id":"dbc298b4ccc8f776791745496ff303387ad88f24"}]}