vmbuilder team mailing list archive

[Scott Moser <smoser@xxxxxxxxxxxxx>]

Scott Moser has proposed merging lp:~smoser/vmbuilder/lp795358 into lp:vmbuilder.

Requested reviews:
  VMBuilder (vmbuilder)
Related bugs:
  Bug #795358 in VMBuilder: "vmbuilder script fails on deploy/cleanup"
  https://bugs.launchpad.net/vmbuilder/+bug/795358

For more details, see:
https://code.launchpad.net/~smoser/vmbuilder/lp795358/+merge/65704
-- 
https://code.launchpad.net/~smoser/vmbuilder/lp795358/+merge/65704
Your team VMBuilder is requested to review the proposed merge of lp:~smoser/vmbuilder/lp795358 into lp:vmbuilder.

=== modified file 'VMBuilder/plugins/libvirt/__init__.py'
--- VMBuilder/plugins/libvirt/__init__.py	2010-06-15 19:10:04 +0000
+++ VMBuilder/plugins/libvirt/__init__.py	2011-06-23 18:03:42 +0000
@@ -77,7 +77,7 @@
         else:
             vmxml = VMBuilder.util.render_template('libvirt', self.context, 'libvirtxml', tmpl_ctxt)
 
-        if hostname in self.all_domains() and not self.vm.overwrite:
+        if hostname in self.all_domains() and not self.context.overwrite:
             raise VMBuilderUserError('Domain %s already exists at %s' % (hostname, libvirt_uri))
         else:
             self.conn.defineXML(vmxml)