)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"37a0b1cac004cde3b28a3fb2eb43a2c4b75e7a38","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fixes consistency snapshot creation"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This fixes the bug where creation of consistency snapshot would pass even if"},{"line_number":10,"context_line":"the volume is in error state. Also removing force option from"},{"line_number":11,"context_line":"create_snapshots_in_db() since it is not needed."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1aa78d24_27748534","line":9,"updated":"2016-07-11 20:38:20.000000000","message":"consistency group snapshot\n\nPlease make the line shorter than 72 characters.","commit_id":"ae442d6f3ecaf5a373c77aed5c837e40fcf31b0c"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"37a0b1cac004cde3b28a3fb2eb43a2c4b75e7a38","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This fixes the bug where creation of consistency snapshot would pass even if"},{"line_number":10,"context_line":"the volume is in error state. Also removing force option from"},{"line_number":11,"context_line":"create_snapshots_in_db() since it is not needed."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Closes-Bug: #1592451"},{"line_number":14,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1aa78d24_e7a43da7","line":11,"updated":"2016-07-11 20:38:20.000000000","message":"because this function is only used by create cgsnapshot and the force flag is always True.","commit_id":"ae442d6f3ecaf5a373c77aed5c837e40fcf31b0c"}],"cinder/tests/unit/api/contrib/test_cgsnapshots.py":[{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"04499bf85b6d61f8983c780524781f6a3d6de938","unresolved":false,"context_lines":[{"line_number":216,"context_line":""},{"line_number":217,"context_line":"    @mock.patch("},{"line_number":218,"context_line":"        \u0027cinder.api.openstack.wsgi.Controller.validate_name_and_description\u0027)"},{"line_number":219,"context_line":"    def test_create_cgsnapshot_when_volume_in_error_status(self, mock_validate):"},{"line_number":220,"context_line":"        consistencygroup \u003d utils.create_consistencygroup(self.context)"},{"line_number":221,"context_line":"        utils.create_volume("},{"line_number":222,"context_line":"            self.context,"}],"source_content_type":"text/x-python","patch_set":4,"id":"1aa78d24_d44ec036","line":219,"updated":"2016-07-13 02:55:16.000000000","message":"You got a pep8 error here:\n\nE501 line too long (80 \u003e 79 characters)\n\nEach line should be \u003c\u003d 79 characters.","commit_id":"1fc5348bd720bd4bfd62208e8d0f8b25290b6220"}],"cinder/volume/api.py":[{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"b0be3f68de6001d3b64d50d09269542b4a82b998","unresolved":false,"context_lines":[{"line_number":793,"context_line":"    def create_snapshots_in_db(self, context,"},{"line_number":794,"context_line":"                               volume_list,"},{"line_number":795,"context_line":"                               name, description,"},{"line_number":796,"context_line":"                               cgsnapshot_id):"},{"line_number":797,"context_line":"        snapshot_list \u003d []"},{"line_number":798,"context_line":"        for volume in volume_list:"},{"line_number":799,"context_line":"            self._create_snapshot_in_db_validate(context, volume, True)"}],"source_content_type":"text/x-python","patch_set":2,"id":"1aa78d24_bae5280a","line":796,"updated":"2016-07-11 21:00:18.000000000","message":"I think it would be better to not change the method signatures here, and just fix the bug in this patch.","commit_id":"942801b696a854feeee0fec206aade4d1f9d13ac"}]}