)]}'
{"nova/virt/xenapi/imageupload/glance.py":[{"author":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"change_message_id":"82d35727cdab135df7b2e6c78348119211a1a266","unresolved":false,"context_lines":[{"line_number":51,"context_line":"                 glance_port,"},{"line_number":52,"context_line":"                 glance_use_ssl) \u003d glance_api_servers.next()"},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"                properties \u003d {"},{"line_number":55,"context_line":"                    \u0027auto_disk_config\u0027: instance[\u0027auto_disk_config\u0027],"},{"line_number":56,"context_line":"                    \u0027os_type\u0027: instance[\u0027os_type\u0027] or CONF.default_os_type,"},{"line_number":57,"context_line":"                }"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAM3%2F%2BjAg%3D","line":54,"updated":"2013-02-27 20:41:19.000000000","message":"This can be set outside of the loop... since it doesn\u0027t change.","commit_id":"8044130b6da8ff8c551334a2ec109af788e5bdea"},{"author":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"change_message_id":"82d35727cdab135df7b2e6c78348119211a1a266","unresolved":false,"context_lines":[{"line_number":55,"context_line":"                    \u0027auto_disk_config\u0027: instance[\u0027auto_disk_config\u0027],"},{"line_number":56,"context_line":"                    \u0027os_type\u0027: instance[\u0027os_type\u0027] or CONF.default_os_type,"},{"line_number":57,"context_line":"                }"},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"                params \u003d {\u0027vdi_uuids\u0027: vdi_uuids,"},{"line_number":60,"context_line":"                          \u0027image_id\u0027: image_id,"},{"line_number":61,"context_line":"                          \u0027glance_host\u0027: glance_host,"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAM3%2F%2BjA4%3D","line":58,"updated":"2013-02-27 20:41:19.000000000","message":"It\u0027d be useful to move the log message to here as well, and include the glance host/port so we know which one is being attempted.  Also perhaps include the \u0027attempt_num\u0027.... like we do for download_vhd.\n\nWould be really nice to have those in the exception msg too.","commit_id":"8044130b6da8ff8c551334a2ec109af788e5bdea"},{"author":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"change_message_id":"82d35727cdab135df7b2e6c78348119211a1a266","unresolved":false,"context_lines":[{"line_number":70,"context_line":"                                                      **params)"},{"line_number":71,"context_line":"            except session.XenAPI.Failure as exc:"},{"line_number":72,"context_line":"                _type, _method, error \u003d exc.details[:3]"},{"line_number":73,"context_line":"                if error \u003d\u003d \u0027RetryableError\u0027:"},{"line_number":74,"context_line":"                    LOG.error(_(\u0027upload_vhd failed: %r\u0027) %"},{"line_number":75,"context_line":"                              (exc.details[3:],))"},{"line_number":76,"context_line":"                else:"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAM3%2F%2BjAQ%3D","line":73,"updated":"2013-02-27 20:41:19.000000000","message":"Currently in the plugin, it doesn\u0027t look like anything will raise RetryableError.  Looks like you\u0027ll need to make the plugin raise this appropriately...  probably in the _upload_tarball() method..","commit_id":"8044130b6da8ff8c551334a2ec109af788e5bdea"}]}