Re: [Libguestfs] [PATCH] v2v: remove MAC address related information

On Tue, Feb 27, 2018 at 12:53:08PM +0100, Pino Toscano wrote: We preserve the MAC address in metadata. On the other hand AIUI this patch only removes the association in the ifcfg files and the guest will reassociate it when it boots (albeit it might then mix up the ethernet interfaces so that's not good). There's IMHO a bigger problem which is not being addressed: https://bugzilla.redhat.com/show_bug.cgi?id=1318922 There's a bit of background which is missing. Tomáš and I had some discussions (privately, unfortunately) with the ManageIQ developers who are integrating virt-v2v into MIQ/CloudForms. Their existing software runs a separate virt-sysprep step on guests after they have been converted by virt-v2v. They disable all sysprep the operations except for just a couple, including ‘net-hwaddr’, so the effect is roughly the same as this patch. The question was raised why they need to do that as a separate step and why virt-v2v doesn't do it. And indeed there was some discussion about whether or not converted guests need a new MAC address -- it's at best unclear -- it is thought that VMware might reuse MAC addresses which have "left" the hypervisor, although no one knows if that's really true or not. I don't have much opinion on this. Maybe it's best for CFME to continue to run virt-sysprep as a separate step. Rich. -- Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones Read my programming and virtualization blog: http://rwmj.wordpress.com libguestfs lets you edit virtual machines. Supports shell scripting, bindings from many languages. http://libguestfs.org