)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":17405,"name":"Jacob Gregor","email":"jgregor@us.ibm.com","username":"jgregor"},"change_message_id":"8f45e7a2c98db577b5aec35af864c3a9cab3f917","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2015-10-27 22:27:14 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"List backups return \"parent_id\" field"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"change-ID: Iec6750241c8581f581a0c6eada55711d806d788d"},{"line_number":10,"context_line":"Implements: blueprint list-backups-return-parent-id"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"1a26ad4f_41db011c","line":8,"updated":"2015-10-27 15:56:18.000000000","message":"It would be helpful if you added a summary describing the purpose of this spec.","commit_id":"496716fd96a40f7814727dcae2d36f2c3837d402"}],"specs/mitaka/list-backups-return-parent-id.rst":[{"author":{"_account_id":17405,"name":"Jacob Gregor","email":"jgregor@us.ibm.com","username":"jgregor"},"change_message_id":"8f45e7a2c98db577b5aec35af864c3a9cab3f917","unresolved":false,"context_lines":[{"line_number":8,"context_line":"This is very useful for horizon or other customized openstack portal to draw"},{"line_number":9,"context_line":"the dependency relationship from backups which is made by a volume."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"if we have the parent_id field, we can draw the backups chain,like:"},{"line_number":12,"context_line":"volume----\u003ebackup1--(backup2 is rely on backup1)--\u003ebackup2--(3 is rely on 2)-\u003ebackup3"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Endpoint user see the backups chain will know how cinder organize the backups directly"}],"source_content_type":"text/x-rst","patch_set":3,"id":"1a26ad4f_e1ec8d91","line":11,"updated":"2015-10-27 15:56:18.000000000","message":"if -\u003e If\n\nAdd a space between \u0027,\u0027 and \u0027like\u0027.","commit_id":"496716fd96a40f7814727dcae2d36f2c3837d402"},{"author":{"_account_id":17405,"name":"Jacob Gregor","email":"jgregor@us.ibm.com","username":"jgregor"},"change_message_id":"8f45e7a2c98db577b5aec35af864c3a9cab3f917","unresolved":false,"context_lines":[{"line_number":11,"context_line":"if we have the parent_id field, we can draw the backups chain,like:"},{"line_number":12,"context_line":"volume----\u003ebackup1--(backup2 is rely on backup1)--\u003ebackup2--(3 is rely on 2)-\u003ebackup3"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Endpoint user see the backups chain will know how cinder organize the backups directly"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Problem description"}],"source_content_type":"text/x-rst","patch_set":3,"id":"1a26ad4f_21f36575","line":14,"updated":"2015-10-27 15:56:18.000000000","message":"Missing period at EOL.","commit_id":"496716fd96a40f7814727dcae2d36f2c3837d402"},{"author":{"_account_id":17405,"name":"Jacob Gregor","email":"jgregor@us.ibm.com","username":"jgregor"},"change_message_id":"8f45e7a2c98db577b5aec35af864c3a9cab3f917","unresolved":false,"context_lines":[{"line_number":17,"context_line":"Problem description"},{"line_number":18,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Now in cinder, list backups or show backup do not return parent_id field"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"Use Cases"},{"line_number":23,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":3,"id":"1a26ad4f_61e2bda1","line":20,"updated":"2015-10-27 15:56:18.000000000","message":"Missing period at EOL.","commit_id":"496716fd96a40f7814727dcae2d36f2c3837d402"},{"author":{"_account_id":16708,"name":"Kendall Nelson","display_name":"Kendall (diablo_rojo)","email":"kennelson11@gmail.com","username":"kjnelson"},"change_message_id":"b00512b1683078d84d6707748d6a13f265c42cec","unresolved":false,"context_lines":[{"line_number":18,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Now in cinder, list backups or show backup do not return parent_id field"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"Use Cases"},{"line_number":23,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":24,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"1a26ad4f_28826553","line":21,"updated":"2015-10-27 16:26:30.000000000","message":"Please elaborate on why this is a problem.","commit_id":"496716fd96a40f7814727dcae2d36f2c3837d402"},{"author":{"_account_id":17405,"name":"Jacob Gregor","email":"jgregor@us.ibm.com","username":"jgregor"},"change_message_id":"8f45e7a2c98db577b5aec35af864c3a9cab3f917","unresolved":false,"context_lines":[{"line_number":21,"context_line":""},{"line_number":22,"context_line":"Use Cases"},{"line_number":23,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"Proposed change"},{"line_number":26,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":27,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"1a26ad4f_65908896","line":24,"updated":"2015-10-27 15:56:18.000000000","message":"Are there no use cases? Should there \u0027None\u0027 be here?","commit_id":"496716fd96a40f7814727dcae2d36f2c3837d402"},{"author":{"_account_id":16708,"name":"Kendall Nelson","display_name":"Kendall (diablo_rojo)","email":"kennelson11@gmail.com","username":"kjnelson"},"change_message_id":"b00512b1683078d84d6707748d6a13f265c42cec","unresolved":false,"context_lines":[{"line_number":21,"context_line":""},{"line_number":22,"context_line":"Use Cases"},{"line_number":23,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"Proposed change"},{"line_number":26,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":27,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"1a26ad4f_0873e9b2","line":24,"updated":"2015-10-27 16:26:30.000000000","message":"There have to be use cases here otherwise there is no reason to add this to Cinder.","commit_id":"496716fd96a40f7814727dcae2d36f2c3837d402"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"e558dfcc199cb97ed3c64fd707d578f85b4997dd","unresolved":false,"context_lines":[{"line_number":21,"context_line":""},{"line_number":22,"context_line":"Use Cases"},{"line_number":23,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"Proposed change"},{"line_number":26,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":27,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"1a26ad4f_4256505d","line":24,"in_reply_to":"1a26ad4f_0873e9b2","updated":"2015-10-30 01:10:41.000000000","message":"Use Cases are defined in the commit message of the patch and should be added here as well:\n\nlist backups or show backup return backup parent_id can make\nthe caller(user/operator) known the dependency relationship\nbetween a volume\u0027s backups.\n\nThe parent_id also help horizon or other openstack portal\nto draw the backups relationship graph.","commit_id":"496716fd96a40f7814727dcae2d36f2c3837d402"},{"author":{"_account_id":17405,"name":"Jacob Gregor","email":"jgregor@us.ibm.com","username":"jgregor"},"change_message_id":"8f45e7a2c98db577b5aec35af864c3a9cab3f917","unresolved":false,"context_lines":[{"line_number":25,"context_line":"Proposed change"},{"line_number":26,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"modify function def detail in api/views/backup.py to return parent_id field"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"Alternatives"},{"line_number":31,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":3,"id":"1a26ad4f_2165a544","line":28,"updated":"2015-10-27 15:56:18.000000000","message":"modify -\u003e Modify\n\nMissing period at EOL.","commit_id":"496716fd96a40f7814727dcae2d36f2c3837d402"},{"author":{"_account_id":17405,"name":"Jacob Gregor","email":"jgregor@us.ibm.com","username":"jgregor"},"change_message_id":"8f45e7a2c98db577b5aec35af864c3a9cab3f917","unresolved":false,"context_lines":[{"line_number":40,"context_line":"REST API impact"},{"line_number":41,"context_line":"---------------"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"have impact to cinder api:"},{"line_number":44,"context_line":"1. GET /v2/{tenant_id}/backups/detail"},{"line_number":45,"context_line":"2. GET /v2/{tenant_id}/backups/{backup_id}"},{"line_number":46,"context_line":"The two api will return one more field named with \"parent_id\""}],"source_content_type":"text/x-rst","patch_set":3,"id":"1a26ad4f_61d87d1e","line":43,"updated":"2015-10-27 15:56:18.000000000","message":"have -\u003e Have","commit_id":"496716fd96a40f7814727dcae2d36f2c3837d402"},{"author":{"_account_id":17405,"name":"Jacob Gregor","email":"jgregor@us.ibm.com","username":"jgregor"},"change_message_id":"8f45e7a2c98db577b5aec35af864c3a9cab3f917","unresolved":false,"context_lines":[{"line_number":117,"context_line":"Work Items"},{"line_number":118,"context_line":"----------"},{"line_number":119,"context_line":""},{"line_number":120,"context_line":"- modify function def detail in api/views/backup.py to return parent_id field"},{"line_number":121,"context_line":""},{"line_number":122,"context_line":"Dependencies"},{"line_number":123,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":3,"id":"1a26ad4f_21092596","line":120,"updated":"2015-10-27 15:56:18.000000000","message":"modify -\u003e Modify\n\nMissing period at EOL.","commit_id":"496716fd96a40f7814727dcae2d36f2c3837d402"},{"author":{"_account_id":17405,"name":"Jacob Gregor","email":"jgregor@us.ibm.com","username":"jgregor"},"change_message_id":"8f45e7a2c98db577b5aec35af864c3a9cab3f917","unresolved":false,"context_lines":[{"line_number":126,"context_line":""},{"line_number":127,"context_line":"Testing"},{"line_number":128,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":129,"context_line":""},{"line_number":130,"context_line":""},{"line_number":131,"context_line":"Documentation Impact"},{"line_number":132,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":3,"id":"1a26ad4f_2117e5a4","line":129,"updated":"2015-10-27 15:56:18.000000000","message":"None? Standard unit tests?","commit_id":"496716fd96a40f7814727dcae2d36f2c3837d402"},{"author":{"_account_id":17405,"name":"Jacob Gregor","email":"jgregor@us.ibm.com","username":"jgregor"},"change_message_id":"8f45e7a2c98db577b5aec35af864c3a9cab3f917","unresolved":false,"context_lines":[{"line_number":137,"context_line":""},{"line_number":138,"context_line":"References"},{"line_number":139,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":140,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"1a26ad4f_e556d805","line":140,"updated":"2015-10-27 15:56:18.000000000","message":"None?","commit_id":"496716fd96a40f7814727dcae2d36f2c3837d402"},{"author":{"_account_id":1207,"name":"Duncan Thomas","email":"duncan.thomas@gmail.com","username":"duncan-thomas"},"change_message_id":"5f784c2e497cc9e8dd6fc48ed48ae7717601a2e8","unresolved":false,"context_lines":[{"line_number":32,"context_line":"volume backups."},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"The parent_id also help horizon or other openstack portal to draw "},{"line_number":35,"context_line":"the backups relationship graph."},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"Proposed change"},{"line_number":38,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":8,"id":"1a26ad4f_3018fa1d","line":35,"updated":"2015-11-02 15:11:26.000000000","message":"Missing from both these use-cases is *why* the user would want to do that. What do they gain by knowing it?\n\nPretty sure the answer is knowing what they can/can\u0027t delete, but it needs writing down IMO. Just being able to peer into the internals of cinder without a good reason is not a good thing, so we should make a habit of justifying why something is desirable.\n\nOne reason it might be a bad thing is that we auto-pick backup parents, and it may generate support load when people start asking why it picked a specific backup as parent.","commit_id":"b6d77597c3340f789d3a7ca2053bf47f1202d235"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"207a17bf45ae8121399496b8d0d05515e1756e0e","unresolved":false,"context_lines":[{"line_number":155,"context_line":""},{"line_number":156,"context_line":"References"},{"line_number":157,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":158,"context_line":"- The blueprint of list-backups-return-parent-id:"},{"line_number":159,"context_line":"https://review.openstack.org/#/c/239364/"}],"source_content_type":"text/x-rst","patch_set":11,"id":"fa80f949_3c3dac2b","line":158,"updated":"2015-11-04 14:51:00.000000000","message":"The review link you provided below is a patch that implements the blueprint, not the blueprint itself.  Please change the text to reflect that.","commit_id":"2f6691e3bfaeeacf8de0eb7b3871212ed11f6a16"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"c86e90e0a6dc854f08f38cb90ac83f5e7e94877e","unresolved":false,"context_lines":[{"line_number":28,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"List backups or show backup return backup parent_id can make the"},{"line_number":31,"context_line":"caller(user/operator) known the dependency relationship between a"},{"line_number":32,"context_line":"volume backups."},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"The parent_id also help horizon or other openstack portal to draw"}],"source_content_type":"text/x-rst","patch_set":13,"id":"ba8a016a_1c79ebfc","line":31,"updated":"2015-11-16 16:03:21.000000000","message":"nit: s/a//","commit_id":"b5e2b1d7ea098d6f0c940e36118d64c9efe69477"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"c86e90e0a6dc854f08f38cb90ac83f5e7e94877e","unresolved":false,"context_lines":[{"line_number":31,"context_line":"caller(user/operator) known the dependency relationship between a"},{"line_number":32,"context_line":"volume backups."},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"The parent_id also help horizon or other openstack portal to draw"},{"line_number":35,"context_line":"the backups relationship graph."},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"Proposed change"}],"source_content_type":"text/x-rst","patch_set":13,"id":"ba8a016a_7c74b7fc","line":34,"updated":"2015-11-16 16:03:21.000000000","message":"nit: s/help/helps","commit_id":"b5e2b1d7ea098d6f0c940e36118d64c9efe69477"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"c86e90e0a6dc854f08f38cb90ac83f5e7e94877e","unresolved":false,"context_lines":[{"line_number":53,"context_line":"REST API impact"},{"line_number":54,"context_line":"---------------"},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"Have impact to Cinder api:"},{"line_number":57,"context_line":"1. GET /v2/{tenant_id}/backups/detail"},{"line_number":58,"context_line":"2. GET /v2/{tenant_id}/backups/{backup_id}"},{"line_number":59,"context_line":"The two api will return one more field named with \"parent_id\""}],"source_content_type":"text/x-rst","patch_set":13,"id":"ba8a016a_dcd043cc","line":56,"updated":"2015-11-16 16:03:21.000000000","message":"nit: s/Have/Has","commit_id":"b5e2b1d7ea098d6f0c940e36118d64c9efe69477"}]}