)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7051,"name":"Swapnil Kulkarni","email":"skulkarn@redhat.com","username":"swapnilkulkarni2608"},"change_message_id":"29800a59573516391844b624524804cbcf4bddec","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Jeff Applewhite \u003cjeff.applewhite@netapp.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2014-01-30 14:11:08 -0500"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Closes-Bug: #1247743"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I5478d00442ce855c91a7f04c5ba2b40733d44902"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAAUH%2F%2BCQU%3D","line":7,"updated":"2014-01-30 19:36:47.000000000","message":"Please add appropriate commit message.","commit_id":"596a2cf2a6c4ff76f2c4752cf3ec2c118f53d42e"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"6d50d383f335b3fb763011cb3f8ad9cb57309c98","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Jeff Applewhite \u003cjeff.applewhite@netapp.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2014-01-30 14:46:57 -0500"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"removes not implemented stubs"},{"line_number":8,"context_line":"Closes-Bug: #1247743"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"Change-Id: I5478d00442ce855c91a7f04c5ba2b40733d44902"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAAUH%2F%2BCK8%3D","line":7,"updated":"2014-01-30 19:51:46.000000000","message":"Please note what the effect of this is in the commit message.  (Backups work for NFS?)","commit_id":"20ae8318c5f2053cadb29ab09b6fc30e869b44ee"},{"author":{"_account_id":8532,"name":"Ryan Brady","email":"rbrady@redhat.com","username":"rbrady"},"change_message_id":"4a28495c656e67fda5c79c77bba6f61e2ffd3e0c","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Jeff Applewhite \u003cjeff.applewhite@netapp.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2014-01-30 14:46:57 -0500"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"removes not implemented stubs"},{"line_number":8,"context_line":"Closes-Bug: #1247743"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"Change-Id: I5478d00442ce855c91a7f04c5ba2b40733d44902"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAAUH%2F%2BCIg%3D","line":7,"updated":"2014-01-30 19:58:14.000000000","message":"Please take a look at https://wiki.openstack.org/wiki/GitCommitMessages#Information_in_commit_messages\n\nIt would be great to answer \"why is this change being made?\" and provide additional context.","commit_id":"20ae8318c5f2053cadb29ab09b6fc30e869b44ee"},{"author":{"_account_id":8532,"name":"Ryan Brady","email":"rbrady@redhat.com","username":"rbrady"},"change_message_id":"73029600a5dc27b006db7cbf01687901e395bf5a","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Jeff Applewhite \u003cjeff.applewhite@netapp.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2014-01-30 14:55:28 -0500"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"removes not implemented stubs for backup_volume and restore_volume. After manual testing of this change backups succeed without the not implemented stubs."},{"line_number":8,"context_line":"Closes-Bug: #1247743"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"Change-Id: I5478d00442ce855c91a7f04c5ba2b40733d44902"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"AAAAUH%2F%2BCEY%3D","line":7,"updated":"2014-01-30 20:07:17.000000000","message":"Please take a look at https://wiki.openstack.org/wiki/GitCommitMessages#Information_in_commit_messages.\n\nSpecifically \"Do not assume the reviewer has access to external web services/site.\"\n\nMaybe something more like the following:\n\nFixes cinder failed to create a backup with NFS driver\n\nThe action cinder backup-create fails when trying to backup a newly created volume when using cinder.volume.drivers.nfs.NfsDriver.  This patch removes not implemented stubs for backup_volume and restore_volume. The inherited methods succeed in creating a backup and in restoring it.  After manual testing of this cha\nnge backups succeed without the not implemented stubs.","commit_id":"a6969ad46b16820e30f3687a846407a6d4a9290a"},{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"0d49b2c6a87a98742f7d3231cc3b896345788601","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Jeff Applewhite \u003cjeff.applewhite@netapp.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2014-01-30 14:55:28 -0500"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"removes not implemented stubs for backup_volume and restore_volume. After manual testing of this change backups succeed without the not implemented stubs."},{"line_number":8,"context_line":"Closes-Bug: #1247743"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"Change-Id: I5478d00442ce855c91a7f04c5ba2b40733d44902"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"AAAAUH%2F%2BCDM%3D","line":7,"in_reply_to":"AAAAUH%2F%2BCEY%3D","updated":"2014-01-30 20:11:29.000000000","message":"Can you please also indicate where the inherited methods are coming from?  Just curious why they were listed and not implemented but actually are.","commit_id":"a6969ad46b16820e30f3687a846407a6d4a9290a"},{"author":{"_account_id":8532,"name":"Ryan Brady","email":"rbrady@redhat.com","username":"rbrady"},"change_message_id":"6baa4cb19a33f51e0b86eeaf327aeccb2916fd68","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fixes cinder failed to create/restore a backup with NFS driver"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The action cinder backup-create fails when trying to backup a newly created volume when using cinder.volume.drivers.nfs.NfsDriver. This patch removes not implemented stubs for backup_volume and restore_volume. The inherited methods in cinder/volume/driver.py succeed in creating a backup and in restoring it. After manual testing of this change backups/restores succeed without the not implemented stubs."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Closes-Bug: #1247743"},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"AAAAUH%2F%2BBwc%3D","line":9,"updated":"2014-01-30 21:15:40.000000000","message":"You\u0027re getting closer.  Please wrap the commit message at 70 chars.  Gerrit should have warned you.","commit_id":"fa30f9c7b69b5c2bfee81e9e99f6ba90383c8a95"}]}