Code review comment for lp:~morita-kazutaka/nova/snapshot-volume

Revision history for this message
Brian Waldon (bcwaldon) wrote :

Looking good! I'm more than happy to send this in once a couple minor things are fixed:

108: This description doesn't seem right. Is this a copy/paste error?

Should the "snapshot_name_template" be used in places like line 23, 58, 93, 388 and 419? This is more of a question, as I do not actually know the answer :)

review: Needs Fixing

« Back to merge proposal