)]}'
{"magnum/common/cinder.py":[{"author":{"_account_id":6484,"name":"Feilong Wang","email":"hustemb@gmail.com","username":"flwang"},"change_message_id":"55d925162661c6c45097f127cf09a8c2bd0b2ac4","unresolved":false,"context_lines":[{"line_number":37,"context_line":"            _get_random_volume_type(context, size))"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"def _get_random_volume_type(context, size):"},{"line_number":41,"context_line":"    if int(size) \u003e 0:"},{"line_number":42,"context_line":"        c_client \u003d clients.OpenStackClients(context).cinder()"},{"line_number":43,"context_line":"        volume_types \u003d c_client.volume_types.list()"}],"source_content_type":"text/x-python","patch_set":3,"id":"3fa7e38b_f7b718d7","line":40,"updated":"2019-11-22 01:15:15.000000000","message":"Pass in the \u0027size\u0027 to here looks weird for me. How about just checking the size where doing the volume type setting instead of changing it here?","commit_id":"6848dc52fde168808fc0d24817ce94ea8184eb63"},{"author":{"_account_id":28022,"name":"Bharat Kunwar","email":"brtknr@bath.edu","username":"brtknr"},"change_message_id":"35f1192e15467f1ab5ad2b207c0ad450af26f7b7","unresolved":false,"context_lines":[{"line_number":37,"context_line":"            _get_random_volume_type(context, size))"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"def _get_random_volume_type(context, size):"},{"line_number":41,"context_line":"    if int(size) \u003e 0:"},{"line_number":42,"context_line":"        c_client \u003d clients.OpenStackClients(context).cinder()"},{"line_number":43,"context_line":"        volume_types \u003d c_client.volume_types.list()"}],"source_content_type":"text/x-python","patch_set":3,"id":"3fa7e38b_60f4b687","line":40,"in_reply_to":"3fa7e38b_f7b718d7","updated":"2019-11-22 17:01:23.000000000","message":"Done","commit_id":"6848dc52fde168808fc0d24817ce94ea8184eb63"}]}