# HG changeset patch # User "Bryan M. Kearney <bkearney@xxxxxxxxxx>" # Date 1215011615 14400 # Node ID 3e86a75795e73277056bfcea0c54b97f7a5945e9 # Parent 5d9cf6c2662494d21d84fbafbc0b7980096b1998 Add a --remove flag to allow virt-image to install new images when an existing one with the same name is defined diff -r 5d9cf6c26624 -r 3e86a75795e7 virt-image --- a/virt-image Wed Jul 02 10:50:05 2008 +0100 +++ b/virt-image Wed Jul 02 11:13:35 2008 -0400 @@ -142,7 +142,10 @@ def parse_args(): help=_("The zero-based index of the boot record to use")) parser.add_option("", "--force", action="store_true", dest="force", help=_("Do not prompt for input. Answers yes where applicable, terminates for all other prompts"), - default=False) + default=False) + parser.add_option("", "--replace",action="store_true", dest="replace", + help=_("Overwrite, or destroy, an existing image with the same name"), + default=False) (options,args) = parser.parse_args() if len(args) < 1: @@ -211,7 +214,7 @@ def main(): try: print _("\n\nCreating guest %s...") % guest.name - dom = guest.start_install(None, progresscb) + dom = guest.start_install(None, progresscb, options.replace) if dom is None: print _("Guest creation failed") sys.exit(1) diff -r 5d9cf6c26624 -r 3e86a75795e7 virtinst/Guest.py --- a/virtinst/Guest.py Wed Jul 02 10:50:05 2008 +0100 +++ b/virtinst/Guest.py Wed Jul 02 11:13:35 2008 -0400 @@ -914,7 +914,7 @@ class Guest(object): "action": action } - def start_install(self, consolecb = None, meter = None): + def start_install(self, consolecb = None, meter = None, removeOld = False): """Do the startup of the guest installation.""" self.validate_parms() @@ -924,7 +924,7 @@ class Guest(object): self._prepare_install(meter) try: - return self._do_install(consolecb, meter) + return self._do_install(consolecb, meter, removeOld) finally: self._installer.cleanup() @@ -932,10 +932,22 @@ class Guest(object): self._install_disks = self.disks[:] self._install_nics = self.nics[:] - def _do_install(self, consolecb, meter): + def _do_install(self, consolecb, meter ,removeOld = False): try: - if self.conn.lookupByName(self.name) is not None: - raise RuntimeError, _("Domain named %s already exists!") %(self.name,) + vm = self.conn.lookupByName(self.name) + if removeOld: + if vm is not None: + try: + if vm.ID() != -1: + logging.info("Destroying image %s" %(self.name)) + vm.destroy() + logging.info("Removing old definition for image %s" %(self.name)) + vm.undefine() + except libvirt.libvirtError, e: + raise RuntimeError, _("Could not remove old vm '%s': %s") %(self.name, str(e)) + else: + if vm is not None: + raise RuntimeError, _("Domain named %s already exists!") %(self.name,) except libvirt.libvirtError: pass _______________________________________________ et-mgmt-tools mailing list et-mgmt-tools@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/et-mgmt-tools