Code review comment for lp:~zulcss/nova/nova-lxc

Revision history for this message
Jay Pipes (jaypipes) wrote :

Hi Chuck!

149 + utils.execute('mkdir -p %s' % container_dir)

Does the above need a sudo?

Also, I note that there are no test cases included for this new functionality. Could we add some?

Cheers!
jay

review: Needs Information

« Back to merge proposal