)]}'
{"cinder/volume/flows/manager/create_volume.py":[{"author":{"_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},"change_message_id":"3dc61f06fe8c8428f5e2510be8a1453ef6478d64","unresolved":false,"context_lines":[{"line_number":786,"context_line":"                    self.driver.extend_volume(volume, original_size)"},{"line_number":787,"context_line":"                finally:"},{"line_number":788,"context_line":"                    # if failed extending volume, volume size should be"},{"line_number":789,"context_line":"                    # put back."},{"line_number":790,"context_line":"                    volume.size \u003d original_size"},{"line_number":791,"context_line":"                    volume.save()"},{"line_number":792,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"bff0334d_49d7f524","line":789,"updated":"2017-04-07 14:08:55.000000000","message":"should we log a message here about setting the size to it\u0027s original requested size because extend failed?","commit_id":"52aab525917d491389049cf9de40ffa7bb0c78f7"},{"author":{"_account_id":17564,"name":"jiamin","email":"shljia@cn.ibm.com","username":"jiamin"},"change_message_id":"a5befa243e04dd02ea5f2a3bac23b98bc8c15daa","unresolved":false,"context_lines":[{"line_number":786,"context_line":"                    self.driver.extend_volume(volume, original_size)"},{"line_number":787,"context_line":"                finally:"},{"line_number":788,"context_line":"                    # if failed extending volume, volume size should be"},{"line_number":789,"context_line":"                    # put back."},{"line_number":790,"context_line":"                    volume.size \u003d original_size"},{"line_number":791,"context_line":"                    volume.save()"},{"line_number":792,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"bff0334d_06965340","line":789,"in_reply_to":"bff0334d_49d7f524","updated":"2017-04-10 08:42:26.000000000","message":"Thanks Walter, I think it is better to merge this comment into the comment above, so I change it in the new patch. but considering that no matter extending volume failed or not, original size should be put back(put it back is not caused by extending volume failed), log here may be not necessary.","commit_id":"52aab525917d491389049cf9de40ffa7bb0c78f7"}]}