)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":170,"name":"Mike Perez","email":"thingee@gmail.com","username":"thingee"},"change_message_id":"b46e62b1f918747c5f1b3ae15dfc2b89de3b9d37","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Nexenta drivers ignore does not exist exception"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Ignore does not exist exception in NFS and iSCSI driver,"},{"line_number":10,"context_line":"delete_volume and delete_snapshot methods."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Closes-Bug: 1240650"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAATH%2F%2FYlo%3D","line":9,"updated":"2013-10-16 20:53:46.000000000","message":"This commit message alone is not very descriptive of why have the driver ignore these exceptions. From the patch I can see this is because the resource no longer exists in the backend. Can you mention that as being the case in the commit message, so we know that at glance? Thanks.","commit_id":"567e1d8be029a93a16c8f382bcce2be1a84abdf0"},{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"028789370eab90d9e5a0745d12927bb7f95ba1b3","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Victor Rodionov \u003cvito.ordaz@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2013-10-17 10:40:11 -0700"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Nexenta drivers ignore does not exist exception"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Ignore does not exist exception in NFS and iSCSI driver,"},{"line_number":10,"context_line":"delete_volume and delete_snapshot methods."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"AAAATH%2F%2FLKc%3D","line":7,"updated":"2013-10-21 23:29:35.000000000","message":"Don\u0027t think there are rules against quotes in the first line.  I would do \"does not exist\" for clarity.  If not there, I would do it in the following sentence so it is clearer what to expect when looking at the code.","commit_id":"cb4ea908ee479584445eccb74dfe45f67e9f0a41"},{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"9abb2dd2dc179e9aa42d260bbf491485491f6476","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Victor Rodionov \u003cvito.ordaz@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2013-10-21 17:29:27 -0700"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Nexenta \"drivers ignore\" does not exist exception"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Ignore \"does not exist\" exception in NFS and iSCSI driver,"},{"line_number":10,"context_line":"delete_volume and delete_snapshot methods."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"AAAATH%2F%2FI%2BI%3D","line":7,"updated":"2013-10-22 15:54:41.000000000","message":"I think you meant:  Nexenta drivers ignore \"does not exist\" exception","commit_id":"5b827d413a4e8944dd1888cae38eff62184b10fe"},{"author":{"_account_id":4355,"name":"Avishay Traeger","email":"avishay@stratoscale.com","username":"avishay-il"},"change_message_id":"9064e212c233acf554bdf8c5ebaa1665f96b41d3","unresolved":false,"context_lines":[{"line_number":12,"context_line":"backend, so we can avoid \"delete\" errors, when the resource was already"},{"line_number":13,"context_line":"deleted."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Closes-Bug: 1240650"},{"line_number":16,"context_line":"Change-Id: I3b2684097dcb95ba50a67c94a353dc3f17374777"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":8,"id":"AAAATX%2F%2F%2F0s%3D","line":15,"updated":"2013-10-23 05:33:59.000000000","message":"In the future, please use \"Closes-Bug: #1234567\" so that it creates a link here.","commit_id":"d4b7d5fd427d3c46bb61a45edd2623305e23b90b"}],"cinder/volume/drivers/nexenta/iscsi.py":[{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"028789370eab90d9e5a0745d12927bb7f95ba1b3","unresolved":false,"context_lines":[{"line_number":216,"context_line":"        except nexenta.NexentaException as exc:"},{"line_number":217,"context_line":"            if \"does not exist\" in exc.args[0]:"},{"line_number":218,"context_line":"                LOG.info(_(\u0027Snapshot %s does not exist, it seems it was \u0027"},{"line_number":219,"context_line":"                           \u0027already deleted\u0027), snapshot_name)"},{"line_number":220,"context_line":"                return"},{"line_number":221,"context_line":"            if \"snapshot has dependent clones\" in exc.args[0]:"},{"line_number":222,"context_line":"                raise exception.SnapshotIsBusy(snapshot_name\u003dsnapshot[\u0027name\u0027])"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAATH%2F%2FLMw%3D","line":219,"updated":"2013-10-21 23:29:35.000000000","message":"I think an \u0027info\u0027 message should end with \u0027.\u0027","commit_id":"cb4ea908ee479584445eccb74dfe45f67e9f0a41"}],"cinder/volume/drivers/nexenta/nfs.py":[{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"028789370eab90d9e5a0745d12927bb7f95ba1b3","unresolved":false,"context_lines":[{"line_number":53,"context_line":"        1.1.0 - Auto sharing for enclosing folder."},{"line_number":54,"context_line":"        1.1.1 - Added caching for NexentaStor appliance \u0027volroot\u0027 value."},{"line_number":55,"context_line":"        1.1.2 - Ignore folder does not exist error in delete_volume and"},{"line_number":56,"context_line":"                delete_snapshot method."},{"line_number":57,"context_line":"    \"\"\""},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"    VERSION \u003d VERSION"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAATH%2F%2FLKw%3D","line":56,"updated":"2013-10-21 23:29:35.000000000","message":"Recommend rewording this to be consistent with the other change:\nIgnore \"folder does not exist\" error in delete_volume and\ndelete_snapshot method.","commit_id":"cb4ea908ee479584445eccb74dfe45f67e9f0a41"},{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"028789370eab90d9e5a0745d12927bb7f95ba1b3","unresolved":false,"context_lines":[{"line_number":224,"context_line":"            except nexenta.NexentaException as exc:"},{"line_number":225,"context_line":"                if \u0027does not exist\u0027 in exc.args[0]:"},{"line_number":226,"context_line":"                    LOG.info(_(\u0027Folder %s does not exist, it seems it was \u0027"},{"line_number":227,"context_line":"                               \u0027already deleted\u0027), folder)"},{"line_number":228,"context_line":"                    return"},{"line_number":229,"context_line":"                raise"},{"line_number":230,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAATH%2F%2FLMY%3D","line":227,"updated":"2013-10-21 23:29:35.000000000","message":"End with a \u0027.\u0027","commit_id":"cb4ea908ee479584445eccb74dfe45f67e9f0a41"},{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"028789370eab90d9e5a0745d12927bb7f95ba1b3","unresolved":false,"context_lines":[{"line_number":256,"context_line":"            if \u0027does not exist\u0027 in exc.args[0]:"},{"line_number":257,"context_line":"                LOG.info(_(\u0027Snapshot %s does not exist, it seems it was \u0027"},{"line_number":258,"context_line":"                           \u0027already deleted\u0027), \u0027%s@%s\u0027 % (folder, snapshot))"},{"line_number":259,"context_line":"                return"},{"line_number":260,"context_line":"            raise"},{"line_number":261,"context_line":""},{"line_number":262,"context_line":"    def _create_sparsed_file(self, nms, path, size):"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAATH%2F%2FLMU%3D","line":259,"updated":"2013-10-21 23:29:35.000000000","message":"end with a \u0027.\u0027","commit_id":"cb4ea908ee479584445eccb74dfe45f67e9f0a41"}]}