)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":14274,"name":"XinXiaohui","email":"xiaohui.xin@intel.com","username":"XinXiaohui"},"change_message_id":"98270ef54796e0d685c5ecb63e29c130d364caf8","unresolved":false,"context_lines":[{"line_number":10,"context_line":"status of the volume to \"error\" if it is caused by other"},{"line_number":11,"context_line":"problems like without expected backup status and backup service."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"The status of Volume is error which means it unable to use."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: I6145cf7793ff77af05670fb0e8c411d9f3711526"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5a5ae5dd_0702d26f","line":13,"range":{"start_line":13,"start_character":45,"end_line":13,"end_character":51},"updated":"2016-02-04 03:43:38.000000000","message":"s/Volume/volume\ns/unable/is unable","commit_id":"fc39145ab82d207ddf8e4334d50db108db6fe6b9"}],"cinder/backup/manager.py":[{"author":{"_account_id":15961,"name":"lisali","email":"xiaoyan.li@intel.com","username":"lisali"},"change_message_id":"e742e9ea89474a8bea8875a452f51c8595ffd677","unresolved":false,"context_lines":[{"line_number":461,"context_line":"                   {\u0027expected_status\u0027: expected_status,"},{"line_number":462,"context_line":"                    \u0027actual_status\u0027: actual_status})"},{"line_number":463,"context_line":"            self._update_backup_error(backup, context, err)"},{"line_number":464,"context_line":"            self.db.volume_update(context, volume_id, {\u0027status\u0027: \u0027available\u0027})"},{"line_number":465,"context_line":"            raise exception.InvalidBackup(reason\u003derr)"},{"line_number":466,"context_line":""},{"line_number":467,"context_line":"        if volume[\u0027size\u0027] \u003e backup[\u0027size\u0027]:"}],"source_content_type":"text/x-python","patch_set":2,"id":"5a5ae5dd_bb97528f","line":464,"updated":"2016-02-04 08:22:02.000000000","message":"I don\u0027t think it is better to put the volume in available status. \n\nIt may make users misunderstand that the restore is completed. But in fact data is not correct in this volume.","commit_id":"f24bb628680e91c94c3288bee8471ad8fddde718"},{"author":{"_account_id":20105,"name":"Cao ShuFeng","email":"caosf.fnst@cn.fujitsu.com","username":"caosf.fnst"},"change_message_id":"3f8675f440ca1c43b96c33c3d6ba13b95a0af066","unresolved":false,"context_lines":[{"line_number":394,"context_line":"                   {\u0027expected_status\u0027: expected_status,"},{"line_number":395,"context_line":"                    \u0027actual_status\u0027: actual_status})"},{"line_number":396,"context_line":"            self._update_backup_error(backup, context, err)"},{"line_number":397,"context_line":"            self.db.volume_update(context, volume_id, {\u0027status\u0027: \u0027error_restoring\u0027})"},{"line_number":398,"context_line":"            raise exception.InvalidBackup(reason\u003derr)"},{"line_number":399,"context_line":""},{"line_number":400,"context_line":"        if volume[\u0027size\u0027] \u003e backup[\u0027size\u0027]:"}],"source_content_type":"text/x-python","patch_set":6,"id":"9abb7d3a_428a92c0","line":397,"range":{"start_line":397,"start_character":12,"end_line":397,"end_character":84},"updated":"2016-05-30 11:03:55.000000000","message":"Jenkins fails several times because this line and the 421st line is too long.\nPlease run \"tox -e pep8\" before you commit change to Gerrit.\n\nhttp://logs.openstack.org/42/275542/6/check/gate-cinder-pep8/58a80ea/console.html#_2016-05-30_09_45_43_744","commit_id":"93fdf717c526b09fb106fdb73f9574b2ca13ffc2"},{"author":{"_account_id":9171,"name":"Vipin Balachandran","email":"vipin.bl@gmail.com","username":"vbala"},"change_message_id":"31f6c5a93fc956bd485f6f9715c272d80c917f46","unresolved":false,"context_lines":[{"line_number":420,"context_line":"            backup.status \u003d fields.BackupStatus.AVAILABLE"},{"line_number":421,"context_line":"            backup.save()"},{"line_number":422,"context_line":"            self.db.volume_update(context, volume_id,"},{"line_number":423,"context_line":"                                  {\u0027status\u0027: \u0027error_restoring\u0027})"},{"line_number":424,"context_line":"            raise exception.InvalidBackup(reason\u003derr)"},{"line_number":425,"context_line":""},{"line_number":426,"context_line":"        try:"}],"source_content_type":"text/x-python","patch_set":7,"id":"9abb7d3a_1e026cb4","line":423,"range":{"start_line":423,"start_character":45,"end_line":423,"end_character":62},"updated":"2016-05-31 12:02:11.000000000","message":"As Duncan said, if there is an error in restoring, the volume may be corrupted and unusable, and we should use the status \u0027error_restoring\u0027 to indicate only that (line 431). So this should be left as \u0027error\u0027 to make a distinction.","commit_id":"66c2c323a6667d9a2eae59f8cc6d6ed90793db40"}]}