)]}'
{"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":"f90d360f5bbb1fe244462f9d39ffa6dd6194617a","unresolved":false,"context_lines":[{"line_number":207,"context_line":"        if resp.status in (httplib.UNAUTHORIZED,"},{"line_number":208,"context_line":"                           httplib.REQUEST_ENTITY_TOO_LARGE,"},{"line_number":209,"context_line":"                           httplib.PRECONDITION_FAILED,"},{"line_number":210,"context_line":"                           httplib.CONFLICT):"},{"line_number":211,"context_line":"            # No point in retrying for these conditions"},{"line_number":212,"context_line":"            raise PluginError(\"Got Error response [%i] while uploading \""},{"line_number":213,"context_line":"                            \"image [%s] \""}],"source_content_type":"application/octet-stream","patch_set":1,"id":"AAAAQn%2F%2FihE%3D","line":210,"updated":"2013-07-29 15:39:32.000000000","message":"is there any reason https://github.com/openstack/glance/blob/master/glance/api/v1/upload_utils.py#L153 Forbidden was excluded?","commit_id":"ad5702fa819fc30d3a3de789a5d40ba64e83d507"},{"author":{"_account_id":4340,"name":"Sulochan Acharya","email":"sulochan@gmail.com","username":"sulochan-acharya"},"change_message_id":"a923c952528d23d8989e731fa0feb399e0fcb1b6","unresolved":false,"context_lines":[{"line_number":207,"context_line":"        if resp.status in (httplib.UNAUTHORIZED,"},{"line_number":208,"context_line":"                           httplib.REQUEST_ENTITY_TOO_LARGE,"},{"line_number":209,"context_line":"                           httplib.PRECONDITION_FAILED,"},{"line_number":210,"context_line":"                           httplib.CONFLICT):"},{"line_number":211,"context_line":"            # No point in retrying for these conditions"},{"line_number":212,"context_line":"            raise PluginError(\"Got Error response [%i] while uploading \""},{"line_number":213,"context_line":"                            \"image [%s] \""}],"source_content_type":"application/octet-stream","patch_set":1,"id":"AAAAQn%2F%2FiUQ%3D","line":210,"in_reply_to":"AAAAQn%2F%2FihE%3D","updated":"2013-07-29 16:15:14.000000000","message":"Including it now. Thanks!","commit_id":"ad5702fa819fc30d3a3de789a5d40ba64e83d507"},{"author":{"_account_id":2537,"name":"Nikhil Komawar","email":"nik.komawar@gmail.com","username":"nikhil-komawar"},"change_message_id":"93828e215d923a2cba97ff23c99bdb7a0e26756f","unresolved":false,"context_lines":[{"line_number":207,"context_line":"        if resp.status in (httplib.UNAUTHORIZED,"},{"line_number":208,"context_line":"                           httplib.REQUEST_ENTITY_TOO_LARGE,"},{"line_number":209,"context_line":"                           httplib.PRECONDITION_FAILED,"},{"line_number":210,"context_line":"                           httplib.CONFLICT):"},{"line_number":211,"context_line":"            # No point in retrying for these conditions"},{"line_number":212,"context_line":"            raise PluginError(\"Got Error response [%i] while uploading \""},{"line_number":213,"context_line":"                            \"image [%s] \""}],"source_content_type":"application/octet-stream","patch_set":1,"id":"AAAAQn%2F%2FiYg%3D","line":210,"in_reply_to":"AAAAQn%2F%2FihE%3D","updated":"2013-07-29 16:05:19.000000000","message":"agree","commit_id":"ad5702fa819fc30d3a3de789a5d40ba64e83d507"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"c3caea24522e06b7c722623505c3db75f2ff063b","unresolved":false,"context_lines":[{"line_number":205,"context_line":"                      % (url, resp.status, resp.read()))"},{"line_number":206,"context_line":""},{"line_number":207,"context_line":"        if resp.status in (httplib.UNAUTHORIZED,"},{"line_number":208,"context_line":"                           httplib.REQUEST_ENTITY_TOO_LARGE,"},{"line_number":209,"context_line":"                           httplib.PRECONDITION_FAILED,"},{"line_number":210,"context_line":"                           httplib.CONFLICT,"},{"line_number":211,"context_line":"                           httplib.FORBIDDEN):"}],"source_content_type":"application/octet-stream","patch_set":2,"id":"AAAAQn%2F%2Fb6U%3D","line":208,"updated":"2013-07-31 08:37:40.000000000","message":"unless the font is a bit funny, this should really be indented one more space, so the h lines up.","commit_id":"6c8661d7390c5a614ac5d63bfc9442870849e977"},{"author":{"_account_id":4340,"name":"Sulochan Acharya","email":"sulochan@gmail.com","username":"sulochan-acharya"},"change_message_id":"08919b1f33b7cee7d5a937b2843ba31ccbb8c854","unresolved":false,"context_lines":[{"line_number":205,"context_line":"                      % (url, resp.status, resp.read()))"},{"line_number":206,"context_line":""},{"line_number":207,"context_line":"        if resp.status in (httplib.UNAUTHORIZED,"},{"line_number":208,"context_line":"                           httplib.REQUEST_ENTITY_TOO_LARGE,"},{"line_number":209,"context_line":"                           httplib.PRECONDITION_FAILED,"},{"line_number":210,"context_line":"                           httplib.CONFLICT,"},{"line_number":211,"context_line":"                           httplib.FORBIDDEN):"}],"source_content_type":"application/octet-stream","patch_set":2,"id":"AAAAQn%2F%2FZFU%3D","line":208,"in_reply_to":"AAAAQn%2F%2Fb6U%3D","updated":"2013-07-31 21:42:02.000000000","message":"yeah, definitely missed one space here. Fixing.","commit_id":"6c8661d7390c5a614ac5d63bfc9442870849e977"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"c3caea24522e06b7c722623505c3db75f2ff063b","unresolved":false,"context_lines":[{"line_number":211,"context_line":"                           httplib.FORBIDDEN):"},{"line_number":212,"context_line":"            # No point in retrying for these conditions"},{"line_number":213,"context_line":"            raise PluginError(\"Got Error response [%i] while uploading \""},{"line_number":214,"context_line":"                            \"image [%s] \""},{"line_number":215,"context_line":"                            \"to glance host [%s:%s]\""},{"line_number":216,"context_line":"                            % (resp.status, image_id, glance_host, glance_port))"},{"line_number":217,"context_line":"        else:"}],"source_content_type":"application/octet-stream","patch_set":2,"id":"AAAAQn%2F%2Fb6M%3D","line":214,"updated":"2013-07-31 08:37:40.000000000","message":"my bad on this one, this should be indented a bit more too.","commit_id":"6c8661d7390c5a614ac5d63bfc9442870849e977"},{"author":{"_account_id":4340,"name":"Sulochan Acharya","email":"sulochan@gmail.com","username":"sulochan-acharya"},"change_message_id":"08919b1f33b7cee7d5a937b2843ba31ccbb8c854","unresolved":false,"context_lines":[{"line_number":211,"context_line":"                           httplib.FORBIDDEN):"},{"line_number":212,"context_line":"            # No point in retrying for these conditions"},{"line_number":213,"context_line":"            raise PluginError(\"Got Error response [%i] while uploading \""},{"line_number":214,"context_line":"                            \"image [%s] \""},{"line_number":215,"context_line":"                            \"to glance host [%s:%s]\""},{"line_number":216,"context_line":"                            % (resp.status, image_id, glance_host, glance_port))"},{"line_number":217,"context_line":"        else:"}],"source_content_type":"application/octet-stream","patch_set":2,"id":"AAAAQn%2F%2FZFM%3D","line":214,"in_reply_to":"AAAAQn%2F%2Fb6M%3D","updated":"2013-07-31 21:42:02.000000000","message":"fixing it here also.","commit_id":"6c8661d7390c5a614ac5d63bfc9442870849e977"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"c3caea24522e06b7c722623505c3db75f2ff063b","unresolved":false,"context_lines":[{"line_number":213,"context_line":"            raise PluginError(\"Got Error response [%i] while uploading \""},{"line_number":214,"context_line":"                            \"image [%s] \""},{"line_number":215,"context_line":"                            \"to glance host [%s:%s]\""},{"line_number":216,"context_line":"                            % (resp.status, image_id, glance_host, glance_port))"},{"line_number":217,"context_line":"        else:"},{"line_number":218,"context_line":"            raise RetryableError(\"Unexpected response [%i] while uploading \""},{"line_number":219,"context_line":"                            \"image [%s] \""}],"source_content_type":"application/octet-stream","patch_set":2,"id":"AAAAQn%2F%2Fb6E%3D","line":216,"updated":"2013-07-31 08:37:40.000000000","message":"we should really start using a dict here, but we don\u0027t have localization anyways, so its not a big deal.","commit_id":"6c8661d7390c5a614ac5d63bfc9442870849e977"},{"author":{"_account_id":5441,"name":"Andrew Laski","email":"andrew@lascii.com","username":"alaski"},"change_message_id":"0788b38b603de8e412b562cfbbce0af90b11766e","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 in (httplib.UNAUTHORIZED,"},{"line_number":208,"context_line":"                           httplib.REQUEST_ENTITY_TOO_LARGE,"},{"line_number":209,"context_line":"                           httplib.PRECONDITION_FAILED,"},{"line_number":210,"context_line":"                           httplib.CONFLICT,"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"AAAAQn%2F%2FBFQ%3D","line":207,"updated":"2013-08-08 12:46:43.000000000","message":"I\u0027m not too concerned about the whitespace here because you\u0027re modifying this code, and everything lines up.","commit_id":"7aa6f8465ca4d0aed656a1faf271a597fbdb6f05"},{"author":{"_account_id":4340,"name":"Sulochan Acharya","email":"sulochan@gmail.com","username":"sulochan-acharya"},"change_message_id":"c89aabf3c19e2499c694a5bc8b8ae919a0943f85","unresolved":false,"context_lines":[{"line_number":208,"context_line":"                           httplib.REQUEST_ENTITY_TOO_LARGE,"},{"line_number":209,"context_line":"                           httplib.PRECONDITION_FAILED,"},{"line_number":210,"context_line":"                           httplib.CONFLICT,"},{"line_number":211,"context_line":"                           httplib.FORBIDDEN):"},{"line_number":212,"context_line":"            # No point in retrying for these conditions"},{"line_number":213,"context_line":"            raise PluginError(\"Got Error response [%i] while uploading \""},{"line_number":214,"context_line":"                              \"image [%s] \""}],"source_content_type":"application/octet-stream","patch_set":3,"id":"AAAAQn%2F%2FZBs%3D","line":211,"updated":"2013-07-31 21:58:48.000000000","message":"Not sure whats making the indentation look funny here, it looks ok on my editor.","commit_id":"7aa6f8465ca4d0aed656a1faf271a597fbdb6f05"},{"author":{"_account_id":5441,"name":"Andrew Laski","email":"andrew@lascii.com","username":"alaski"},"change_message_id":"0788b38b603de8e412b562cfbbce0af90b11766e","unresolved":false,"context_lines":[{"line_number":215,"context_line":"                              \"to glance host [%s:%s]\""},{"line_number":216,"context_line":"                              % (resp.status, image_id,"},{"line_number":217,"context_line":"                                 glance_host, glance_port))"},{"line_number":218,"context_line":"        else:"},{"line_number":219,"context_line":"            raise RetryableError(\"Unexpected response [%i] while uploading \""},{"line_number":220,"context_line":"                                 \"image [%s] \""},{"line_number":221,"context_line":"                                 \"to glance host [%s:%s]\""}],"source_content_type":"application/octet-stream","patch_set":3,"id":"AAAAQn%2F%2FBE4%3D","line":218,"updated":"2013-08-08 12:46:43.000000000","message":"I would like to see this part fixed though.  There shouldn\u0027t be any whitespace changes for code that hasn\u0027t been modified.","commit_id":"7aa6f8465ca4d0aed656a1faf271a597fbdb6f05"},{"author":{"_account_id":4340,"name":"Sulochan Acharya","email":"sulochan@gmail.com","username":"sulochan-acharya"},"change_message_id":"c89aabf3c19e2499c694a5bc8b8ae919a0943f85","unresolved":false,"context_lines":[{"line_number":220,"context_line":"                                 \"image [%s] \""},{"line_number":221,"context_line":"                                 \"to glance host [%s:%s]\""},{"line_number":222,"context_line":"                                 % (resp.status, image_id,"},{"line_number":223,"context_line":"                                    glance_host, glance_port))"},{"line_number":224,"context_line":"    finally:"},{"line_number":225,"context_line":"        conn.close()"},{"line_number":226,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":3,"id":"AAAAQn%2F%2FZBk%3D","line":223,"updated":"2013-07-31 21:58:48.000000000","message":"Same here, indentation looks ok on my editor.","commit_id":"7aa6f8465ca4d0aed656a1faf271a597fbdb6f05"}]}