)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":1207,"name":"Duncan Thomas","email":"duncan.thomas@gmail.com","username":"duncan-thomas"},"change_message_id":"6ec5a2b5a75fed496c7d0a43c90835605185c09e","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2016-03-13 04:22:21 -0400"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add backup import from file to cinderclient api"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I1f3e2d183a988959616efec442bc5948ee8da5bf"},{"line_number":10,"context_line":"Implements: blueprint cinder-backup-import-record-from-backup-metadata-file"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3afc51ec_39a86a1f","line":8,"updated":"2016-03-13 13:14:16.000000000","message":"Please add a sentence or two explaining the change, so that people reading the log don\u0027t have to go following links to get a basic idea of what was changed and why.","commit_id":"4b6d9b3517353d4a3e5c72fb5485d02a9e5baf40"},{"author":{"_account_id":1207,"name":"Duncan Thomas","email":"duncan.thomas@gmail.com","username":"duncan-thomas"},"change_message_id":"6ec5a2b5a75fed496c7d0a43c90835605185c09e","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Add backup import from file to cinderclient api"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I1f3e2d183a988959616efec442bc5948ee8da5bf"},{"line_number":10,"context_line":"Implements: blueprint cinder-backup-import-record-from-backup-metadata-file"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3afc51ec_99637e1a","line":10,"range":{"start_line":10,"start_character":0,"end_line":10,"end_character":10},"updated":"2016-03-13 13:14:16.000000000","message":"Please add a depends-on: line for the server-side change","commit_id":"4b6d9b3517353d4a3e5c72fb5485d02a9e5baf40"}],"cinderclient/tests/unit/v2/test_shell.py":[{"author":{"_account_id":16708,"name":"Kendall Nelson","display_name":"Kendall (diablo_rojo)","email":"kennelson11@gmail.com","username":"kjnelson"},"change_message_id":"70ac0378dc28423e586de546ace611276c4f60c4","unresolved":false,"context_lines":[{"line_number":476,"context_line":"            raise"},{"line_number":477,"context_line":"        self.run_command(\u0027backup-import-record-from-backup-metadata-file %s\u0027"},{"line_number":478,"context_line":"        % path)"},{"line_number":479,"context_line":"        expected \u003d \\"},{"line_number":480,"context_line":"        {\u0027backup-record\u0027: {\u0027backup_service\u0027: \u0027fake.driver\u0027,"},{"line_number":481,"context_line":"        \u0027backup_url\u0027: \u0027URL_STRING\u0027}}"},{"line_number":482,"context_line":"        self.assert_called(\u0027POST\u0027, \u0027/backups/import_record\u0027, expected)"}],"source_content_type":"text/x-python","patch_set":4,"id":"5a18252c_3c4ce3c6","line":479,"range":{"start_line":479,"start_character":19,"end_line":479,"end_character":20},"updated":"2016-04-12 20:09:28.000000000","message":"Can you use () instead of the \\ here?","commit_id":"75b858a19a5dc17546f3fd800cc4ef4e2ebdbba6"}],"cinderclient/v2/shell.py":[{"author":{"_account_id":1207,"name":"Duncan Thomas","email":"duncan.thomas@gmail.com","username":"duncan-thomas"},"change_message_id":"6ec5a2b5a75fed496c7d0a43c90835605185c09e","unresolved":false,"context_lines":[{"line_number":1522,"context_line":"    \"\"\"Import backup metadata record from backup_metadata file.\"\"\""},{"line_number":1523,"context_line":"    with open(args.file_path) as data_file:"},{"line_number":1524,"context_line":"        data \u003d json.load(data_file)"},{"line_number":1525,"context_line":"        backup_url \u003d data.get(\u0027extra_metadata\u0027).get(\u0027encode_record\u0027)"},{"line_number":1526,"context_line":"        backup_service \u003d data.get(\u0027extra_metadata\u0027).get(\u0027driver_name\u0027)"},{"line_number":1527,"context_line":"    info \u003d cs.backups.import_record(backup_service, backup_url)"},{"line_number":1528,"context_line":"    info.pop(\u0027links\u0027, None)"}],"source_content_type":"text/x-python","patch_set":1,"id":"3afc51ec_b9bbda71","line":1525,"range":{"start_line":1525,"start_character":26,"end_line":1525,"end_character":29},"updated":"2016-03-13 13:14:16.000000000","message":"We need some error handling here to explain to the user the problem if the backup metadata is too old to have this record added","commit_id":"4b6d9b3517353d4a3e5c72fb5485d02a9e5baf40"},{"author":{"_account_id":1207,"name":"Duncan Thomas","email":"duncan.thomas@gmail.com","username":"duncan-thomas"},"change_message_id":"76cdb4cc9b0eae6387a8e1d403fc0eba5f04711d","unresolved":false,"context_lines":[{"line_number":1507,"context_line":"@utils.arg(\u0027backup_url\u0027, metavar\u003d\u0027\u003cbackup_url\u003e\u0027,"},{"line_number":1508,"context_line":"           help\u003d\u0027Backup URL for importing the backup metadata.\u0027)"},{"line_number":1509,"context_line":"@utils.service_type(\u0027volumev2\u0027)"},{"line_number":1510,"context_line":"def do_backup_import(cs, args):"},{"line_number":1511,"context_line":"    \"\"\"Import backup metadata record.\"\"\""},{"line_number":1512,"context_line":"    info \u003d cs.backups.import_record(args.backup_service, args.backup_url)"},{"line_number":1513,"context_line":"    info.pop(\u0027links\u0027, None)"}],"source_content_type":"text/x-python","patch_set":4,"id":"9a061dce_40a0ff90","line":1510,"range":{"start_line":1510,"start_character":4,"end_line":1510,"end_character":20},"updated":"2016-04-04 10:25:37.000000000","message":"I think it would be better to add a file option to this command than to create a completely separate command.","commit_id":"75b858a19a5dc17546f3fd800cc4ef4e2ebdbba6"}]}