Use VIF.move only if just the network is changed and not the MAC addr…

…ess. If the MAC is changed, recreate the interface instead.

Backgound: VIF.move can just move a VIF to another network, but not change the MAC properly (and this is also not expected from an action named move).

#128

XenModel/Actions/VM/ImportVmAction.cs

@@ -191,14 +191,16 @@ protected override void Run()
                         if (network != null)
                             networks.Add(network);
 
+                        string oldmac = VIF.get_MAC(Session, vif);
+
                         if (canMoveVifs)
                         {
                             var vifObj = Connection.Resolve(vif);
                             if (vifObj == null)
                                 continue;
                             // try to find a matching VIF in the m_proxyVIFs list, based on the device field
                             var matchingProxyVif = m_VIFs.FirstOrDefault(proxyVIF => proxyVIF.device == vifObj.device);
-                            if (matchingProxyVif != null)
+                            if (matchingProxyVif != null && matchingProxyVif.MAC == oldmac)
                             {
                                 // move the VIF to the desired network
                                 VIF.move(Session, vif, matchingProxyVif.network);