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

Proposed by Wren Turkal
Status: Merged
Merged at revision: 486
Proposed branch: lp:~wt-penguintechs-org/vmbuilder/vmbuilder_add_debug_logs
Merge into: lp:vmbuilder
Diff against target: 16 lines (+3/-0)
1 file modified
VMBuilder/contrib/cli.py (+3/-0)
To merge this branch: bzr merge lp:~wt-penguintechs-org/vmbuilder/vmbuilder_add_debug_logs
Reviewer Review Type Date Requested Status
VMBuilder Pending
Review via email: mp+152044@code.launchpad.net

Commit message

Add a couple debug logs that seem generally useful.

Description of the change

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.

To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'VMBuilder/contrib/cli.py'
2--- VMBuilder/contrib/cli.py 2012-11-23 11:06:06 +0000
3+++ VMBuilder/contrib/cli.py 2013-03-06 19:49:21 +0000
4@@ -153,9 +153,12 @@
5 if os.geteuid() != 0:
6 raise VMBuilderUserError('Must run as root')
7
8+ logging.debug("Launch directory: {}".format(os.getcwd()))
9+
10 distro.overwrite = hypervisor.overwrite = self.options.overwrite
11 destdir = self.options.destdir or ('%s-%s' % (distro.arg,
12 hypervisor.arg))
13+ logging.debug("Output destdir: {}".format(destdir))
14
15 if self.options.tmpfs and self.options.chroot_dir:
16 raise VMBuilderUserError('--chroot-dir and --tmpfs can not be used together.')

Subscribers

People subscribed via source and target branches