)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":12000,"name":"Ian Cordasco","email":"sigmavirus24@gmail.com","username":"sigmavirus24"},"change_message_id":"50bf9a3265accf30b781fb12cdc33b0e3304335a","unresolved":false,"context_lines":[{"line_number":12,"context_line":"option to glance image-list"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I85424780b2afe346b218b87949093c157e485371"},{"line_number":15,"context_line":"Closes-Bug: #1433430"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"7a016987_b9908444","line":15,"updated":"2015-05-17 23:19:06.000000000","message":"I\u0027d like to second Cindy\u0027s sugestion.","commit_id":"1ae0e8b08b792206193c34ac2866400b0b099437"},{"author":{"_account_id":9096,"name":"Cindy Pallares","email":"u1@cindy.sh","username":"cpallares"},"change_message_id":"f5d432cfe373e874ee01891fa1f0c8ac7ddb2cd6","unresolved":false,"context_lines":[{"line_number":12,"context_line":"option to glance image-list"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I85424780b2afe346b218b87949093c157e485371"},{"line_number":15,"context_line":"Closes-Bug: #1433430"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9a0c5dc1_903f3ad6","line":15,"updated":"2015-05-12 03:53:40.000000000","message":"This could probably use a DocImpact flag.\n\nIt would be nice to have an example of the command usage. \nhttps://wiki.openstack.org/wiki/Documentation/DocImpact","commit_id":"1ae0e8b08b792206193c34ac2866400b0b099437"}],"glanceclient/common/utils.py":[{"author":{"_account_id":12000,"name":"Ian Cordasco","email":"sigmavirus24@gmail.com","username":"sigmavirus24"},"change_message_id":"50bf9a3265accf30b781fb12cdc33b0e3304335a","unresolved":false,"context_lines":[{"line_number":130,"context_line":""},{"line_number":131,"context_line":""},{"line_number":132,"context_line":"def print_list(objs, fields, human_readable\u003dFalse,"},{"line_number":133,"context_line":"               formatters\u003dNone, field_settings\u003dNone):"},{"line_number":134,"context_line":"    formatters \u003d formatters or {}"},{"line_number":135,"context_line":"    field_settings \u003d field_settings or {}"},{"line_number":136,"context_line":"    pt \u003d prettytable.PrettyTable([f for f in fields], caching\u003dFalse)"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a016987_d9722049","line":133,"updated":"2015-05-17 23:19:06.000000000","message":"Nit: Even though those are optional parameters, there *may* be people importing this library. In that case they may be using positional only parameters so we should probably tack \"human_readable\u003dFalse\" onto the end of the argument list instead of placing it in the middle.","commit_id":"1ae0e8b08b792206193c34ac2866400b0b099437"}],"glanceclient/v2/shell.py":[{"author":{"_account_id":6484,"name":"Feilong Wang","email":"hustemb@gmail.com","username":"flwang"},"change_message_id":"ef8f597c05ab65a15b3baabbf0189e7a2cc728ea","unresolved":false,"context_lines":[{"line_number":135,"context_line":"           choices\u003dimages.SORT_DIR_VALUES,"},{"line_number":136,"context_line":"           help\u003d\u0027Sort image list in specified directions.\u0027)"},{"line_number":137,"context_line":"@utils.arg(\u0027--sort\u0027, metavar\u003d\u0027\u003ckey\u003e[:\u003cdirection\u003e]\u0027, default\u003dNone,"},{"line_number":138,"context_line":"           help\u003d((\u0027Comma-separated list of sort keys and directions in the \u0027"},{"line_number":139,"context_line":"                  \u0027form of \u003ckey\u003e[:\u003casc|desc\u003e]. Valid keys: %s. OPTIONAL: \u0027"},{"line_number":140,"context_line":"                  \u0027Default\u003d\"name:asc\".\u0027) % \u0027, \u0027.join(images.SORT_KEY_VALUES)))"},{"line_number":141,"context_line":"def do_image_list(gc, args):"}],"source_content_type":"text/x-python","patch_set":1,"id":"7aaa499b_c96514e2","line":138,"updated":"2015-04-12 23:30:14.000000000","message":"I think those \" to \u0027 are not related to this patch.","commit_id":"1ae0e8b08b792206193c34ac2866400b0b099437"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"efe6bb221833a53fb56ad54a02e5dfb2636eae14","unresolved":false,"context_lines":[{"line_number":135,"context_line":"           choices\u003dimages.SORT_DIR_VALUES,"},{"line_number":136,"context_line":"           help\u003d\u0027Sort image list in specified directions.\u0027)"},{"line_number":137,"context_line":"@utils.arg(\u0027--sort\u0027, metavar\u003d\u0027\u003ckey\u003e[:\u003cdirection\u003e]\u0027, default\u003dNone,"},{"line_number":138,"context_line":"           help\u003d((\u0027Comma-separated list of sort keys and directions in the \u0027"},{"line_number":139,"context_line":"                  \u0027form of \u003ckey\u003e[:\u003casc|desc\u003e]. Valid keys: %s. OPTIONAL: \u0027"},{"line_number":140,"context_line":"                  \u0027Default\u003d\"name:asc\".\u0027) % \u0027, \u0027.join(images.SORT_KEY_VALUES)))"},{"line_number":141,"context_line":"def do_image_list(gc, args):"}],"source_content_type":"text/x-python","patch_set":1,"id":"7aaa499b_df3ed8b7","line":138,"in_reply_to":"7aaa499b_c96514e2","updated":"2015-04-13 13:10:57.000000000","message":"Yes, you are right, this is not related to this patch, i updated just for consistency.","commit_id":"1ae0e8b08b792206193c34ac2866400b0b099437"},{"author":{"_account_id":9096,"name":"Cindy Pallares","email":"u1@cindy.sh","username":"cpallares"},"change_message_id":"f5d432cfe373e874ee01891fa1f0c8ac7ddb2cd6","unresolved":false,"context_lines":[{"line_number":135,"context_line":"           choices\u003dimages.SORT_DIR_VALUES,"},{"line_number":136,"context_line":"           help\u003d\u0027Sort image list in specified directions.\u0027)"},{"line_number":137,"context_line":"@utils.arg(\u0027--sort\u0027, metavar\u003d\u0027\u003ckey\u003e[:\u003cdirection\u003e]\u0027, default\u003dNone,"},{"line_number":138,"context_line":"           help\u003d((\u0027Comma-separated list of sort keys and directions in the \u0027"},{"line_number":139,"context_line":"                  \u0027form of \u003ckey\u003e[:\u003casc|desc\u003e]. Valid keys: %s. OPTIONAL: \u0027"},{"line_number":140,"context_line":"                  \u0027Default\u003d\"name:asc\".\u0027) % \u0027, \u0027.join(images.SORT_KEY_VALUES)))"},{"line_number":141,"context_line":"def do_image_list(gc, args):"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a0c5dc1_d07db2e8","line":138,"in_reply_to":"7aaa499b_df3ed8b7","updated":"2015-05-12 03:53:40.000000000","message":"Thanks!","commit_id":"1ae0e8b08b792206193c34ac2866400b0b099437"},{"author":{"_account_id":12000,"name":"Ian Cordasco","email":"sigmavirus24@gmail.com","username":"sigmavirus24"},"change_message_id":"50bf9a3265accf30b781fb12cdc33b0e3304335a","unresolved":false,"context_lines":[{"line_number":167,"context_line":""},{"line_number":168,"context_line":"    images \u003d gc.images.list(**kwargs)"},{"line_number":169,"context_line":""},{"line_number":170,"context_line":"    columns \u003d [\u0027ID\u0027, \u0027Name\u0027, \u0027Size\u0027, \u0027Status\u0027]"},{"line_number":171,"context_line":"    utils.print_list(images, columns,"},{"line_number":172,"context_line":"                     human_readable\u003dargs.human_readable)"},{"line_number":173,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"7a016987_9993884f","line":170,"updated":"2015-05-17 23:19:06.000000000","message":"I agree with Cindy. These are good additions. Adding them to the end will not break anyone\u0027s command-line parsing either if I understand this correctly.","commit_id":"1ae0e8b08b792206193c34ac2866400b0b099437"},{"author":{"_account_id":6484,"name":"Feilong Wang","email":"hustemb@gmail.com","username":"flwang"},"change_message_id":"ef8f597c05ab65a15b3baabbf0189e7a2cc728ea","unresolved":false,"context_lines":[{"line_number":167,"context_line":""},{"line_number":168,"context_line":"    images \u003d gc.images.list(**kwargs)"},{"line_number":169,"context_line":""},{"line_number":170,"context_line":"    columns \u003d [\u0027ID\u0027, \u0027Name\u0027, \u0027Size\u0027, \u0027Status\u0027]"},{"line_number":171,"context_line":"    utils.print_list(images, columns,"},{"line_number":172,"context_line":"                     human_readable\u003dargs.human_readable)"},{"line_number":173,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"7aaa499b_0973fcb2","line":170,"updated":"2015-04-12 23:30:14.000000000","message":"I think this line change is arguable. But unfortunately, I can\u0027t really remember why we only keep ID and Name, we may need to discuss it in IRC channel.","commit_id":"1ae0e8b08b792206193c34ac2866400b0b099437"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"efe6bb221833a53fb56ad54a02e5dfb2636eae14","unresolved":false,"context_lines":[{"line_number":167,"context_line":""},{"line_number":168,"context_line":"    images \u003d gc.images.list(**kwargs)"},{"line_number":169,"context_line":""},{"line_number":170,"context_line":"    columns \u003d [\u0027ID\u0027, \u0027Name\u0027, \u0027Size\u0027, \u0027Status\u0027]"},{"line_number":171,"context_line":"    utils.print_list(images, columns,"},{"line_number":172,"context_line":"                     human_readable\u003dargs.human_readable)"},{"line_number":173,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"7aaa499b_7f74649b","line":170,"in_reply_to":"7aaa499b_0973fcb2","updated":"2015-04-13 13:10:57.000000000","message":"Ok, let me know your IRC Nick name","commit_id":"1ae0e8b08b792206193c34ac2866400b0b099437"},{"author":{"_account_id":9096,"name":"Cindy Pallares","email":"u1@cindy.sh","username":"cpallares"},"change_message_id":"f5d432cfe373e874ee01891fa1f0c8ac7ddb2cd6","unresolved":false,"context_lines":[{"line_number":167,"context_line":""},{"line_number":168,"context_line":"    images \u003d gc.images.list(**kwargs)"},{"line_number":169,"context_line":""},{"line_number":170,"context_line":"    columns \u003d [\u0027ID\u0027, \u0027Name\u0027, \u0027Size\u0027, \u0027Status\u0027]"},{"line_number":171,"context_line":"    utils.print_list(images, columns,"},{"line_number":172,"context_line":"                     human_readable\u003dargs.human_readable)"},{"line_number":173,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"9a0c5dc1_b06c7e09","line":170,"in_reply_to":"7aaa499b_7f74649b","updated":"2015-05-12 03:53:40.000000000","message":"I\u0027d prefer to add the size and status, but if there\u0027s a good reason against it I\u0027d love to hear it.","commit_id":"1ae0e8b08b792206193c34ac2866400b0b099437"}]}