Code review comment for lp:~nttdata/nova/lp746821

Revision history for this message
Brian Lamar (blamar) wrote :

Good stuff Kei! Just a few really minor nitpicks:

Line 67 -- might be considered unnecessary spacing.

Line 74 -- I'd suggest a LOG.debug in there for the libvirt error, just in case someone needs to know exactly what is failing. I've been bitten by that in the past.

review: Needs Fixing

« Back to merge proposal