diff options
author | Cédric Ollivier <cedric.ollivier@orange.com> | 2018-02-13 11:50:34 +0100 |
---|---|---|
committer | Cédric Ollivier <cedric.ollivier@orange.com> | 2018-02-13 14:58:48 +0100 |
commit | 0424a6d98739ac7499c28799570ea8a625c0779c (patch) | |
tree | c5eed980bf32ab82f77da94f26e4fe6ae6cbde44 | |
parent | 2fbff506d2bece39db007af98c7c5292329b7f31 (diff) |
Remove INSTALLER_IP from Functest
It's mandatory for testcases which are out of Functest.
Change-Id: Ia5f5134334ad9f69d44409d817616fa9033fae95
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
-rw-r--r-- | docs/testing/user/configguide/configguide.rst | 5 | ||||
-rw-r--r-- | functest/cli/commands/cli_env.py | 4 | ||||
-rw-r--r-- | functest/opnfv_tests/openstack/tempest/conf_utils.py | 1 | ||||
-rw-r--r-- | functest/tests/unit/cli/commands/test_cli_env.py | 6 | ||||
-rw-r--r-- | functest/tests/unit/cli/commands/test_cli_os.py | 1 | ||||
-rw-r--r-- | functest/utils/env.py | 1 |
6 files changed, 1 insertions, 17 deletions
diff --git a/docs/testing/user/configguide/configguide.rst b/docs/testing/user/configguide/configguide.rst index 9adf79e1..c4d9a6d7 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=<Specific IP Address> * DEPLOY_SCENARIO=<vim>-<controller>-<nfv_feature>-<ha_mode> -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 a5c0e399..f93ef612 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 efcbe7fd..f128784c 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 d17da166..7d6f7787 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 02392d2d..a5e562a2 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 110164be..b626473a 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, |