)]}'
{"plugins/xenserver/xenapi/etc/xapi.d/plugins/glance":[{"author":{"_account_id":4463,"name":"iccha-sethi","email":"iccha.sethi@rackspace.com","username":"iccha-sethi"},"change_message_id":"0869d0e6641a900bca862e8755f4265a488cff6b","unresolved":false,"context_lines":[{"line_number":204,"context_line":"                      \"Response Status: %i, Response body: %s\""},{"line_number":205,"context_line":"                      % (url, resp.status, resp.read()))"},{"line_number":206,"context_line":""},{"line_number":207,"context_line":"        if resp.status \u003d\u003d httplib.UNAUTHORIZED:"},{"line_number":208,"context_line":"            # NOTE(johngarbutt): little point in retrying when token invalid"},{"line_number":209,"context_line":"            raise PluginError(\"Unauthorized error while uploading \""},{"line_number":210,"context_line":"                            \"image [%s] \""}],"source_content_type":"application/octet-stream","patch_set":2,"id":"AAAAP3%2F%2F2R8%3D","line":207,"updated":"2013-07-10 13:41:16.000000000","message":"Thanks this is awesome! There may be other cases we do not require retries as well. Not sure if that is out of scope of this patch. https://github.com/openstack/glance/blob/master/glance/api/v1/upload_utils.py#L123\nfor example the 412, 405, entity too large, etc.","commit_id":"74b0671c3e5d5e7f8335798e131676a65c850104"},{"author":{"_account_id":7780,"name":"Jared Culp","email":"jared.culp@rackspace.com","username":"jaredculp"},"change_message_id":"5d3dd394e2f74c8fd2402c45672b7e6f1c1dba43","unresolved":false,"context_lines":[{"line_number":204,"context_line":"                      \"Response Status: %i, Response body: %s\""},{"line_number":205,"context_line":"                      % (url, resp.status, resp.read()))"},{"line_number":206,"context_line":""},{"line_number":207,"context_line":"        if resp.status \u003d\u003d httplib.UNAUTHORIZED:"},{"line_number":208,"context_line":"            # NOTE(johngarbutt): little point in retrying when token invalid"},{"line_number":209,"context_line":"            raise PluginError(\"Unauthorized error while uploading \""},{"line_number":210,"context_line":"                            \"image [%s] \""}],"source_content_type":"application/octet-stream","patch_set":2,"id":"AAAAP3%2F%2F2RQ%3D","line":207,"in_reply_to":"AAAAP3%2F%2F2R8%3D","updated":"2013-07-10 13:44:08.000000000","message":"^ Agreed. Seems like there would be little point in retrying in these cases where failures are inevitable beyond the initial attempt.","commit_id":"74b0671c3e5d5e7f8335798e131676a65c850104"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"d001333627814baedfe37e690b78e3b191c3d19a","unresolved":false,"context_lines":[{"line_number":204,"context_line":"                      \"Response Status: %i, Response body: %s\""},{"line_number":205,"context_line":"                      % (url, resp.status, resp.read()))"},{"line_number":206,"context_line":""},{"line_number":207,"context_line":"        if resp.status \u003d\u003d httplib.UNAUTHORIZED:"},{"line_number":208,"context_line":"            # NOTE(johngarbutt): little point in retrying when token invalid"},{"line_number":209,"context_line":"            raise PluginError(\"Unauthorized error while uploading \""},{"line_number":210,"context_line":"                            \"image [%s] \""}],"source_content_type":"application/octet-stream","patch_set":2,"id":"AAAAP3%2F%2Fy5w%3D","line":207,"in_reply_to":"AAAAP3%2F%2F2RQ%3D","updated":"2013-07-11 10:02:34.000000000","message":"Yeah, I think there are other cases, I was really just worried about the move obvious one.\n\nNeed to work out which ones are errors we can retry with, and which ones mean we really are dead.","commit_id":"74b0671c3e5d5e7f8335798e131676a65c850104"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"0c6e78194d28aeec8f0cc3dc766c67bf4b53ce5f","unresolved":false,"context_lines":[{"line_number":204,"context_line":"                      \"Response Status: %i, Response body: %s\""},{"line_number":205,"context_line":"                      % (url, resp.status, resp.read()))"},{"line_number":206,"context_line":""},{"line_number":207,"context_line":"        if resp.status \u003d\u003d httplib.UNAUTHORIZED:"},{"line_number":208,"context_line":"            # NOTE(johngarbutt): little point in retrying when token invalid"},{"line_number":209,"context_line":"            raise PluginError(\"Unauthorized error while uploading \""},{"line_number":210,"context_line":"                            \"image [%s] \""}],"source_content_type":"application/octet-stream","patch_set":2,"id":"AAAAP3%2F%2Fy2s%3D","line":207,"in_reply_to":"AAAAP3%2F%2Fy5w%3D","updated":"2013-07-11 10:30:15.000000000","message":"I guess I also meant to say, the other errors can go in a different patch, if this goes in soon.","commit_id":"74b0671c3e5d5e7f8335798e131676a65c850104"}]}