Correct the mapping of nodes and roles in virtual deploy

JIRA: DAISY-56

Now the roles are assigned to nodes randomly, because the function
'add_hosts_interface' in tempest.py just uses zip to map the
host's name to the host.

libvirt_utils.py: get mac addresses from VM
environment.py: save the mac addresses
daisy_server.py: write a new deploy.yml which contains the mac addresses
    and copy the file to daisy server
get_conf.py: get mac addresses from the new deploy.yml
tempest.py: assigned roles to nodes when the mac addresses matched
controller.xml: increase the RAM to make difference with computer nodes
deploy.sh: apply this change to bash script

Change-Id: Ia61b60d39d319c5d01e3505727fafc63a0585858
Signed-off-by: Alex Yang <yangyang1@zte.com.cn>

5 files changed, 42 insertions, 15 deletions

diff --git a/deploy/daisy_server.py b/deploy/daisy_server.py
index eb711afd..a899ea2b 100644
--- a/deploy/daisy_server.py
+++ b/deploy/daisy_server.py
@@ -10,7 +10,9 @@
 import os
 import paramiko
 import scp
+import tempfile
 import time
+import yaml
 
 from utils import (
     WORKSPACE,
@@ -230,6 +232,13 @@ class DaisyServer(object):
             net_file=path_join(self.remote_dir, self.net_file_name))
         self.ssh_run(cmd, check=True)
 
+    def copy_new_deploy_config(self, data):
+        (dummy, conf_file) = tempfile.mkstemp()
+        with open(conf_file, 'w') as fh:
+            fh.write(yaml.safe_dump(data))
+            fh.flush()
+            self.scp_put(conf_file, path_join(self.remote_dir, self.deploy_file_name))
+
     def prepare_host_and_pxe(self):
         LI('Prepare host and PXE')
         cmd = "python {script} --dha {deploy_file} --network {net_file} --host \'yes\' --isbare {is_bare} --scenario {scenario}".format(
diff --git a/deploy/environment.py b/deploy/environment.py
index abea6210..87c16937 100644
--- a/deploy/environment.py
+++ b/deploy/environment.py
@@ -26,7 +26,8 @@ from libvirt_utils import (
     reboot_vm,
     delete_vm_and_disk,
     create_virtual_network,
-    delete_virtual_network
+    delete_virtual_network,
+    get_vm_mac_addresses
 )
 from utils import (
     WORKSPACE,
@@ -252,7 +253,7 @@ class VirtualEnvironment(DaisyEnvironmentBase):
             create_virtual_disk(ceph_disk_file, ceph_size)
             disks.append(ceph_disk_file)
 
-        create_vm(template, name, disks)
+        return create_vm(template, name, disks)
 
     def create_nodes(self):
         # TODO: support virtNetTemplatePath in deploy.yml
@@ -263,7 +264,8 @@ class VirtualEnvironment(DaisyEnvironmentBase):
         self._daisy_keepalived_net = net_name
 
         for node in self.deploy_struct['hosts']:
-            self.create_virtual_node(node)
+            domain = self.create_virtual_node(node)
+            node['mac_addresses'] = get_vm_mac_addresses(domain)
         time.sleep(20)
 
     def reboot_nodes(self, boot_devs=None):
@@ -298,6 +300,7 @@ class VirtualEnvironment(DaisyEnvironmentBase):
     def deploy(self, deploy_file, net_file):
         self.server.prepare_cluster(deploy_file, net_file)
         self.create_nodes()
+        self.server.copy_new_deploy_config(self.deploy_struct)
         self.server.prepare_host_and_pxe()
         LI('Begin Daisy virtual-deploy os and openstack')
         self.reboot_nodes()
diff --git a/deploy/get_conf.py b/deploy/get_conf.py
index ca9e2cf6..1f4500a4 100755
--- a/deploy/get_conf.py
+++ b/deploy/get_conf.py
@@ -81,6 +81,13 @@ def host(host=None):
     return map
 
 
+@decorator_mk('hosts')
+def mac_address(host=None):
+    mac_addresses = host.get('mac_addresses', [])
+    map = {host['name']: mac_addresses}
+    return map
+
+
 def network_config_parse(s, dha_file):
     network_map = network(s)
     vip = s.get('internal_vip')
@@ -100,9 +107,10 @@ def config(dha_file, network_file):
     data = init(dha_file)
     ceph_disk_name = data.get('ceph_disk_name')
     hosts_name = dha_config_parse(data, dha_file)
+    mac_address_map = mac_address(data)
     data = init(network_file)
     network_map, vip, interface_map = network_config_parse(data, network_file)
-    return interface_map, hosts_name, network_map, vip, ceph_disk_name
+    return interface_map, hosts_name, network_map, vip, ceph_disk_name, mac_address_map
 
 
 def parse(conf, args):
diff --git a/deploy/libvirt_utils.py b/deploy/libvirt_utils.py
index 8c694c13..8e4523de 100644
--- a/deploy/libvirt_utils.py
+++ b/deploy/libvirt_utils.py
@@ -104,9 +104,10 @@ def create_vm(template, name=None, disks=None):
     if domain.create() < 0:
         err_exit('Failed to start VM %s' % template)
     domain.setAutostart(1)
+    conn.close()
 
     LI('VM %s is started' % domain.name())
-    return
+    return domain
 
 
 def reboot_vm(vm_name, boot_devs=None):
@@ -123,18 +124,13 @@ def reboot_vm(vm_name, boot_devs=None):
             vm.destroy()
             LI('Destroy VM %s' % vm_name)
 
-        # root = ET.fromstring(vm.XMLDesc())
-        temp_file = path_join(WORKSPACE, 'tmp.xml')
-        commands.getoutput('virsh dumpxml %s > %s' % (vm_name, temp_file))
-        tree = ET.parse(temp_file)
-        root = tree.getroot()
+        root = ET.fromstring(vm.XMLDesc())
         LI('Modify the boot order %s' % boot_devs)
         modify_vm_boot_order(root, boot_devs)
-        tree.write(temp_file)
 
         LI('Re-define and start the VM %s' % vm_name)
         vm.undefine()
-        vm = conn.defineXML(commands.getoutput('cat %s' % temp_file))
+        vm = conn.defineXML(ET.tostring(root))
         vm.create()
         vm.setAutostart(1)
     else:
@@ -222,3 +218,9 @@ def delete_virtual_network(network_xml):
     conn.close()
     if not result:
         LI('Network %s is not found' % name)
+
+
+def get_vm_mac_addresses(domain):
+    root = ET.fromstring(domain.XMLDesc())
+    macs = root.findall('./devices/interface/mac')
+    return [mac.attrib['address'] for mac in macs]
diff --git a/deploy/tempest.py b/deploy/tempest.py
index ebd040ba..3c1a6c7d 100644
--- a/deploy/tempest.py
+++ b/deploy/tempest.py
@@ -68,7 +68,7 @@ def prepare_install():
         print("get config...")
         conf = cfg.ConfigOpts()
         parse(conf, sys.argv[1:])
-        host_interface_map, hosts_name, network_map, vip, ceph_disk_name = \
+        host_interface_map, hosts_name, network_map, vip, ceph_disk_name, mac_address_map = \
             get_conf.config(conf['dha'], conf['network'])
         if conf['cluster'] and conf['cluster'] == 'yes':
             print("add cluster...")
@@ -90,7 +90,7 @@ def prepare_install():
             hosts_info = get_hosts()
             cluster_info = get_cluster()
             cluster_id = cluster_info.id
-            add_hosts_interface(cluster_id, hosts_info, hosts_name,
+            add_hosts_interface(cluster_id, hosts_info, hosts_name, mac_address_map,
                                 host_interface_map, vip, isbare)
             if len(hosts_name) == 1:
                 protocol_type = 'LVM'
@@ -183,7 +183,8 @@ def get_cluster():
     return cluster_info
 
 
-def add_hosts_interface(cluster_id, hosts_info, hosts_name, host_interface_map,
+def add_hosts_interface(cluster_id, hosts_info, hosts_name, mac_address_map,
+                        host_interface_map,
                         vip, isbare):
     for host_name, host in zip(hosts_name, hosts_info):
         host = host.to_dict()
@@ -196,6 +197,10 @@ def add_hosts_interface(cluster_id, hosts_info, hosts_name, host_interface_map,
             if interface_name in host_interface_map:
                 interface['assigned_networks'] = \
                     host_interface_map[interface_name]
+            if mac_address_map:
+                for nodename in mac_address_map:
+                    if interface['mac'] in mac_address_map[nodename]:
+                        host_name = nodename
         pathlist = os.listdir(iso_path)
         for filename in pathlist:
             if filename.endswith('iso'):