)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":16708,"name":"Kendall Nelson","display_name":"Kendall (diablo_rojo)","email":"kennelson11@gmail.com","username":"kjnelson"},"change_message_id":"24dae08a8cf25a18f344ba3132fe3f3a401666bc","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"improve the function _create_volume_from_image"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Adding a buildin function which name is get_image_id_by_type"},{"line_number":10,"context_line":"to get image_id by container_format rather than hard code."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I302c69f11b29b396b2196994b9f4a16396408ee2"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"dab17558_b146dcbd","line":9,"range":{"start_line":9,"start_character":9,"end_line":9,"end_character":16},"updated":"2016-05-16 14:48:10.000000000","message":"\u0027buildin\u0027 -\u003e \u0027built in\u0027","commit_id":"149288ca5d9a8f9115cbbb6d78102e0f2dba2a1d"}],"cinder/tests/unit/test_rbd.py":[{"author":{"_account_id":16708,"name":"Kendall Nelson","display_name":"Kendall (diablo_rojo)","email":"kennelson11@gmail.com","username":"kjnelson"},"change_message_id":"24dae08a8cf25a18f344ba3132fe3f3a401666bc","unresolved":false,"context_lines":[{"line_number":1139,"context_line":""},{"line_number":1140,"context_line":"        # See tests.unit.image.fake for image types."},{"line_number":1141,"context_line":"        # container_format: raw, ami, None, ova"},{"line_number":1142,"context_line":"        def get_image_id_by_type(container_format\u003d\u0027raw\u0027):"},{"line_number":1143,"context_line":"            image_ids \u003d image_fake.FakeImageService().images"},{"line_number":1144,"context_line":"            for image_id in image_ids:"},{"line_number":1145,"context_line":"                image \u003d image_fake.FakeImageService().show(None, image_id)"},{"line_number":1146,"context_line":"                if image[\u0027container_format\u0027] \u003d\u003d container_format:"},{"line_number":1147,"context_line":"                    return image_id"},{"line_number":1148,"context_line":""},{"line_number":1149,"context_line":"        if raw:"},{"line_number":1150,"context_line":"            image_id \u003d get_image_id_by_type(container_format\u003d\u0027raw\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"dab17558_f14a2482","line":1147,"range":{"start_line":1142,"start_character":8,"end_line":1147,"end_character":35},"updated":"2016-05-16 14:48:10.000000000","message":"Shouldn\u0027t this be outside the _create_volume_from_image function? and then call it inside here?","commit_id":"149288ca5d9a8f9115cbbb6d78102e0f2dba2a1d"},{"author":{"_account_id":14305,"name":"Yuriy Nesenenko","email":"ynesenenko@mirantis.com","username":"yuriy_n"},"change_message_id":"f7f5f899455e0b232ab4b16c5395590a027d1813","unresolved":false,"context_lines":[{"line_number":1149,"context_line":"        if raw:"},{"line_number":1150,"context_line":"            image_id \u003d get_image_id_by_type(container_format\u003d\u0027raw\u0027)"},{"line_number":1151,"context_line":"        else:"},{"line_number":1152,"context_line":"            image_id \u003d get_image_id_by_type(container_format\u003dNone)"},{"line_number":1153,"context_line":""},{"line_number":1154,"context_line":"        # creating volume testdata"},{"line_number":1155,"context_line":"        db_volume \u003d {\u0027display_description\u0027: \u0027Test Desc\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"dab17558_5c8d00d2","line":1152,"range":{"start_line":1152,"start_character":65,"end_line":1152,"end_character":66},"updated":"2016-05-16 16:09:27.000000000","message":"What about another format qcow2?","commit_id":"149288ca5d9a8f9115cbbb6d78102e0f2dba2a1d"}]}
