)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":16258,"name":"nikesh","email":"nike.niec@gmail.com","username":"nikem"},"change_message_id":"58ba02684860e9d02fb94a842ccccb664df8f3ab","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     chenying \u003cying.chen@huawei.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2015-09-10 11:09:33 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix a snapshots not being deleted by using its name"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Snapshot client side filtering fails in the function findall. The reason is that"},{"line_number":10,"context_line":"snapshot object \u003cclass \u0027cinderclient.v2.volume_snapshots.Snapshot\u0027\u003e do not have"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"ba15a1d1_00d171cd","line":7,"updated":"2015-09-14 13:19:03.000000000","message":"The first line should be limited to 50 characters\n\nSubsequent lines should be wrapped at 72 characters.","commit_id":"288a98893fafb959caa5d85d79a0ac835f00632c"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"c4f223feeb77705723f9425960934a9a6496d790","unresolved":false,"context_lines":[{"line_number":15,"context_line":"NAME_ATTR \u003d \u0027display_name\u0027. The superclass of Snapshot \u0027Resource\u0027 has defined"},{"line_number":16,"context_line":"NAME_ATTR \u003d \u0027name\u0027."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Closes-Bug: #1495428"},{"line_number":19,"context_line":"Change-Id: I3c89fd9a82918e8a02385cfccb32dc9c3f4c4095"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"ba15a1d1_48e8eb7c","line":18,"updated":"2015-09-14 14:38:23.000000000","message":"This doesn\u0027t seem to be the right bug number?","commit_id":"288a98893fafb959caa5d85d79a0ac835f00632c"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"b1c576d9e8097c47c58e3a8006e3b3dfb5e27969","unresolved":false,"context_lines":[{"line_number":15,"context_line":"NAME_ATTR \u003d \u0027display_name\u0027. The superclass of Snapshot \u0027Resource\u0027 has defined"},{"line_number":16,"context_line":"NAME_ATTR \u003d \u0027name\u0027."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Closes-Bug: #1495428"},{"line_number":19,"context_line":"Change-Id: I3c89fd9a82918e8a02385cfccb32dc9c3f4c4095"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"ba15a1d1_aadbb41a","line":18,"in_reply_to":"ba15a1d1_14d1a5cd","updated":"2015-09-14 17:22:04.000000000","message":"Oops, ignore that.","commit_id":"288a98893fafb959caa5d85d79a0ac835f00632c"},{"author":{"_account_id":16258,"name":"nikesh","email":"nike.niec@gmail.com","username":"nikem"},"change_message_id":"515f422865fb14e938c1ba9cdfeb7474172f2a8f","unresolved":false,"context_lines":[{"line_number":15,"context_line":"NAME_ATTR \u003d \u0027display_name\u0027. The superclass of Snapshot \u0027Resource\u0027 has defined"},{"line_number":16,"context_line":"NAME_ATTR \u003d \u0027name\u0027."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Closes-Bug: #1495428"},{"line_number":19,"context_line":"Change-Id: I3c89fd9a82918e8a02385cfccb32dc9c3f4c4095"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"ba15a1d1_14d1a5cd","line":18,"in_reply_to":"ba15a1d1_48e8eb7c","updated":"2015-09-14 16:42:45.000000000","message":"Bug number seems like correct\nhttps://bugs.launchpad.net/cinder/+bug/1495428\n\nBut i am unable to regenerate this bug","commit_id":"288a98893fafb959caa5d85d79a0ac835f00632c"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"8937dab9535b902917fac8c9d211a93cd2606984","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix a snapshots not being deleted by using name"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Snapshot client side filtering fails in the function findall."},{"line_number":10,"context_line":"The reason is that snapshot object"},{"line_number":11,"context_line":"\u003cclass \u0027cinderclient.v2.volume_snapshots.Snapshot\u0027\u003e do not have"},{"line_number":12,"context_line":"a attribute \u0027display_name\u0027."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"ba15a1d1_9c25b0c3","line":9,"updated":"2015-09-22 11:03:22.000000000","message":"-1: Since this is not only fixing Snapshots commit message needs amending.","commit_id":"70a5241e2ce6fefa11c63b541ff31f614b91715f"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"8937dab9535b902917fac8c9d211a93cd2606984","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Snapshot client side filtering fails in the function findall."},{"line_number":10,"context_line":"The reason is that snapshot object"},{"line_number":11,"context_line":"\u003cclass \u0027cinderclient.v2.volume_snapshots.Snapshot\u0027\u003e do not have"},{"line_number":12,"context_line":"a attribute \u0027display_name\u0027."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"The response of /v2/?{tenant_id}?/volumes/detail do not have"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"ba15a1d1_1c31a00b","line":11,"updated":"2015-09-22 11:03:22.000000000","message":"nit: s/do/does","commit_id":"70a5241e2ce6fefa11c63b541ff31f614b91715f"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"8937dab9535b902917fac8c9d211a93cd2606984","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Snapshot client side filtering fails in the function findall."},{"line_number":10,"context_line":"The reason is that snapshot object"},{"line_number":11,"context_line":"\u003cclass \u0027cinderclient.v2.volume_snapshots.Snapshot\u0027\u003e do not have"},{"line_number":12,"context_line":"a attribute \u0027display_name\u0027."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"The response of /v2/?{tenant_id}?/volumes/detail do not have"},{"line_number":15,"context_line":"a parameter \u0027display_name\u0027, but have a parameter \u0027name\u0027."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"ba15a1d1_3c34a4f9","line":12,"updated":"2015-09-22 11:03:22.000000000","message":"nit: s/a/and","commit_id":"70a5241e2ce6fefa11c63b541ff31f614b91715f"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"8937dab9535b902917fac8c9d211a93cd2606984","unresolved":false,"context_lines":[{"line_number":11,"context_line":"\u003cclass \u0027cinderclient.v2.volume_snapshots.Snapshot\u0027\u003e do not have"},{"line_number":12,"context_line":"a attribute \u0027display_name\u0027."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"The response of /v2/?{tenant_id}?/volumes/detail do not have"},{"line_number":15,"context_line":"a parameter \u0027display_name\u0027, but have a parameter \u0027name\u0027."},{"line_number":16,"context_line":"So class Snapshot do not need define NAME_ATTR \u003d \u0027display_name\u0027."},{"line_number":17,"context_line":"The superclass of Snapshot \u0027Resource\u0027 has defined"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"ba15a1d1_7c2a1c92","line":14,"updated":"2015-09-22 11:03:22.000000000","message":"nit: s/do/does","commit_id":"70a5241e2ce6fefa11c63b541ff31f614b91715f"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"8937dab9535b902917fac8c9d211a93cd2606984","unresolved":false,"context_lines":[{"line_number":13,"context_line":""},{"line_number":14,"context_line":"The response of /v2/?{tenant_id}?/volumes/detail do not have"},{"line_number":15,"context_line":"a parameter \u0027display_name\u0027, but have a parameter \u0027name\u0027."},{"line_number":16,"context_line":"So class Snapshot do not need define NAME_ATTR \u003d \u0027display_name\u0027."},{"line_number":17,"context_line":"The superclass of Snapshot \u0027Resource\u0027 has defined"},{"line_number":18,"context_line":"NAME_ATTR \u003d \u0027name\u0027."},{"line_number":19,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"ba15a1d1_9c0cd043","line":16,"updated":"2015-09-22 11:03:22.000000000","message":"nit: s/do/does\n\nnit: s/need define/need to define","commit_id":"70a5241e2ce6fefa11c63b541ff31f614b91715f"},{"author":{"_account_id":2861,"name":"Vincent Hou","email":"shou@us.ibm.com","username":"houshengbo"},"change_message_id":"f91de3013997c7a166215de5df98dfadc6cb4522","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Fix three resources not being deleted by using name"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"There are three resources: VolumeBackup, Snapshot, VolumeTransfer."},{"line_number":10,"context_line":"They all can not been deleted by using their names."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"For example, Snapshot client side filtering fails in the function"},{"line_number":13,"context_line":"findall. The reason is that snapshot object"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"ba15a1d1_42286633","line":10,"updated":"2015-09-23 07:20:52.000000000","message":"It is better to be \"None of them can be deleted...\"","commit_id":"23a4638c7b0624f008b44ed7a0972e06978cb3ec"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"29434a58cf77a5091ab2e8271f056444dc6dd651","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Fix three resources not being deleted by using name"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"There are three resources: VolumeBackup, Snapshot, VolumeTransfer."},{"line_number":10,"context_line":"They all can not been deleted by using their names."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"For example, Snapshot client side filtering fails in the function"},{"line_number":13,"context_line":"findall. The reason is that snapshot object"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"ba15a1d1_3279fd82","line":10,"in_reply_to":"ba15a1d1_42286633","updated":"2015-09-23 09:00:43.000000000","message":"nit: No mention that this only happens on V2, but it can be inferred from later paragraphs, so it\u0027s ok.","commit_id":"23a4638c7b0624f008b44ed7a0972e06978cb3ec"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"29434a58cf77a5091ab2e8271f056444dc6dd651","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Fix three resources not being deleted by using name"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"There are three resources: VolumeBackup, Snapshot, VolumeTransfer."},{"line_number":10,"context_line":"They all can not been deleted by using their names."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"For example, Snapshot client side filtering fails in the function"},{"line_number":13,"context_line":"findall. The reason is that snapshot object"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"ba15a1d1_92880973","line":10,"in_reply_to":"ba15a1d1_42286633","updated":"2015-09-23 09:00:43.000000000","message":"nit: No mention that this only happens on V2, but it can be inferred from later paragraphs, so it\u0027s ok.","commit_id":"23a4638c7b0624f008b44ed7a0972e06978cb3ec"}]}