From 698cd3f4246c4e472c306fcad57d8d7b6431333f Mon Sep 17 00:00:00 2001 From: Moshe Date: Sun, 4 Mar 2018 15:42:23 +0200 Subject: Integrate dovetail with ONAP Change-Id: I6a1fa5f27df2f5127eb00156b3135dc79caf5e77 Issue-ID: DOVETAIL-629 Signed-off-by: Moshe --- dovetail/container.py | 38 ++++++++++++++++++++++++++- dovetail/report.py | 69 +++++++++++++++++++++++++++++++++++++++++++++++-- dovetail/run.py | 7 +++++ dovetail/test_runner.py | 8 ++++++ dovetail/testcase.py | 10 +++++++ 5 files changed, 129 insertions(+), 3 deletions(-) (limited to 'dovetail') diff --git a/dovetail/container.py b/dovetail/container.py index 66923e6a..5c16d802 100644 --- a/dovetail/container.py +++ b/dovetail/container.py @@ -18,7 +18,7 @@ class Container(object): container_list = {} has_pull_latest_image = {'yardstick': False, 'functest': False, - 'bottlenecks': False} + 'bottlenecks': False, 'vnftest': False} logger = None @@ -139,6 +139,23 @@ class Container(object): .format(dovetail_config['report_dest'])) return "{} {} {}".format(docker_vol, env, report) + @classmethod + def set_vnftest_config(cls): + dovetail_config = dt_cfg.dovetail_config + + log_vol = '-v %s:%s ' % (dovetail_config['result_dir'], + dovetail_config["vnftest"]['result']['log']) + + key_file = os.path.join(dovetail_config['config_dir'], + dovetail_config['pri_key']) + key_container_path = dovetail_config["vnftest"]['result']['key_path'] + if not os.path.isfile(key_file): + cls.logger.debug("Key file {} is not found".format(key_file)) + key_vol = '' + else: + key_vol = '-v %s:%s ' % (key_file, key_container_path) + return "%s %s" % (log_vol, key_vol) + @classmethod def create(cls, type, testcase_name): dovetail_config = dt_cfg.dovetail_config @@ -170,6 +187,8 @@ class Container(object): config = cls.set_yardstick_config() if type.lower() == "bottlenecks": config = cls.set_bottlenecks_config(testcase_name) + if type.lower() == "vnftest": + config = cls.set_vnftest_config() if not config: return None @@ -214,6 +233,8 @@ class Container(object): if type.lower() == 'yardstick': cls.set_yardstick_conf_file(container_id) + elif type.lower() == 'vnftest': + cls.set_vnftest_conf_file(container_id) return container_id @@ -322,6 +343,13 @@ class Container(object): cmd = 'cp %s %s' % (src_path, dest_path) return cls.exec_cmd(container_id, cmd, exit_on_error) + @classmethod + def docker_copy(cls, container_id, src_path, dest_path): + if not src_path or not dest_path: + return (1, 'src_path or dest_path is empty') + cmd = 'docker cp %s %s:%s' % (src_path, container_id, dest_path) + return dt_utils.exec_cmd(cmd, cls.logger) + @classmethod def set_yardstick_conf_file(cls, container_id): valid_type = 'yardstick' @@ -336,3 +364,11 @@ class Container(object): if url.lower() == 'file': cmd = ("sed -i '13s/http/file/g' {}".format(dest)) cls.exec_cmd(container_id, cmd) + + @classmethod + def set_vnftest_conf_file(cls, container_id): + valid_type = 'vnftest' + for conf_file in dt_cfg.dovetail_config[valid_type]['vnftest_conf']: + src = conf_file['src_file'] + dest = conf_file['dest_file'] + cls.docker_copy(container_id, src, dest) diff --git a/dovetail/report.py b/dovetail/report.py index 9d0517b4..dea9e426 100644 --- a/dovetail/report.py +++ b/dovetail/report.py @@ -27,7 +27,8 @@ from testcase import Testcase class Report(object): - results = {'functest': {}, 'yardstick': {}, 'bottlenecks': {}, 'shell': {}} + results = {'functest': {}, 'yardstick': {}, + 'bottlenecks': {}, 'shell': {}, 'vnftest': {}} logger = None @@ -427,11 +428,56 @@ class ShellCrawler(object): return None +class VnftestCrawler(object): + + logger = None + + def __init__(self): + self.type = 'vnftest' + self.logger.debug('Create crawler: {}'.format(self.type)) + + @classmethod + def create_log(cls): + cls.logger = \ + dt_logger.Logger(__name__ + '.VnftestCrawler').getLogger() + + def crawl(self, testcase): + report_dest = dt_cfg.dovetail_config['report_dest'] + if report_dest.lower() == 'file': + return self.crawl_from_file(testcase) + + if report_dest.lower().startswith('http'): + return self.crawl_from_url(testcase) + + def crawl_from_file(self, testcase): + + file_path = os.path.join(dt_cfg.dovetail_config['result_dir'], + testcase.name() + '.out') + if not os.path.exists(file_path): + self.logger.error('Result file not found: {}'.format(file_path)) + return None + criteria = 'FAIL' + with open(file_path, 'r') as f: + for jsonfile in f: + data = json.loads(jsonfile) + try: + criteria = data['result']['criteria'] + except KeyError as e: + self.logger.exception('Pass flag not found {}'.format(e)) + json_results = {'criteria': criteria} + self.logger.debug('Results: {}'.format(str(json_results))) + return json_results + + def crawl_from_url(self, testcase=None): + return None + + class CrawlerFactory(object): CRAWLER_MAP = {'functest': FunctestCrawler, 'yardstick': YardstickCrawler, 'bottlenecks': BottlenecksCrawler, + 'vnftest': VnftestCrawler, 'shell': ShellCrawler} @classmethod @@ -558,12 +604,31 @@ class ShellChecker(object): testcase.passed(False) +class VnftestChecker(object): + + logger = None + + @classmethod + def create_log(cls): + cls.logger = \ + dt_logger.Logger(__name__ + '.VnftestCheckers').getLogger() + + @staticmethod + def check(testcase, result): + if not result: + testcase.passed('FAIL') + else: + testcase.passed(result['criteria']) + return + + class CheckerFactory(object): CHECKER_MAP = {'functest': FunctestChecker, 'yardstick': YardstickChecker, 'bottlenecks': BottlenecksChecker, - 'shell': ShellChecker} + 'shell': ShellChecker, + 'vnftest': VnftestChecker} @classmethod def create(cls, type): diff --git a/dovetail/run.py b/dovetail/run.py index 1537fb6b..13e365c9 100755 --- a/dovetail/run.py +++ b/dovetail/run.py @@ -21,7 +21,9 @@ from container import Container from dovetail import constants from parser import Parser from report import BottlenecksChecker, FunctestChecker, YardstickChecker +from report import VnftestChecker from report import BottlenecksCrawler, FunctestCrawler, YardstickCrawler +from report import VnftestCrawler from report import Report from test_runner import DockerRunner, ShellRunner from testcase import Testcase @@ -97,6 +99,8 @@ def check_tc_result(testcase, logger): result_file = os.path.join(result_dir, functest_result) elif validate_type.lower() == 'bottlenecks': result_file = os.path.join(result_dir, testcase.name() + '.out') + elif validate_type.lower() == 'vnftest': + result_file = os.path.join(result_dir, testcase.name() + '.out') else: logger.error("Don't support {} now.".format(validate_type)) return @@ -180,9 +184,11 @@ def create_logs(): Report.create_log() FunctestCrawler.create_log() YardstickCrawler.create_log() + VnftestCrawler.create_log() BottlenecksCrawler.create_log() FunctestChecker.create_log() YardstickChecker.create_log() + VnftestChecker.create_log() BottlenecksChecker.create_log() Testcase.create_log() Testsuite.create_log() @@ -265,6 +271,7 @@ def main(*args, **kwargs): os.environ['DEBUG'] = 'true' create_logs() logger = dt_logger.Logger('run').getLogger() + logger.info('================================================') logger.info('Dovetail compliance: {}!'.format(kwargs['testsuite'])) logger.info('================================================') diff --git a/dovetail/test_runner.py b/dovetail/test_runner.py index c9260418..26b85a1e 100644 --- a/dovetail/test_runner.py +++ b/dovetail/test_runner.py @@ -260,6 +260,13 @@ class ShellRunner(object): 'exception: {}'.format(result_filename, e)) +class VnftestRunner(DockerRunner): + + def __init__(self, testcase): + self.type = 'vnftest' + super(VnftestRunner, self).__init__(testcase) + + class TestRunnerFactory(object): TEST_RUNNER_MAP = { @@ -267,6 +274,7 @@ class TestRunnerFactory(object): "yardstick": YardstickRunner, "bottlenecks": BottlenecksRunner, "shell": ShellRunner, + "vnftest": VnftestRunner } @classmethod diff --git a/dovetail/testcase.py b/dovetail/testcase.py index c9aef0e7..86f8061d 100644 --- a/dovetail/testcase.py +++ b/dovetail/testcase.py @@ -325,12 +325,22 @@ class ShellTestcase(Testcase): self.type = 'shell' +class VnftestTestcase(Testcase): + + validate_testcase_list = {} + + def __init__(self, testcase_yaml): + super(VnftestTestcase, self).__init__(testcase_yaml) + self.type = 'vnftest' + + class TestcaseFactory(object): TESTCASE_TYPE_MAP = { 'functest': FunctestTestcase, 'yardstick': YardstickTestcase, 'bottlenecks': BottlenecksTestcase, 'shell': ShellTestcase, + 'vnftest': VnftestTestcase } @classmethod -- cgit 1.2.3-korg