)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},"change_message_id":"678a4ca48a4da0cfc88ca4a69eabd0433f427ef9","unresolved":false,"context_lines":[{"line_number":10,"context_line":"would be properly tested."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I26d8d2f1fe6cf7362d23d9b9668936dbb0509251"},{"line_number":13,"context_line":"Closes-Bug: #1604892"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3ac371cc_d663e9da","line":13,"range":{"start_line":13,"start_character":0,"end_line":13,"end_character":20},"updated":"2016-08-16 08:13:34.000000000","message":"As the bug is reported in cinder project,this bug NO maybe wrong in project cinderclient.can you create a new bug report in cinderclient?","commit_id":"b5f2fa6144fcc806f5290a0f86603b0f4b55160d"},{"author":{"_account_id":7173,"name":"Scott DAngelo","email":"scott.dangelo@gmail.com","username":"scottda"},"change_message_id":"f1292d2a0238caafed0b9b2c5ce47d154ce87dad","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Changed backup-restore to accept backup name"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Fixing error in patch rebase. Edited test to ensure that this"},{"line_number":10,"context_line":"would be properly tested."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I26d8d2f1fe6cf7362d23d9b9668936dbb0509251"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3ac371cc_dfa1c795","line":9,"updated":"2016-08-17 21:51:30.000000000","message":"I don\u0027t think this information is useful. You can just delete L#9-10","commit_id":"83a1660a92b4c88e70c834ea438dfcd22f9a7593"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"f42b1881e247ebb50b5ebbbb98255a2ea13b3e18","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Changed backup-restore to accept backup name"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I26d8d2f1fe6cf7362d23d9b9668936dbb0509251"},{"line_number":10,"context_line":"Closes-Bug: #1604892"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"1ac06dbe_1241f6a9","line":9,"updated":"2016-08-22 10:34:48.000000000","message":"nit: This commit message is too brief, it would be good to mention that it doesn\u0027t currently accept it unlike other backup actions and that the description says it does.","commit_id":"7ad67cba06f91ae138477b603d107db1fb2d421a"}],"cinderclient/tests/unit/v2/test_shell.py":[{"author":{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},"change_message_id":"678a4ca48a4da0cfc88ca4a69eabd0433f427ef9","unresolved":false,"context_lines":[{"line_number":449,"context_line":"    @mock.patch(\u0027cinderclient.v3.shell._find_backup\u0027)"},{"line_number":450,"context_line":"    @mock.patch(\u0027cinderclient.utils.print_dict\u0027)"},{"line_number":451,"context_line":"    @mock.patch(\u0027cinderclient.utils.find_volume\u0027)"},{"line_number":452,"context_line":"    def test_do_backup_restore(self,"},{"line_number":453,"context_line":"                               mock_find_volume,"},{"line_number":454,"context_line":"                               mock_print_dict,"},{"line_number":455,"context_line":"                               mock_find_backup):"}],"source_content_type":"text/x-python","patch_set":1,"id":"3ac371cc_9e92234a","line":452,"range":{"start_line":452,"start_character":8,"end_line":452,"end_character":30},"updated":"2016-08-16 08:13:34.000000000","message":"So, this testcase will not cover the situation when restoring by backup_id?","commit_id":"b5f2fa6144fcc806f5290a0f86603b0f4b55160d"},{"author":{"_account_id":7173,"name":"Scott DAngelo","email":"scott.dangelo@gmail.com","username":"scottda"},"change_message_id":"f1292d2a0238caafed0b9b2c5ce47d154ce87dad","unresolved":false,"context_lines":[{"line_number":449,"context_line":"    @mock.patch(\u0027cinderclient.v3.shell._find_backup\u0027)"},{"line_number":450,"context_line":"    @mock.patch(\u0027cinderclient.utils.print_dict\u0027)"},{"line_number":451,"context_line":"    @mock.patch(\u0027cinderclient.utils.find_volume\u0027)"},{"line_number":452,"context_line":"    def test_do_backup_restore(self,"},{"line_number":453,"context_line":"                               mock_find_volume,"},{"line_number":454,"context_line":"                               mock_print_dict,"},{"line_number":455,"context_line":"                               mock_find_backup):"}],"source_content_type":"text/x-python","patch_set":2,"id":"3ac371cc_9f5c6f7e","line":452,"updated":"2016-08-17 21:51:30.000000000","message":"Better to name this:\ntest_do_backup_restore_with_name","commit_id":"83a1660a92b4c88e70c834ea438dfcd22f9a7593"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"f42b1881e247ebb50b5ebbbb98255a2ea13b3e18","unresolved":false,"context_lines":[{"line_number":449,"context_line":"    @mock.patch(\u0027cinderclient.v3.shell._find_backup\u0027)"},{"line_number":450,"context_line":"    @mock.patch(\u0027cinderclient.utils.print_dict\u0027)"},{"line_number":451,"context_line":"    @mock.patch(\u0027cinderclient.utils.find_volume\u0027)"},{"line_number":452,"context_line":"    def test_do_backup_restore_with_name(self,"},{"line_number":453,"context_line":"                               mock_find_volume,"},{"line_number":454,"context_line":"                               mock_print_dict,"},{"line_number":455,"context_line":"                               mock_find_backup):"}],"source_content_type":"text/x-python","patch_set":3,"id":"1ac06dbe_52052eba","line":452,"updated":"2016-08-22 10:34:48.000000000","message":"-1: This is the same as test_do_backup_restore_with_id except we change the backup value in the input.  Please use ddt module instead.","commit_id":"7ad67cba06f91ae138477b603d107db1fb2d421a"}]}
