)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":17405,"name":"Jacob Gregor","email":"jgregor@us.ibm.com","username":"jgregor"},"change_message_id":"d4fa2f02a088e6778cd61ade896d6d4acbe134d9","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add qcow2 img format when copy img to a gluster volume"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Now it\u0027s allways raw type, when we create a volume from image."},{"line_number":10,"context_line":"The type for the volume should according to the configuration"},{"line_number":11,"context_line":"of the glusterfs_qcow2_volumes rather than default raw format."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"da85f559_655de5eb","line":9,"updated":"2015-11-13 15:12:12.000000000","message":"Suggestion: \u0027Currently when we create a volume from an image it\u0027s type is always set as raw.\u0027.","commit_id":"4637af92e1bfbf4a6a33a7a18e068ad58e3901d6"},{"author":{"_account_id":17405,"name":"Jacob Gregor","email":"jgregor@us.ibm.com","username":"jgregor"},"change_message_id":"d4fa2f02a088e6778cd61ade896d6d4acbe134d9","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Add qcow2 img format when copy img to a gluster volume"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Now it\u0027s allways raw type, when we create a volume from image."},{"line_number":10,"context_line":"The type for the volume should according to the configuration"},{"line_number":11,"context_line":"of the glusterfs_qcow2_volumes rather than default raw format."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: Ic9d3fbbdc34f7dc1b8a240a99b5e0eccf7cc965b"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"da85f559_45b8e1e5","line":10,"updated":"2015-11-13 15:12:12.000000000","message":"Suggestion: Change \u0027should according to ....\u0027 to \u0027should update according to ...\u0027","commit_id":"4637af92e1bfbf4a6a33a7a18e068ad58e3901d6"},{"author":{"_account_id":17405,"name":"Jacob Gregor","email":"jgregor@us.ibm.com","username":"jgregor"},"change_message_id":"d4fa2f02a088e6778cd61ade896d6d4acbe134d9","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Now it\u0027s allways raw type, when we create a volume from image."},{"line_number":10,"context_line":"The type for the volume should according to the configuration"},{"line_number":11,"context_line":"of the glusterfs_qcow2_volumes rather than default raw format."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: Ic9d3fbbdc34f7dc1b8a240a99b5e0eccf7cc965b"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"da85f559_0579792f","line":11,"updated":"2015-11-13 15:12:12.000000000","message":"I think there should be a bug associated with this patch since you are fixing an issue.","commit_id":"4637af92e1bfbf4a6a33a7a18e068ad58e3901d6"}],"cinder/image/image_utils.py":[{"author":{"_account_id":12952,"name":"ice yao","email":"yao3690093@gmail.com","username":"xiabing.yao"},"change_message_id":"eb0d2ac1d5870e1de1a5d3e7b67a44cef2812b1b","unresolved":false,"context_lines":[{"line_number":232,"context_line":"                   user_id\u003dNone, project_id\u003dNone, size\u003dNone, run_as_root\u003dTrue):"},{"line_number":233,"context_line":"    fetch_to_volume_format(context, image_service, image_id, dest, \u0027qcow2\u0027,"},{"line_number":234,"context_line":"                           blocksize, user_id, project_id, size,"},{"line_number":235,"context_line":"                           run_as_root\u003drun_as_root)"},{"line_number":236,"context_line":""},{"line_number":237,"context_line":""},{"line_number":238,"context_line":"def fetch_to_volume_format(context, image_service,"}],"source_content_type":"text/x-python","patch_set":2,"id":"da85f559_98a3799e","line":235,"updated":"2015-11-14 04:54:49.000000000","message":"Can you use common function between fetch_to_qcow2 and fetch_to_raw?","commit_id":"4637af92e1bfbf4a6a33a7a18e068ad58e3901d6"},{"author":{"_account_id":8358,"name":"zhangguoqing","email":"zhang.guoqing@99cloud.net","username":"zhangguoqing"},"change_message_id":"ca76615a4592a8c73031af19f49e9a0f7ef57bb1","unresolved":false,"context_lines":[{"line_number":232,"context_line":"                   user_id\u003dNone, project_id\u003dNone, size\u003dNone, run_as_root\u003dTrue):"},{"line_number":233,"context_line":"    fetch_to_volume_format(context, image_service, image_id, dest, \u0027qcow2\u0027,"},{"line_number":234,"context_line":"                           blocksize, user_id, project_id, size,"},{"line_number":235,"context_line":"                           run_as_root\u003drun_as_root)"},{"line_number":236,"context_line":""},{"line_number":237,"context_line":""},{"line_number":238,"context_line":"def fetch_to_volume_format(context, image_service,"}],"source_content_type":"text/x-python","patch_set":2,"id":"da85f559_f51f2b59","line":235,"in_reply_to":"da85f559_98a3799e","updated":"2015-11-14 11:24:42.000000000","message":"I have thought about it before commit this. It\u0027s not necessary for us to use common function between fetch_to_qcow2 and fetch_to_raw. The below function of fetch_to_volume_format acts it and only two call functions now.\nThank you. :)","commit_id":"4637af92e1bfbf4a6a33a7a18e068ad58e3901d6"}]}