)]}'
{"ceilometer/objectstore/swift.py":[{"author":{"_account_id":2813,"name":"Mehdi Abaakouk","email":"sileht@sileht.net","username":"sileht"},"change_message_id":"fd4563ba7ab1d73ee6c682315004c47b20d41356","unresolved":false,"context_lines":[{"line_number":62,"context_line":"        try:"},{"line_number":63,"context_line":"            endpoint \u003d ksclient.service_catalog.url_for("},{"line_number":64,"context_line":"                service_type\u003d\u0027object-store\u0027,"},{"line_number":65,"context_line":"                endpoint_type\u003dcfg.CONF.service_credentials.os_endpoint_type)"},{"line_number":66,"context_line":"        except exceptions.EndpointNotFound:"},{"line_number":67,"context_line":"            LOG.debug(_(\"Swift endpoint not found\"))"},{"line_number":68,"context_line":"            raise StopIteration()"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAQn%2F%2Frks%3D","line":65,"updated":"2013-07-25 06:18:00.000000000","message":"if we do this the \u0027_neaten_url\u0027 should be enhanced to remove the path of the endpoint url returned by the keystone catalog because is the endpoint selected if not the admin one, the endpoint url already contains \u0027v1/AUTH_...\u0027","commit_id":"d87a1c003733f08b65a2cc274b810cd27d2022ba"},{"author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"change_message_id":"e8002d5ee327b1fece2998282ec3b95f08468b13","unresolved":false,"context_lines":[{"line_number":62,"context_line":"        try:"},{"line_number":63,"context_line":"            endpoint \u003d ksclient.service_catalog.url_for("},{"line_number":64,"context_line":"                service_type\u003d\u0027object-store\u0027,"},{"line_number":65,"context_line":"                endpoint_type\u003dcfg.CONF.service_credentials.os_endpoint_type)"},{"line_number":66,"context_line":"        except exceptions.EndpointNotFound:"},{"line_number":67,"context_line":"            LOG.debug(_(\"Swift endpoint not found\"))"},{"line_number":68,"context_line":"            raise StopIteration()"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAQn%2F%2FrRE%3D","line":65,"in_reply_to":"AAAAQn%2F%2FrUA%3D","updated":"2013-07-25 09:38:16.000000000","message":"Actually reading neaten url code, it seems pretty solid since it uses urljoin. I\u0027ve added one more test to it in next patchset.","commit_id":"d87a1c003733f08b65a2cc274b810cd27d2022ba"},{"author":{"_account_id":2813,"name":"Mehdi Abaakouk","email":"sileht@sileht.net","username":"sileht"},"change_message_id":"4a525cbe1059694a0ca7b1b965f504a275ca2400","unresolved":false,"context_lines":[{"line_number":62,"context_line":"        try:"},{"line_number":63,"context_line":"            endpoint \u003d ksclient.service_catalog.url_for("},{"line_number":64,"context_line":"                service_type\u003d\u0027object-store\u0027,"},{"line_number":65,"context_line":"                endpoint_type\u003dcfg.CONF.service_credentials.os_endpoint_type)"},{"line_number":66,"context_line":"        except exceptions.EndpointNotFound:"},{"line_number":67,"context_line":"            LOG.debug(_(\"Swift endpoint not found\"))"},{"line_number":68,"context_line":"            raise StopIteration()"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAQn%2F%2FrUA%3D","line":65,"in_reply_to":"AAAAQn%2F%2FrXA%3D","updated":"2013-07-25 09:22:04.000000000","message":"yep patched #1 seems do the job, but not the merged one.","commit_id":"d87a1c003733f08b65a2cc274b810cd27d2022ba"},{"author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"change_message_id":"b3895027138b88981ce110dee0d096faff6b98ed","unresolved":false,"context_lines":[{"line_number":62,"context_line":"        try:"},{"line_number":63,"context_line":"            endpoint \u003d ksclient.service_catalog.url_for("},{"line_number":64,"context_line":"                service_type\u003d\u0027object-store\u0027,"},{"line_number":65,"context_line":"                endpoint_type\u003dcfg.CONF.service_credentials.os_endpoint_type)"},{"line_number":66,"context_line":"        except exceptions.EndpointNotFound:"},{"line_number":67,"context_line":"            LOG.debug(_(\"Swift endpoint not found\"))"},{"line_number":68,"context_line":"            raise StopIteration()"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAQn%2F%2FrXA%3D","line":65,"in_reply_to":"AAAAQn%2F%2Frks%3D","updated":"2013-07-25 08:50:33.000000000","message":"!?!?!\n\nI am pretty sure someone already proposed a patch to enhance neaten_url and though it was merged…\n\n\nSearching:\n\nhttps://review.openstack.org/#/c/31169/\n\nThe patchset #3 has been merged whereas I wanted the #1 to get back, /me cries","commit_id":"d87a1c003733f08b65a2cc274b810cd27d2022ba"}]}