)]}'
{"cinder/volume/drivers/remotefs.py":[{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"f7740825be1e6e8210594ea6599765585add6f55","unresolved":false,"context_lines":[{"line_number":1304,"context_line":"                        \u0027while creating snapshot.\u0027)"},{"line_number":1305,"context_line":"                raise exception.RemoteFSException(msg)"},{"line_number":1306,"context_line":""},{"line_number":1307,"context_line":"            elif s[\u0027status\u0027] \u003d\u003d \u0027deleting\u0027:"},{"line_number":1308,"context_line":"                msg \u003d _(\u0027Snapshot %(id)s is being deleted while waiting for \u0027"},{"line_number":1309,"context_line":"                        \u0027for it to become available. Perhaps a concurrent \u0027"},{"line_number":1310,"context_line":"                        \u0027request was made.\u0027) % {\u0027id\u0027: snapshot[\u0027id\u0027]}"}],"source_content_type":"text/x-python","patch_set":1,"id":"dae33548_d84f0890","line":1307,"updated":"2016-02-18 16:36:09.000000000","message":"Seems fine, but by the same reasoning, this should also include \"error_deleting\".","commit_id":"25524dc208c45d20dba39a3989bd4ff6cd848b77"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"3ba32274bebe407e0277bd7fbfe03ba00cd0a0eb","unresolved":false,"context_lines":[{"line_number":1304,"context_line":"                        \u0027while creating snapshot.\u0027)"},{"line_number":1305,"context_line":"                raise exception.RemoteFSException(msg)"},{"line_number":1306,"context_line":""},{"line_number":1307,"context_line":"            elif s[\u0027status\u0027] \u003d\u003d \u0027deleting\u0027:"},{"line_number":1308,"context_line":"                msg \u003d _(\u0027Snapshot %(id)s is being deleted while waiting for \u0027"},{"line_number":1309,"context_line":"                        \u0027for it to become available. Perhaps a concurrent \u0027"},{"line_number":1310,"context_line":"                        \u0027request was made.\u0027) % {\u0027id\u0027: snapshot[\u0027id\u0027]}"}],"source_content_type":"text/x-python","patch_set":1,"id":"dae33548_6248c2b3","line":1307,"in_reply_to":"dae33548_19e691e7","updated":"2016-02-18 18:44:03.000000000","message":"I\u0027d prefer to, since I think it\u0027s basically the same race.  A failure occurring in the path that set this to \u0027deleting\u0027 in this case could result in \u0027error_deleting\u0027 during the same window.","commit_id":"25524dc208c45d20dba39a3989bd4ff6cd848b77"},{"author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"change_message_id":"ef7925a5aa065fc4b6cdba65657c17a7918c3e04","unresolved":false,"context_lines":[{"line_number":1304,"context_line":"                        \u0027while creating snapshot.\u0027)"},{"line_number":1305,"context_line":"                raise exception.RemoteFSException(msg)"},{"line_number":1306,"context_line":""},{"line_number":1307,"context_line":"            elif s[\u0027status\u0027] \u003d\u003d \u0027deleting\u0027:"},{"line_number":1308,"context_line":"                msg \u003d _(\u0027Snapshot %(id)s is being deleted while waiting for \u0027"},{"line_number":1309,"context_line":"                        \u0027for it to become available. Perhaps a concurrent \u0027"},{"line_number":1310,"context_line":"                        \u0027request was made.\u0027) % {\u0027id\u0027: snapshot[\u0027id\u0027]}"}],"source_content_type":"text/x-python","patch_set":1,"id":"dae33548_4d984f78","line":1307,"in_reply_to":"dae33548_6248c2b3","updated":"2016-02-18 19:02:06.000000000","message":"Agree that \"error_deleting\" should also be included here.","commit_id":"25524dc208c45d20dba39a3989bd4ff6cd848b77"},{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"71f7b87fa8a02cc96d4e7ea3eb1be145fc14681e","unresolved":false,"context_lines":[{"line_number":1304,"context_line":"                        \u0027while creating snapshot.\u0027)"},{"line_number":1305,"context_line":"                raise exception.RemoteFSException(msg)"},{"line_number":1306,"context_line":""},{"line_number":1307,"context_line":"            elif s[\u0027status\u0027] \u003d\u003d \u0027deleting\u0027:"},{"line_number":1308,"context_line":"                msg \u003d _(\u0027Snapshot %(id)s is being deleted while waiting for \u0027"},{"line_number":1309,"context_line":"                        \u0027for it to become available. Perhaps a concurrent \u0027"},{"line_number":1310,"context_line":"                        \u0027request was made.\u0027) % {\u0027id\u0027: snapshot[\u0027id\u0027]}"}],"source_content_type":"text/x-python","patch_set":1,"id":"dae33548_19e691e7","line":1307,"in_reply_to":"dae33548_d84f0890","updated":"2016-02-18 17:40:18.000000000","message":"I don\"t have a reproducer for this case. But I can also include \"error_deleting\", if you feel it is the correct thing to do.","commit_id":"25524dc208c45d20dba39a3989bd4ff6cd848b77"}]}