)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"25b533753ce6491b7fa8d6d4ace5afd49fde719a","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     shaoxj \u003cshaoxj@inspur.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2018-02-23 19:37:18 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"The precondition of specified volume can be restored"},{"line_number":8,"context_line":"the status of volume and the size of volume"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"Change-Id: I89bb72741aec5e73789a5a5cec1bc363f79c766c"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3fa0c359_39bef1ad","line":7,"updated":"2018-02-23 13:59:19.000000000","message":"The format of the commit message needs to be updated to have a summary line, followed by a blank line, then more detail. It should state what is being done by applying this change.\n\nFor more reference, see https://wiki.openstack.org/wiki/GitCommitMessages#Summary_of_Git_commit_message_structure\n\nA summary line similar to the follow would be good:\n\nApi-ref: Add target volume preconditions for backup restore\n\nSomething along those lines.","commit_id":"316ca0057c1945bcd3607d32dac36ecbb8857a30"}],"api-ref/source/v3/ext-backups.inc":[{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"25b533753ce6491b7fa8d6d4ace5afd49fde719a","unresolved":false,"context_lines":[{"line_number":182,"context_line":"The name parameter will work only if a new volume is created."},{"line_number":183,"context_line":""},{"line_number":184,"context_line":"If UUID is specified, the backup will be restored to the specified volume."},{"line_number":185,"context_line":"Precondition of specified volume can be restored"},{"line_number":186,"context_line":"- the specified volume status is ``available``."},{"line_number":187,"context_line":"- the size of specified volume not less than the size of backup."},{"line_number":188,"context_line":""}],"source_content_type":"text/x-c++src","patch_set":2,"id":"3fa0c359_19753532","line":185,"updated":"2018-02-23 13:59:19.000000000","message":"The wording of this is a little strange. How about something more like:\n\nThe specified volume has the following requirements:\n\n    * The volume status must be ``available``\n    * The size of the volume must be equal to or greater than the backup size","commit_id":"316ca0057c1945bcd3607d32dac36ecbb8857a30"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"4bee1442b954dba94831675c474be28777fb0f41","unresolved":false,"context_lines":[{"line_number":182,"context_line":"The name parameter will work only if a new volume is created."},{"line_number":183,"context_line":""},{"line_number":184,"context_line":"If UUID is specified, the backup will be restored to the specified volume."},{"line_number":185,"context_line":"The specified volume has the following requirements:"},{"line_number":186,"context_line":"* the specified volume status is ``available``."},{"line_number":187,"context_line":"* the size of specified volume must be equal to or greater than the size of backup."},{"line_number":188,"context_line":""}],"source_content_type":"text/x-c++src","patch_set":4,"id":"3fa0c359_5c915951","line":185,"range":{"start_line":185,"start_character":51,"end_line":185,"end_character":52},"updated":"2018-02-25 10:37:02.000000000","message":"I believe you need a blank line after this to format properly. Check the output http://logs.openstack.org/65/547365/4/check/build-openstack-api-ref/fda9198/html/v3/index.html#restore-a-backup","commit_id":"0e33188eae465dd5ef7227d961cd3803b368909e"}]}