From 701c146bd748bcf9c0228cea593539a3521661ca Mon Sep 17 00:00:00 2001 From: Rabi Mishra Date: Wed, 13 May 2020 13:13:04 +0530 Subject: [PATCH] Remove unused write_overcloudrc() function Change-Id: Ifb3df132802dc1380e9949528416ecfed882cd06 --- .../tests/fixture_data/deployment.py | 3 --- tripleoclient/tests/test_utils.py | 25 ------------------- tripleoclient/utils.py | 12 --------- 3 files changed, 40 deletions(-) diff --git a/tripleoclient/tests/fixture_data/deployment.py b/tripleoclient/tests/fixture_data/deployment.py index 6b1bd3cba..e80ccf775 100644 --- a/tripleoclient/tests/fixture_data/deployment.py +++ b/tripleoclient/tests/fixture_data/deployment.py @@ -68,9 +68,6 @@ class UtilsFixture(fixtures.Fixture): self.mock_remove_known_hosts = self.useFixture(fixtures.MockPatch( 'tripleoclient.utils.remove_known_hosts') ).mock - self.mock_write_overcloudrc = self.useFixture(fixtures.MockPatch( - 'tripleoclient.utils.write_overcloudrc') - ).mock self.mock_run_ansible_playbook = self.useFixture(fixtures.MockPatch( 'tripleoclient.utils.run_ansible_playbook') ).mock diff --git a/tripleoclient/tests/test_utils.py b/tripleoclient/tests/test_utils.py index e4968e855..3dade260f 100644 --- a/tripleoclient/tests/test_utils.py +++ b/tripleoclient/tests/test_utils.py @@ -561,31 +561,6 @@ class TestEnsureRunAsNormalUser(TestCase): self.assertEqual('stack', u) -class TestCreateOvercloudRC(TestCase): - - def test_write_overcloudrc(self): - stack_name = 'teststack' - - tempdir = tempfile.mkdtemp() - rcfile = os.path.join(tempdir, 'teststackrc') - - overcloudrcs = { - "overcloudrc": "overcloudrc", - } - - try: - utils.write_overcloudrc(stack_name, overcloudrcs, - config_directory=tempdir) - with open(rcfile, 'rt') as f: - rc = f.read() - self.assertEqual('overcloudrc', rc) - finally: - if os.path.exists(rcfile): - os.unlink(rcfile) - - os.rmdir(tempdir) - - class TestCreateTempestDeployerInput(TestCase): def test_create_tempest_deployer_input(self): diff --git a/tripleoclient/utils.py b/tripleoclient/utils.py index f29df3d89..79ceafa9a 100644 --- a/tripleoclient/utils.py +++ b/tripleoclient/utils.py @@ -813,18 +813,6 @@ def write_env_file(env_data, env_file, registry_overwrites): yaml.dump(data, f, default_flow_style=False, Dumper=dumper) -def write_overcloudrc(stack_name, overcloudrcs, config_directory='.'): - """Write the overcloudrc files""" - - rcpath = os.path.join(config_directory, '%src' % stack_name) - - with open(rcpath, 'w') as rcfile: - rcfile.write(overcloudrcs['overcloudrc']) - os.chmod(rcpath, 0o600) - - return os.path.abspath(rcpath) - - def store_cli_param(command_name, parsed_args): """write the cli parameters into an history file"""