← Back to team overview

vmbuilder team mailing list archive

[Merge] lp:~wt-penguintechs-org/vmbuilder/vmbuilder_add_debug_logs into lp:vmbuilder

 

Warren Turkal has proposed merging lp:~wt-penguintechs-org/vmbuilder/vmbuilder_add_debug_logs into lp:vmbuilder.

Commit message:
Add a couple debug logs that seem generally useful.

Requested reviews:
  VMBuilder (vmbuilder)

For more details, see:
https://code.launchpad.net/~wt-penguintechs-org/vmbuilder/vmbuilder_add_debug_logs/+merge/152044

I just added a couple debug logs to output some useful info. Seemed general enough that I just broke it out into a separate change.
-- 
https://code.launchpad.net/~wt-penguintechs-org/vmbuilder/vmbuilder_add_debug_logs/+merge/152044
Your team VMBuilder is requested to review the proposed merge of lp:~wt-penguintechs-org/vmbuilder/vmbuilder_add_debug_logs into lp:vmbuilder.
=== modified file 'VMBuilder/contrib/cli.py'
--- VMBuilder/contrib/cli.py	2012-11-23 11:06:06 +0000
+++ VMBuilder/contrib/cli.py	2013-03-06 19:49:21 +0000
@@ -153,9 +153,12 @@
             if os.geteuid() != 0:
                 raise VMBuilderUserError('Must run as root')
 
+            logging.debug("Launch directory: {}".format(os.getcwd()))
+
             distro.overwrite = hypervisor.overwrite = self.options.overwrite
             destdir = self.options.destdir or ('%s-%s' % (distro.arg,
                                                           hypervisor.arg))
+            logging.debug("Output destdir: {}".format(destdir))
 
             if self.options.tmpfs and self.options.chroot_dir:
                 raise VMBuilderUserError('--chroot-dir and --tmpfs can not be used together.')


Follow ups