From 95aec0a10931ecc61981dba49c93c5bea8d48960 Mon Sep 17 00:00:00 2001
From: Alex Yang <yangyang1@zte.com.cn>
Date: Tue, 5 Sep 2017 17:32:23 +0800
Subject: Eliminate hard coding about ipmi info

1. Eliminate hard coding
2. Add ipmi info in schemas
3. Add unit test

Change-Id: I4561bbc9454e02fd1de2106645db341a4129245b
Signed-off-by: Alex Yang <yangyang1@zte.com.cn>
---
 deploy/config/schemas.py | 17 +++++++++--------
 deploy/environment.py    | 18 ++++++++----------
 2 files changed, 17 insertions(+), 18 deletions(-)

(limited to 'deploy')

diff --git a/deploy/config/schemas.py b/deploy/config/schemas.py
index 3096d017..0e013eb9 100644
--- a/deploy/config/schemas.py
+++ b/deploy/config/schemas.py
@@ -7,6 +7,7 @@
 # http://www.apache.org/licenses/LICENSE-2.0
 ##############################################################################
 
+import copy
 from jsonschema import Draft4Validator, FormatChecker
 import sys
 import yaml
@@ -73,6 +74,10 @@ deploy_schema = {
     'required': ['hosts', 'daisy_passwd', 'daisy_gateway']
 }
 
+deploy_schema_bm = copy.deepcopy(deploy_schema)
+deploy_schema_bm['properties']['hosts']['items']['required'] = \
+    ['roles', 'ipmi_ip', 'ipmi_user', 'ipmi_pass']
+
 
 def _validate(data, schema):
     v = Draft4Validator(schema, format_checker=FormatChecker())
@@ -80,15 +85,11 @@ def _validate(data, schema):
     return errors
 
 
-def item_validate(data, schema_type):
-    if schema_type not in schema_mapping:
-        return str('Schema Type %s does not exist' % schema_type)
-    else:
-        return _validate(data, schema_mapping.get(schema_type))
-
-
 def deploy_schema_validate(data):
-    return _validate(data, deploy_schema)
+    if data.get('adapter', 'libvirt') == 'ipmi':
+        return _validate(data, deploy_schema_bm)
+    else:
+        return _validate(data, deploy_schema)
 
 
 def _main():
diff --git a/deploy/environment.py b/deploy/environment.py
index dd9e0142..5371e6ca 100644
--- a/deploy/environment.py
+++ b/deploy/environment.py
@@ -147,17 +147,15 @@ class BareMetalEnvironment(DaisyEnvironmentBase):
                   disks=[self.daisy_server_info['image']])
 
     def reboot_nodes(self, boot_dev=None):
-        # TODO: add ipmi info into deploy.yml, or read from PDF
-        address = 106
         for node in self.deploy_struct['hosts']:
-            node['ipmiIp'] = '192.168.1.' + str(address)
-            address += 1
-            if address > 111:
-                err_exit('the ipmi address exceeds the range 106~110')
-            node['ipmiUser'] = 'zteroot'
-            node['ipmiPass'] = 'superuser'
-            ipmi_reboot_node(node['ipmiIp'], node['ipmiUser'],
-                             node['ipmiPass'], boot_source=boot_dev)
+            if 'ipmi_ip' not in node \
+                    or 'ipmi_user' not in node \
+                    or 'ipmi_pass' not in node:
+                err_exit('Missing ipmi information')
+            ipmi_reboot_node(node['ipmi_ip'],
+                             node['ipmi_user'],
+                             node['ipmi_pass'],
+                             boot_source=boot_dev)
 
     def deploy(self, deploy_file, net_file):
         self.server.prepare_cluster(deploy_file, net_file)
-- 
cgit 1.2.3-korg