)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":12112,"name":"Tom Swanson","email":"tom_swanson@dell.com","username":"tomswanson"},"change_message_id":"cca5b8cf971b3799259a5b288f14450ac140505d","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2017-01-12 12:25:15 -0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add support to search snapshots based on metadata key, value pair."},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The snpapshot-list API for cinder gives a list of snapshots based"},{"line_number":10,"context_line":"on certain criteria to the user. From microversion 3.22 onwards"},{"line_number":11,"context_line":"the snapshot-list API has been enhanced to support snapshot list"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"7a3c09a3_e30e90e4","line":8,"updated":"2017-01-12 23:36:25.000000000","message":"The first line should be limited to 50 characters and should not end with a period.","commit_id":"ec818b52e07c6f63a6bc8658b566084a046fdc8e"}],"cinderclient/v3/shell.py":[{"author":{"_account_id":23602,"name":"Michael Dovgal","email":"dovgalmichael@gmail.com","username":"mdovgal"},"change_message_id":"a943d8ae9efc56918384bc88606bb34fb9ea57ec","unresolved":false,"context_lines":[{"line_number":1282,"context_line":""},{"line_number":1283,"context_line":"    if args.display_name is not None:"},{"line_number":1284,"context_line":"        args.name \u003d args.display_name"},{"line_number":1285,"context_line":"    print(\"In do snapshot list\" + str(shell_utils.extract_metadata(args)))"},{"line_number":1286,"context_line":"    search_opts \u003d {"},{"line_number":1287,"context_line":"        \u0027all_tenants\u0027: all_tenants,"},{"line_number":1288,"context_line":"        \u0027name\u0027: args.name,"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a57fde8_d004e8c7","line":1285,"range":{"start_line":1285,"start_character":4,"end_line":1285,"end_character":74},"updated":"2017-01-12 15:12:27.000000000","message":"-1\nI think we should get rid of this print.","commit_id":"5135a29ddc2ad48cfb0f4d9455cf54edacd07c2f"},{"author":{"_account_id":23602,"name":"Michael Dovgal","email":"dovgalmichael@gmail.com","username":"mdovgal"},"change_message_id":"a36c62f3d48c4fbc4f46856bc52e3772a5aec367","unresolved":false,"context_lines":[{"line_number":1282,"context_line":""},{"line_number":1283,"context_line":"    if args.display_name is not None:"},{"line_number":1284,"context_line":"        args.name \u003d args.display_name"},{"line_number":1285,"context_line":"    print(\"In do snapshot list\" + str(shell_utils.extract_metadata(args)))"},{"line_number":1286,"context_line":"    search_opts \u003d {"},{"line_number":1287,"context_line":"        \u0027all_tenants\u0027: all_tenants,"},{"line_number":1288,"context_line":"        \u0027name\u0027: args.name,"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a57fde8_dbfec904","line":1285,"range":{"start_line":1285,"start_character":4,"end_line":1285,"end_character":74},"in_reply_to":"9a57fde8_d004e8c7","updated":"2017-01-12 15:16:03.000000000","message":"Actually, this NoneType error is raised because of this line","commit_id":"5135a29ddc2ad48cfb0f4d9455cf54edacd07c2f"},{"author":{"_account_id":19774,"name":"Vivek Agrawal","email":"vivek.agrawal@veritas.com","username":"viveksagrawal"},"change_message_id":"c662164ed1263e34783fa982546ed39d6842d9f3","unresolved":false,"context_lines":[{"line_number":1282,"context_line":""},{"line_number":1283,"context_line":"    if args.display_name is not None:"},{"line_number":1284,"context_line":"        args.name \u003d args.display_name"},{"line_number":1285,"context_line":"    print(\"In do snapshot list\" + str(shell_utils.extract_metadata(args)))"},{"line_number":1286,"context_line":"    search_opts \u003d {"},{"line_number":1287,"context_line":"        \u0027all_tenants\u0027: all_tenants,"},{"line_number":1288,"context_line":"        \u0027name\u0027: args.name,"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a3c09a3_c22a727e","line":1285,"range":{"start_line":1285,"start_character":4,"end_line":1285,"end_character":74},"in_reply_to":"9a57fde8_d004e8c7","updated":"2017-01-12 22:14:26.000000000","message":"Done","commit_id":"5135a29ddc2ad48cfb0f4d9455cf54edacd07c2f"},{"author":{"_account_id":19774,"name":"Vivek Agrawal","email":"vivek.agrawal@veritas.com","username":"viveksagrawal"},"change_message_id":"c662164ed1263e34783fa982546ed39d6842d9f3","unresolved":false,"context_lines":[{"line_number":1282,"context_line":""},{"line_number":1283,"context_line":"    if args.display_name is not None:"},{"line_number":1284,"context_line":"        args.name \u003d args.display_name"},{"line_number":1285,"context_line":"    print(\"In do snapshot list\" + str(shell_utils.extract_metadata(args)))"},{"line_number":1286,"context_line":"    search_opts \u003d {"},{"line_number":1287,"context_line":"        \u0027all_tenants\u0027: all_tenants,"},{"line_number":1288,"context_line":"        \u0027name\u0027: args.name,"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a3c09a3_e21f6e59","line":1285,"range":{"start_line":1285,"start_character":4,"end_line":1285,"end_character":74},"in_reply_to":"9a57fde8_dbfec904","updated":"2017-01-12 22:14:26.000000000","message":"Done","commit_id":"5135a29ddc2ad48cfb0f4d9455cf54edacd07c2f"},{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"b1d56a3cbfec8bdc9f3017d0fe89acdeab2a09c9","unresolved":false,"context_lines":[{"line_number":1297,"context_line":"                                         limit\u003dargs.limit,"},{"line_number":1298,"context_line":"                                         sort\u003dargs.sort)"},{"line_number":1299,"context_line":"    shell_utils.translate_volume_snapshot_keys(snapshots)"},{"line_number":1300,"context_line":"    if args.sort:"},{"line_number":1301,"context_line":"        sortby_index \u003d None"},{"line_number":1302,"context_line":"    else:"},{"line_number":1303,"context_line":"        sortby_index \u003d 0"}],"source_content_type":"text/x-python","patch_set":5,"id":"7a3c09a3_83b33097","line":1300,"updated":"2017-01-18 11:28:08.000000000","message":"It could be simplified to:\nsortby_index \u003d none if args.sort else 0","commit_id":"c6b0e4504c7ec9e810542a2caf475b95a9deb372"},{"author":{"_account_id":19774,"name":"Vivek Agrawal","email":"vivek.agrawal@veritas.com","username":"viveksagrawal"},"change_message_id":"a5bb680491efe8686534fa3870b1d852854d875b","unresolved":false,"context_lines":[{"line_number":1297,"context_line":"                                         limit\u003dargs.limit,"},{"line_number":1298,"context_line":"                                         sort\u003dargs.sort)"},{"line_number":1299,"context_line":"    shell_utils.translate_volume_snapshot_keys(snapshots)"},{"line_number":1300,"context_line":"    if args.sort:"},{"line_number":1301,"context_line":"        sortby_index \u003d None"},{"line_number":1302,"context_line":"    else:"},{"line_number":1303,"context_line":"        sortby_index \u003d 0"}],"source_content_type":"text/x-python","patch_set":5,"id":"7a3c09a3_465a2bf3","line":1300,"in_reply_to":"7a3c09a3_83b33097","updated":"2017-01-18 14:29:44.000000000","message":"Done","commit_id":"c6b0e4504c7ec9e810542a2caf475b95a9deb372"}]}