)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":1207,"name":"Duncan Thomas","email":"duncan.thomas@gmail.com","username":"duncan-thomas"},"change_message_id":"6263dda6ca36a6201ddc16772a788737da6cb568","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     zwei \u003cleidong@unitedstack.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-03-08 14:15:44 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Bootable filter for listening volumes from CLI"},{"line_number":8,"context_line":"    In this fix, we have provided support for"},{"line_number":9,"context_line":"bootable parameter in cinder api version 1 list"},{"line_number":10,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"5aef4532_3b585732","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":46},"updated":"2016-03-08 08:56:42.000000000","message":"Need a blank line after the subject","commit_id":"db5335626e0d4a6c777d38a4a4ae411d80e79492"},{"author":{"_account_id":1773,"name":"oliver-leahy-l","email":"oliver.leahy@hpe.com","username":"oliver-leahy-l"},"change_message_id":"88e32b7f7479cf5c491141f5081c248921387600","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     zwei \u003cleidong@unitedstack.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-03-08 14:15:44 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Bootable filter for listening volumes from CLI"},{"line_number":8,"context_line":"    In this fix, we have provided support for"},{"line_number":9,"context_line":"bootable parameter in cinder api version 1 list"},{"line_number":10,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"5aef4532_09ea710f","line":7,"range":{"start_line":7,"start_character":20,"end_line":7,"end_character":29},"updated":"2016-03-08 08:37:08.000000000","message":"s/listening/listing/","commit_id":"db5335626e0d4a6c777d38a4a4ae411d80e79492"},{"author":{"_account_id":20105,"name":"Cao ShuFeng","email":"caosf.fnst@cn.fujitsu.com","username":"caosf.fnst"},"change_message_id":"492ec9ff1efea1c6207d31318bd2f1149ed2535a","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2016-03-08 14:15:44 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Bootable filter for listening volumes from CLI"},{"line_number":8,"context_line":"    In this fix, we have provided support for"},{"line_number":9,"context_line":"bootable parameter in cinder api version 1 list"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Closes-Bug: #1543205"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"5aef4532_3b369752","line":8,"range":{"start_line":8,"start_character":0,"end_line":8,"end_character":4},"updated":"2016-03-08 07:49:44.000000000","message":"We need an empty line here, so that the commit message will look neat.","commit_id":"db5335626e0d4a6c777d38a4a4ae411d80e79492"},{"author":{"_account_id":1773,"name":"oliver-leahy-l","email":"oliver.leahy@hpe.com","username":"oliver-leahy-l"},"change_message_id":"88e32b7f7479cf5c491141f5081c248921387600","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Bootable filter for listening volumes from CLI"},{"line_number":8,"context_line":"    In this fix, we have provided support for"},{"line_number":9,"context_line":"bootable parameter in cinder api version 1 list"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Closes-Bug: #1543205"},{"line_number":12,"context_line":"Change-Id: Ife052545d8e6130ca910b1e66ea6dd2160fa85f6"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"5aef4532_29ce4db4","line":9,"range":{"start_line":9,"start_character":33,"end_line":9,"end_character":40},"updated":"2016-03-08 08:37:08.000000000","message":"I think the cinder v1 api is being deprecated: https://review.openstack.org/73856","commit_id":"db5335626e0d4a6c777d38a4a4ae411d80e79492"}],"cinderclient/v1/shell.py":[{"author":{"_account_id":20105,"name":"Cao ShuFeng","email":"caosf.fnst@cn.fujitsu.com","username":"caosf.fnst"},"change_message_id":"7f16f79f540a66e7c3e7dc63e3a2ee45449e2ffd","unresolved":false,"context_lines":[{"line_number":177,"context_line":"    help\u003d\u0027Maximum number of volumes to return. OPTIONAL: Default\u003dNone.\u0027)"},{"line_number":178,"context_line":"@utils.arg("},{"line_number":179,"context_line":"    \u0027--bootable\u0027,"},{"line_number":180,"context_line":"    metavar\u003d\u0027\u003cTrue|trur|False|false\u003e\u0027,"},{"line_number":181,"context_line":"    const\u003dTrue,"},{"line_number":182,"context_line":"    nargs\u003d\u0027?\u0027,"},{"line_number":183,"context_line":"    choices\u003d[\u0027True\u0027, \u0027true\u0027, \u0027False\u0027, \u0027false\u0027],"}],"source_content_type":"text/x-python","patch_set":2,"id":"5aef4532_c2e8697f","line":180,"range":{"start_line":180,"start_character":19,"end_line":180,"end_character":23},"updated":"2016-03-08 06:08:51.000000000","message":"s/trur/true","commit_id":"8d94b56e942d91e23d540c48f485878f346ef8c7"},{"author":{"_account_id":20105,"name":"Cao ShuFeng","email":"caosf.fnst@cn.fujitsu.com","username":"caosf.fnst"},"change_message_id":"492ec9ff1efea1c6207d31318bd2f1149ed2535a","unresolved":false,"context_lines":[{"line_number":177,"context_line":"    help\u003d\u0027Maximum number of volumes to return. OPTIONAL: Default\u003dNone.\u0027)"},{"line_number":178,"context_line":"@utils.arg("},{"line_number":179,"context_line":"    \u0027--bootable\u0027,"},{"line_number":180,"context_line":"    metavar\u003d\u0027\u003cTrue|true|False|false\u003e\u0027,"},{"line_number":181,"context_line":"    const\u003dTrue,"},{"line_number":182,"context_line":"    nargs\u003d\u0027?\u0027,"},{"line_number":183,"context_line":"    choices\u003d[\u0027True\u0027, \u0027true\u0027, \u0027False\u0027, \u0027false\u0027],"}],"source_content_type":"text/x-python","patch_set":3,"id":"5aef4532_1b02bb6f","line":180,"range":{"start_line":180,"start_character":19,"end_line":180,"end_character":23},"updated":"2016-03-08 07:49:44.000000000","message":"The v1 api doesn\u0027t support \"--bootable ture\" and \"--bootable false\".\n\nhttp://paste.openstack.org/show/489638/","commit_id":"db5335626e0d4a6c777d38a4a4ae411d80e79492"},{"author":{"_account_id":1207,"name":"Duncan Thomas","email":"duncan.thomas@gmail.com","username":"duncan-thomas"},"change_message_id":"6263dda6ca36a6201ddc16772a788737da6cb568","unresolved":false,"context_lines":[{"line_number":177,"context_line":"    help\u003d\u0027Maximum number of volumes to return. OPTIONAL: Default\u003dNone.\u0027)"},{"line_number":178,"context_line":"@utils.arg("},{"line_number":179,"context_line":"    \u0027--bootable\u0027,"},{"line_number":180,"context_line":"    metavar\u003d\u0027\u003cTrue|true|False|false\u003e\u0027,"},{"line_number":181,"context_line":"    const\u003dTrue,"},{"line_number":182,"context_line":"    nargs\u003d\u0027?\u0027,"},{"line_number":183,"context_line":"    choices\u003d[\u0027True\u0027, \u0027true\u0027, \u0027False\u0027, \u0027false\u0027],"}],"source_content_type":"text/x-python","patch_set":3,"id":"5aef4532_2424b4ad","line":180,"range":{"start_line":180,"start_character":0,"end_line":180,"end_character":38},"updated":"2016-03-08 08:56:42.000000000","message":"We probably want this to be an int, to be consistent with all_tenants.","commit_id":"db5335626e0d4a6c777d38a4a4ae411d80e79492"},{"author":{"_account_id":19954,"name":"zwei","email":"leidong@unitedstack.com","username":"zwei"},"change_message_id":"9e2dbc6690adc20260e9d1e08fd4543d8c3563b9","unresolved":false,"context_lines":[{"line_number":177,"context_line":"    help\u003d\u0027Maximum number of volumes to return. OPTIONAL: Default\u003dNone.\u0027)"},{"line_number":178,"context_line":"@utils.arg("},{"line_number":179,"context_line":"    \u0027--bootable\u0027,"},{"line_number":180,"context_line":"    metavar\u003d\u0027\u003cTrue|true|False|false\u003e\u0027,"},{"line_number":181,"context_line":"    const\u003dTrue,"},{"line_number":182,"context_line":"    nargs\u003d\u0027?\u0027,"},{"line_number":183,"context_line":"    choices\u003d[\u0027True\u0027, \u0027true\u0027, \u0027False\u0027, \u0027false\u0027],"}],"source_content_type":"text/x-python","patch_set":3,"id":"5aef4532_496f59c5","line":180,"range":{"start_line":180,"start_character":19,"end_line":180,"end_character":23},"in_reply_to":"5aef4532_1b02bb6f","updated":"2016-03-08 08:37:10.000000000","message":"v1 support --bootable\n\nhttps://bugs.launchpad.net/python-cinderclient/+bug/1543205","commit_id":"db5335626e0d4a6c777d38a4a4ae411d80e79492"},{"author":{"_account_id":20105,"name":"Cao ShuFeng","email":"caosf.fnst@cn.fujitsu.com","username":"caosf.fnst"},"change_message_id":"78499831a0641df4f876c4451e6f229db9fe448b","unresolved":false,"context_lines":[{"line_number":177,"context_line":"    help\u003d\u0027Maximum number of volumes to return. OPTIONAL: Default\u003dNone.\u0027)"},{"line_number":178,"context_line":"@utils.arg("},{"line_number":179,"context_line":"    \u0027--bootable\u0027,"},{"line_number":180,"context_line":"    metavar\u003d\u0027\u003cTrue|true|False|false\u003e\u0027,"},{"line_number":181,"context_line":"    const\u003dTrue,"},{"line_number":182,"context_line":"    nargs\u003d\u0027?\u0027,"},{"line_number":183,"context_line":"    choices\u003d[\u0027True\u0027, \u0027true\u0027, \u0027False\u0027, \u0027false\u0027],"}],"source_content_type":"text/x-python","patch_set":3,"id":"5aef4532_69f4d586","line":180,"range":{"start_line":180,"start_character":19,"end_line":180,"end_character":23},"in_reply_to":"5aef4532_496f59c5","updated":"2016-03-08 08:48:56.000000000","message":"Sorry,I didn\u0027t say it clearly.\nI mean that v1 api supports \"--bootable True\" and \"--bootable False\".\nBut It doesn\u0027t support \"--bootable ture\" and \"--bootable false\".\nSo, you should remove \"true\" and \"false\" in metavar.\nmetavar\u003d\u0027\u003cTrue|False\u003e\u0027","commit_id":"db5335626e0d4a6c777d38a4a4ae411d80e79492"}]}
