From 0424a6d98739ac7499c28799570ea8a625c0779c Mon Sep 17 00:00:00 2001 From: Cédric Ollivier Date: Tue, 13 Feb 2018 11:50:34 +0100 Subject: Remove INSTALLER_IP from Functest MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit It's mandatory for testcases which are out of Functest. Change-Id: Ia5f5134334ad9f69d44409d817616fa9033fae95 Signed-off-by: Cédric Ollivier --- docs/testing/user/configguide/configguide.rst | 5 ----- functest/cli/commands/cli_env.py | 4 +--- functest/opnfv_tests/openstack/tempest/conf_utils.py | 1 - functest/tests/unit/cli/commands/test_cli_env.py | 6 ------ functest/tests/unit/cli/commands/test_cli_os.py | 1 - functest/utils/env.py | 1 - 6 files changed, 1 insertion(+), 17 deletions(-) diff --git a/docs/testing/user/configguide/configguide.rst b/docs/testing/user/configguide/configguide.rst index 9adf79e14..c4d9a6d77 100644 --- a/docs/testing/user/configguide/configguide.rst +++ b/docs/testing/user/configguide/configguide.rst @@ -38,7 +38,6 @@ Preparing your environment cat env:: INSTALLER_TYPE=XXX - INSTALLER_IP=XXX EXTERNAL_NETWORK=XXX DEPLOY_SCENARIO=XXX @@ -198,13 +197,9 @@ Environment variables Several environement variables may be specified: * INSTALLER_TYPE=(apex|compass|daisy|fuel|joid) - * INSTALLER_IP= * DEPLOY_SCENARIO=--- -INSTALLER IP may be required by some test cases like SFC or Barometer in order -to access the installer node and the deployment. - The format for the DEPLOY_SCENARIO env variable can be described as follows: * vim: (os|k8s) = OpenStack or Kubernetes * controller is one of ( nosdn | odl ) diff --git a/functest/cli/commands/cli_env.py b/functest/cli/commands/cli_env.py index a5c0e3996..f93ef612c 100644 --- a/functest/cli/commands/cli_env.py +++ b/functest/cli/commands/cli_env.py @@ -20,8 +20,6 @@ class Env(object): # pylint: disable=too-few-public-methods @staticmethod def show(): install_type = os.environ.get('INSTALLER_TYPE', 'Unknown') - installer_ip = os.environ.get('INSTALLER_IP', 'Unknown') - installer_info = ("%s, %s" % (install_type, installer_ip)) scenario = os.environ.get('DEPLOY_SCENARIO', 'Unknown') node = os.environ.get('NODE_NAME', 'Unknown') build_tag = os.environ.get('BUILD_TAG', None) @@ -29,7 +27,7 @@ class Env(object): # pylint: disable=too-few-public-methods build_tag = build_tag.lstrip( "jenkins-").lstrip("functest").lstrip("-") - env_info = {'INSTALLER': installer_info, + env_info = {'INSTALLER': install_type, 'SCENARIO': scenario, 'POD': node, 'BUILD_TAG': build_tag} diff --git a/functest/opnfv_tests/openstack/tempest/conf_utils.py b/functest/opnfv_tests/openstack/tempest/conf_utils.py index efcbe7fdc..f128784c2 100644 --- a/functest/opnfv_tests/openstack/tempest/conf_utils.py +++ b/functest/opnfv_tests/openstack/tempest/conf_utils.py @@ -52,7 +52,6 @@ TEST_ACCOUNTS_FILE = pkg_resources.resource_filename( 'opnfv_tests/openstack/tempest/custom_tests/test_accounts.yaml') CI_INSTALLER_TYPE = CONST.__getattribute__('INSTALLER_TYPE') -CI_INSTALLER_IP = CONST.__getattribute__('INSTALLER_IP') """ logging configuration """ LOGGER = logging.getLogger(__name__) diff --git a/functest/tests/unit/cli/commands/test_cli_env.py b/functest/tests/unit/cli/commands/test_cli_env.py index d17da166e..7d6f77870 100644 --- a/functest/tests/unit/cli/commands/test_cli_env.py +++ b/functest/tests/unit/cli/commands/test_cli_env.py @@ -36,9 +36,6 @@ class CliEnvTesting(unittest.TestCase): if var == 'INSTALLER_TYPE': os.environ['INSTALLER_TYPE'] = '' reg_string = r"| INSTALLER: Unknown, \S+\s*|" - elif var == 'INSTALLER_IP': - os.environ['INSTALLER_IP'] = '' - reg_string = r"| INSTALLER: \S+, Unknown\s*|" elif var == 'SCENARIO': os.environ['DEPLOY_SCENARIO'] = '' reg_string = r"| SCENARIO: Unknown\s*|" @@ -57,9 +54,6 @@ class CliEnvTesting(unittest.TestCase): def test_show_ci_installer_type_ko(self, *args): self._test_show_missing_env_var('INSTALLER_TYPE', *args) - def test_show_ci_installer_ip_ko(self, *args): - self._test_show_missing_env_var('INSTALLER_IP', *args) - def test_show_missing_ci_scenario(self, *args): self._test_show_missing_env_var('SCENARIO', *args) diff --git a/functest/tests/unit/cli/commands/test_cli_os.py b/functest/tests/unit/cli/commands/test_cli_os.py index 02392d2df..a5e562a27 100644 --- a/functest/tests/unit/cli/commands/test_cli_os.py +++ b/functest/tests/unit/cli/commands/test_cli_os.py @@ -23,7 +23,6 @@ class CliOpenStackTesting(unittest.TestCase): self.endpoint_ip = 'test_ip' self.os_auth_url = 'http://test_ip:test_port/v2.0' self.installer_type = 'test_installer_type' - self.installer_ip = 'test_installer_ip' self.openstack_creds = 'test_env_file' self.snapshot_file = 'test_snapshot_file' os.environ["OS_AUTH_URL"] = '' diff --git a/functest/utils/env.py b/functest/utils/env.py index 110164bee..b626473a9 100644 --- a/functest/utils/env.py +++ b/functest/utils/env.py @@ -16,7 +16,6 @@ class Environment(object): # pylint: disable=too-few-public-methods 'DEPLOY_SCENARIO': 'os-nosdn-nofeature-noha', 'DEPLOY_TYPE': 'virt', 'INSTALLER_TYPE': None, - 'INSTALLER_IP': None, 'BUILD_TAG': None, 'OS_ENDPOINT_TYPE': None, 'OS_AUTH_URL': None, -- cgit 1.2.3-korg