)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"89dba874a9066b1e73df3f2b500acec50e418d9f","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"\"volume list\" is not in the v2."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Closes-Bug: #1476193"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I444d27629450578c9ecd7fb93a9db99dc015313a"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"3a50d1a3_df9478e3","line":11,"updated":"2015-07-21 14:13:22.000000000","message":"reference the exisitng blueprint\n\n  implements bp: volume-v2","commit_id":"05b21ebf0d5958f1f28d39c82e7055337ddafcf0"}],"openstackclient/volume/v2/volume.py":[{"author":{"_account_id":8736,"name":"Terry Howe","email":"terrylhowe@gmail.com","username":"terrylhowe"},"change_message_id":"6463256cb334e7aac22a446bba9a55a99c5587b3","unresolved":false,"context_lines":[{"line_number":70,"context_line":"        parser.add_argument("},{"line_number":71,"context_line":"            \u0027--status\u0027,"},{"line_number":72,"context_line":"            metavar\u003d\u0027\u003cstatus\u003e\u0027,"},{"line_number":73,"context_line":"            help\u003d\u0027Filter results by status\u0027,"},{"line_number":74,"context_line":"        )"},{"line_number":75,"context_line":"        parser.add_argument("},{"line_number":76,"context_line":"            \u0027--name\u0027,"}],"source_content_type":"text/x-python","patch_set":4,"id":"3a50d1a3_5673adb1","line":73,"updated":"2015-07-20 15:41:07.000000000","message":"Couldn\u0027t people use grep if they want to filter.  Do we do this kind of search filtering anywhere else?","commit_id":"ac8c58a96a86004d851b83ef6cc2c4bbe20e3f54"},{"author":{"_account_id":1941,"name":"Lin Hua Cheng","email":"os.lcheng@gmail.com","username":"lin-hua-cheng"},"change_message_id":"9fe8ab96f5a28ded83d1da02aca1da113baf58b5","unresolved":false,"context_lines":[{"line_number":70,"context_line":"        parser.add_argument("},{"line_number":71,"context_line":"            \u0027--status\u0027,"},{"line_number":72,"context_line":"            metavar\u003d\u0027\u003cstatus\u003e\u0027,"},{"line_number":73,"context_line":"            help\u003d\u0027Filter results by status\u0027,"},{"line_number":74,"context_line":"        )"},{"line_number":75,"context_line":"        parser.add_argument("},{"line_number":76,"context_line":"            \u0027--name\u0027,"}],"source_content_type":"text/x-python","patch_set":4,"id":"3a50d1a3_7d7d53b0","line":73,"in_reply_to":"3a50d1a3_5673adb1","updated":"2015-07-21 00:07:48.000000000","message":"we do in the identity commands.\n\nSince this is server side filtering, I think it is fine to expose it too","commit_id":"ac8c58a96a86004d851b83ef6cc2c4bbe20e3f54"},{"author":{"_account_id":8736,"name":"Terry Howe","email":"terrylhowe@gmail.com","username":"terrylhowe"},"change_message_id":"6463256cb334e7aac22a446bba9a55a99c5587b3","unresolved":false,"context_lines":[{"line_number":75,"context_line":"        parser.add_argument("},{"line_number":76,"context_line":"            \u0027--name\u0027,"},{"line_number":77,"context_line":"            metavar\u003d\u0027\u003cname\u003e\u0027,"},{"line_number":78,"context_line":"            help\u003d\u0027Filter results by name\u0027,"},{"line_number":79,"context_line":"        )"},{"line_number":80,"context_line":"        parser.add_argument("},{"line_number":81,"context_line":"            \u0027--all-projects\u0027,"}],"source_content_type":"text/x-python","patch_set":4,"id":"3a50d1a3_764d8971","line":78,"updated":"2015-07-20 15:41:07.000000000","message":"People should use show if they want to filter by name.","commit_id":"ac8c58a96a86004d851b83ef6cc2c4bbe20e3f54"},{"author":{"_account_id":1941,"name":"Lin Hua Cheng","email":"os.lcheng@gmail.com","username":"lin-hua-cheng"},"change_message_id":"9fe8ab96f5a28ded83d1da02aca1da113baf58b5","unresolved":false,"context_lines":[{"line_number":75,"context_line":"        parser.add_argument("},{"line_number":76,"context_line":"            \u0027--name\u0027,"},{"line_number":77,"context_line":"            metavar\u003d\u0027\u003cname\u003e\u0027,"},{"line_number":78,"context_line":"            help\u003d\u0027Filter results by name\u0027,"},{"line_number":79,"context_line":"        )"},{"line_number":80,"context_line":"        parser.add_argument("},{"line_number":81,"context_line":"            \u0027--all-projects\u0027,"}],"source_content_type":"text/x-python","patch_set":4,"id":"3a50d1a3_2890eb42","line":78,"in_reply_to":"3a50d1a3_764d8971","updated":"2015-07-21 00:07:48.000000000","message":"I think the display_name is not unique in cinder?","commit_id":"ac8c58a96a86004d851b83ef6cc2c4bbe20e3f54"},{"author":{"_account_id":8736,"name":"Terry Howe","email":"terrylhowe@gmail.com","username":"terrylhowe"},"change_message_id":"6463256cb334e7aac22a446bba9a55a99c5587b3","unresolved":false,"context_lines":[{"line_number":107,"context_line":"            msg \u003d \u0027\u0027"},{"line_number":108,"context_line":"            for attachment in attachments:"},{"line_number":109,"context_line":"                server \u003d attachment[\u0027server_id\u0027]"},{"line_number":110,"context_line":"                if server in server_cache.keys():"},{"line_number":111,"context_line":"                    server \u003d server_cache[server].name"},{"line_number":112,"context_line":"                device \u003d attachment[\u0027device\u0027]"},{"line_number":113,"context_line":"                msg +\u003d \u0027Attached to %s on %s \u0027 % (server, device)"}],"source_content_type":"text/x-python","patch_set":4,"id":"3a50d1a3_76602936","line":110,"updated":"2015-07-20 15:41:07.000000000","message":"Don\u0027t technically need keys() there.","commit_id":"ac8c58a96a86004d851b83ef6cc2c4bbe20e3f54"},{"author":{"_account_id":1941,"name":"Lin Hua Cheng","email":"os.lcheng@gmail.com","username":"lin-hua-cheng"},"change_message_id":"9fe8ab96f5a28ded83d1da02aca1da113baf58b5","unresolved":false,"context_lines":[{"line_number":131,"context_line":"                \u0027Size\u0027,"},{"line_number":132,"context_line":"                \u0027Type\u0027,"},{"line_number":133,"context_line":"                \u0027Bootable\u0027,"},{"line_number":134,"context_line":"                \u0027Attached to\u0027,"},{"line_number":135,"context_line":"                \u0027Properties\u0027,"},{"line_number":136,"context_line":"            )"},{"line_number":137,"context_line":"        else:"}],"source_content_type":"text/x-python","patch_set":4,"id":"3a50d1a3_c822a701","line":134,"updated":"2015-07-21 00:07:48.000000000","message":"duplicated list with only \"attachments\" as the diff.\n\nYou can use the pattern here to reduce the duplicate string: https://github.com/openstack/python-openstackclient/blob/master/openstackclient/identity/v3/user.py#L263-L268","commit_id":"ac8c58a96a86004d851b83ef6cc2c4bbe20e3f54"},{"author":{"_account_id":8736,"name":"Terry Howe","email":"terrylhowe@gmail.com","username":"terrylhowe"},"change_message_id":"4248cd2efa7d344f7627c8112770ac37e6bb41d8","unresolved":false,"context_lines":[{"line_number":75,"context_line":"        parser.add_argument("},{"line_number":76,"context_line":"            \u0027--name\u0027,"},{"line_number":77,"context_line":"            metavar\u003d\u0027\u003cname\u003e\u0027,"},{"line_number":78,"context_line":"            help\u003d\u0027Filter results by name\u0027,"},{"line_number":79,"context_line":"        )"},{"line_number":80,"context_line":"        parser.add_argument("},{"line_number":81,"context_line":"            \u0027--all-projects\u0027,"}],"source_content_type":"text/x-python","patch_set":7,"id":"3a50d1a3_922108a2","line":78,"updated":"2015-07-21 14:32:06.000000000","message":"Maybe it is time for a general option that could be used for all list commands like --query that takes name value pairs?","commit_id":"05b21ebf0d5958f1f28d39c82e7055337ddafcf0"}]}