openstack
/
python-tripleoclient
Code Issues Proposed changes

Default to zero verbosity when using Ansible. 

The OpenStack CLI defaults to 1 level of verbosity
which was being directly passed to Ansible methods.
When the CLI is given "-v", the verbosity level is set to 2.
This patch ensures that Ansible will not be verbose by default.
This lowers the amount of RAM a default
deployment/update/upgrade will use along with lowering
log file sizes.

Conflicts:
	tripleoclient/tests/v1/overcloud_external_update/test_overcloud_external_update.py
	tripleoclient/tests/v1/overcloud_external_upgrade/test_overcloud_external_upgrade.py
	tripleoclient/tests/v1/overcloud_ffwd_upgrade/test_overcloud_ffwd_upgrade.py
	tripleoclient/tests/v1/overcloud_update/test_overcloud_update.py
	tripleoclient/tests/v1/overcloud_upgrade/test_overcloud_upgrade.py
	tripleoclient/utils.py
	tripleoclient/v1/overcloud_deploy.py
	tripleoclient/v1/overcloud_external_update.py
	tripleoclient/v1/overcloud_external_upgrade.py
	tripleoclient/v1/overcloud_update.py
	tripleoclient/v1/overcloud_upgrade.py
	tripleoclient/workflows/deployment.py

Change-Id: Ic59cc6b2b9b979255c10d71e437afa733517e3e1
Signed-off-by: Luke Short <ekultails@gmail.com>
(cherry picked from commit 90eee3abc1)
(cherry picked from commit aef369e7b4)
This commit is contained in:
Luke Short 2020-01-20 16:16:11 -05:00
parent bcb511376a
commit 36cda0a2b5
13 changed files with 22 additions and 22 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
tripleoclient/tests/v1/overcloud_external_update/test_overcloud_external_update.py
View File

@ -61,5 +61,5 @@ class TestOvercloudExternalUpdateRun(fakes.TestOvercloudExternalUpdateRun):
node_user='tripleo-admin',
tags='ceph',
skip_tags='',
verbosity=1
verbosity=0
)

2
tripleoclient/tests/v1/overcloud_external_upgrade/test_overcloud_external_upgrade.py
View File

@ -61,5 +61,5 @@ class TestOvercloudExternalUpgradeRun(fakes.TestOvercloudExternalUpgradeRun):
node_user='tripleo-admin',
tags='ceph',
skip_tags='',
verbosity=1
verbosity=0
)

4
tripleoclient/tests/v1/overcloud_ffwd_upgrade/test_overcloud_ffwd_upgrade.py
View File

@ -169,7 +169,7 @@ class TestFFWDUpgradeRun(fakes.TestFFWDUpgradeRun):
node_user='heat-admin',
tags='',
skip_tags='',
verbosity=1
verbosity=0
)
@mock.patch('tripleoclient.workflows.package_update.update_ansible',
@ -195,7 +195,7 @@ class TestFFWDUpgradeRun(fakes.TestFFWDUpgradeRun):
node_user='my-user',
tags='',
skip_tags='',
verbosity=1
verbosity=0
)
@mock.patch('tripleoclient.workflows.package_update.update_ansible',

8
tripleoclient/tests/v1/overcloud_update/test_overcloud_update.py
View File

@ -151,7 +151,7 @@ class TestOvercloudUpdateRun(fakes.TestOvercloudUpdateRun):
node_user='tripleo-admin',
tags='',
skip_tags='',
verbosity=1
verbosity=0
)
@mock.patch('tripleoclient.workflows.package_update.update_ansible',
@ -182,7 +182,7 @@ class TestOvercloudUpdateRun(fakes.TestOvercloudUpdateRun):
node_user='tripleo-admin',
tags='',
skip_tags='',
verbosity=1
verbosity=0
)
@mock.patch('tripleoclient.workflows.package_update.update_ansible',
@ -212,7 +212,7 @@ class TestOvercloudUpdateRun(fakes.TestOvercloudUpdateRun):
node_user='tripleo-admin',
tags='',
skip_tags='',
verbosity=1
verbosity=0
)
@mock.patch('tripleoclient.workflows.package_update.update_ansible',
@ -258,7 +258,7 @@ class TestOvercloudUpdateConverge(fakes.TestOvercloudUpdateConverge):
# Get the command object to test
app_args = mock.Mock()
app_args.verbose_level = 1
app_args.verbose_level = 0
self.cmd = overcloud_update.UpdateConverge(self.app, app_args)
@mock.patch(

10
tripleoclient/tests/v1/overcloud_upgrade/test_overcloud_upgrade.py
View File

@ -171,7 +171,7 @@ class TestOvercloudUpgradeRun(fakes.TestOvercloudUpgradeRun):
node_user='tripleo-admin',
tags='',
skip_tags='',
verbosity=1
verbosity=0
)
@mock.patch('tripleoclient.workflows.package_update.update_ansible',
@ -204,7 +204,7 @@ class TestOvercloudUpgradeRun(fakes.TestOvercloudUpgradeRun):
node_user='tripleo-admin',
tags='',
skip_tags='validation',
verbosity=1
verbosity=0
)
@mock.patch('tripleoclient.workflows.package_update.update_ansible',
@ -235,7 +235,7 @@ class TestOvercloudUpgradeRun(fakes.TestOvercloudUpgradeRun):
node_user='tripleo-admin',
tags='',
skip_tags='',
verbosity=1
verbosity=0
)
@mock.patch('tripleoclient.workflows.package_update.update_ansible',
@ -266,7 +266,7 @@ class TestOvercloudUpgradeRun(fakes.TestOvercloudUpgradeRun):
node_user='tripleo-admin',
tags='',
skip_tags='',
verbosity=1
verbosity=0
)
@mock.patch('tripleoclient.workflows.package_update.update_ansible',
@ -299,7 +299,7 @@ class TestOvercloudUpgradeRun(fakes.TestOvercloudUpgradeRun):
node_user='tripleo-admin',
tags='',
skip_tags='pre-upgrade,validation',
verbosity=1
verbosity=0
)
@mock.patch('tripleoclient.workflows.package_update.update_ansible',

2
tripleoclient/utils.py
View File

@ -1174,7 +1174,7 @@ def process_multiple_environments(created_env_files, tht_root,
def run_update_ansible_action(log, clients, nodes, inventory, playbook,
all_playbooks, action, ssh_user, tags='',
skip_tags='', verbosity=1):
skip_tags='', verbosity=0):
playbooks = [playbook]
if playbook == "all":
playbooks = all_playbooks

4
tripleoclient/v1/overcloud_deploy.py
View File

@ -233,7 +233,7 @@ class DeployOvercloud(command.Command):
os.path.abspath(tht_root)))
deployment.deploy_and_wait(
self.log, self.clients, stack,
stack_name, self.app_args.verbose_level,
stack_name, self.app_args.verbose_level - 1,
timeout=timeout,
run_validations=run_validations,
skip_deploy_identifier=skip_deploy_identifier)
@ -954,7 +954,7 @@ class DeployOvercloud(command.Command):
parsed_args.output_dir,
parsed_args.override_ansible_cfg,
timeout,
verbosity=self.app_args.verbose_level)
verbosity=(self.app_args.verbose_level - 1))
except Exception:
deployment.set_deployment_status(
self.clients, 'failed',

2
tripleoclient/v1/overcloud_external_update.py
View File

@ -85,7 +85,7 @@ class ExternalUpdateRun(command.Command):
def take_action(self, parsed_args):
self.log.debug("take_action(%s)" % parsed_args)
clients = self.app.client_manager
verbosity = self.app_args.verbose_level
verbosity = self.app_args.verbose_level - 1
stack = parsed_args.stack
# Run ansible:

2
tripleoclient/v1/overcloud_external_upgrade.py
View File

@ -85,7 +85,7 @@ class ExternalUpgradeRun(command.Command):
def take_action(self, parsed_args):
self.log.debug("take_action(%s)" % parsed_args)
clients = self.app.client_manager
verbosity = self.app_args.verbose_level
verbosity = self.app_args.verbose_level - 1
stack = parsed_args.stack
# Run ansible:

2
tripleoclient/v1/overcloud_ffwd_upgrade.py
View File

@ -144,7 +144,7 @@ class FFWDUpgradeRun(command.Command):
def take_action(self, parsed_args):
self.log.debug("take_action(%s)" % parsed_args)
oooutils.ffwd_upgrade_operator_confirm(parsed_args.yes, self.log)
verbosity = self.app_args.verbose_level
verbosity = self.app_args.verbose_level - 1
clients = self.app.client_manager
# Run ansible:

2
tripleoclient/v1/overcloud_update.py
View File

@ -159,7 +159,7 @@ class UpdateRun(command.Command):
def take_action(self, parsed_args):
self.log.debug("take_action(%s)" % parsed_args)
clients = self.app.client_manager
verbosity = self.app_args.verbose_level
verbosity = self.app_args.verbose_level - 1
stack = parsed_args.stack
# Run ansible:

2
tripleoclient/v1/overcloud_upgrade.py
View File

@ -197,7 +197,7 @@ class UpgradeRun(command.Command):
def take_action(self, parsed_args):
self.log.debug("take_action(%s)" % parsed_args)
clients = self.app.client_manager
verbosity = self.app_args.verbose_level
verbosity = self.app_args.verbose_level - 1
stack = parsed_args.stack
# Run ansible:

2
tripleoclient/workflows/deployment.py
View File

@ -284,7 +284,7 @@ def enable_ssh_admin(log, clients, plan_name, hosts, ssh_user, ssh_key):
def config_download(log, clients, stack, templates,
ssh_user, ssh_key, ssh_network,
output_dir, override_ansible_cfg, timeout, verbosity=1):
output_dir, override_ansible_cfg, timeout, verbosity=0):
workflow_client = clients.workflow_engine
tripleoclients = clients.tripleoclient