)]}'
{"nova/image/glance.py":[{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"3d0127cdd083d5b1325dbf86d2c98488a0d6c961","unresolved":false,"context_lines":[{"line_number":187,"context_line":""},{"line_number":188,"context_line":"    configured_servers \u003d ([\u0027%s:%s\u0027 % (CONF.glance.host, CONF.glance.port)]"},{"line_number":189,"context_line":"                          if CONF.glance.api_servers is None"},{"line_number":190,"context_line":"                          else CONF.glance.api_servers)"},{"line_number":191,"context_line":"    for api_server in configured_servers:"},{"line_number":192,"context_line":"        if \u0027//\u0027 not in api_server:"},{"line_number":193,"context_line":"            api_server \u003d \u0027http://\u0027 + api_server"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a8ffd7b_24d29e35","line":190,"updated":"2015-11-30 17:11:12.000000000","message":"note that the way most people configure glance servers is glance.host / glance.port, which means this fix mostly won\u0027t be accessible to people.\n\nAll the conf options above need to get deprecated and us use the catalog entries instead out of the context.","commit_id":"54ab181929fa29eb5dca9d10f6008cce696c53c2"},{"author":{"_account_id":2903,"name":"Morgan Fainberg","email":"morgan.fainberg@gmail.com","username":"mdrnstm"},"change_message_id":"fa9a9fe99700a312373fbf151cc1ceb9e828889f","unresolved":false,"context_lines":[{"line_number":187,"context_line":""},{"line_number":188,"context_line":"    configured_servers \u003d ([\u0027%s:%s\u0027 % (CONF.glance.host, CONF.glance.port)]"},{"line_number":189,"context_line":"                          if CONF.glance.api_servers is None"},{"line_number":190,"context_line":"                          else CONF.glance.api_servers)"},{"line_number":191,"context_line":"    for api_server in configured_servers:"},{"line_number":192,"context_line":"        if \u0027//\u0027 not in api_server:"},{"line_number":193,"context_line":"            api_server \u003d \u0027http://\u0027 + api_server"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a8ffd7b_19e541a7","line":190,"in_reply_to":"9a8ffd7b_24d29e35","updated":"2015-12-01 16:35:03.000000000","message":"The move to the catalog is going to be both out of my depth without sinking days into chasing down all the ways that nova assumes host/port/version/use_ssl (including the Xen plugin).\n\nI\u0027m happy to address the issue with Xen plugin and add an option to allow for a path-prefix in the way you\u0027ve described, but I don\u0027t think I can do the catalog conversion in the timewindow that I\u0027ve allocated (days to chase down nova side of things is just not on the table right now).\n\nSee my comment on the patchset, I\u0027m going to argue that not accepting a smaller fix here (and it will be smaller to not convert to catalog yet) is perfect being the enemy of good.","commit_id":"54ab181929fa29eb5dca9d10f6008cce696c53c2"}],"nova/virt/xenapi/image/glance.py":[{"author":{"_account_id":2903,"name":"Morgan Fainberg","email":"morgan.fainberg@gmail.com","username":"mdrnstm"},"change_message_id":"cf83c9ccfe72d19571d8508d4e93311465fdf6ba","unresolved":false,"context_lines":[{"line_number":36,"context_line":"        glance_api_servers \u003d glance.get_api_servers()"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"        def pick_glance(kwargs):"},{"line_number":39,"context_line":"            g_host, g_port, g_use_ssl, g_path_prefix \u003d next(glance_api_servers)"},{"line_number":40,"context_line":"            kwargs[\u0027glance_host\u0027] \u003d g_host"},{"line_number":41,"context_line":"            kwargs[\u0027glance_port\u0027] \u003d g_port"},{"line_number":42,"context_line":"            kwargs[\u0027glance_use_ssl\u0027] \u003d g_use_ssl"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a8ffd7b_cde0c12b","line":39,"updated":"2015-11-29 14:57:38.000000000","message":"Is it possible to pass the prefix down to glance client? For now this is just eating the value so we don\u0027t have \"too many values to unpack\"","commit_id":"ac185f3698978f26f534d3a11b1dc50b3db8723c"}]}