From 308dc9989bce115b615fa13ff1dc44080e3a731e Mon Sep 17 00:00:00 2001 From: Periyasamy Palanisamy Date: Wed, 18 Oct 2017 12:22:11 +0200 Subject: Gather logs only if fuel or apex installer Change-Id: I11988ad86a916d1d7251aacf951c6362fda961ba Signed-off-by: Periyasamy Palanisamy --- sdnvpn/lib/utils.py | 4 +++- sdnvpn/test/functest/run_sdnvpn_tests.py | 8 +++++++- 2 files changed, 10 insertions(+), 2 deletions(-) diff --git a/sdnvpn/lib/utils.py b/sdnvpn/lib/utils.py index ad0714e..d968010 100644 --- a/sdnvpn/lib/utils.py +++ b/sdnvpn/lib/utils.py @@ -213,7 +213,9 @@ def get_installerHandler(): installer_ip = get_installer_ip() if installer_type not in ["fuel", "apex"]: - raise ValueError("%s is not supported" % installer_type) + logger.warn("installer type %s is neither fuel nor apex." + "returning None for installer handler" % installer_type) + return None else: if installer_type in ["apex"]: developHandler = DeploymentFactory.get_handler( diff --git a/sdnvpn/test/functest/run_sdnvpn_tests.py b/sdnvpn/test/functest/run_sdnvpn_tests.py index 52a6994..8e8cf81 100644 --- a/sdnvpn/test/functest/run_sdnvpn_tests.py +++ b/sdnvpn/test/functest/run_sdnvpn_tests.py @@ -85,7 +85,13 @@ class SdnvpnFunctest(testcase.TestCase): self.stop_time = time.time() try: - gather_logs('overall') + installer_type = str(os.environ['INSTALLER_TYPE'].lower()) + if installer_type in ["fuel", "apex"]: + gather_logs('overall') + else: + self.__logger.info("Skipping log gathering because installer" + "type %s is neither fuel nor apex" % + installer_type) except Exception as ex: self.__logger.error(('Something went wrong in the Log gathering.' 'Ex: %s, Trace: %s') -- cgit 1.2.3-korg