openstack
/
tripleo-validations
Code Issues Proposed changes

Fix the get_deployment_status call 

The get_deployment_status function waits for a keyworded,
variable-length argument dictionary. This patch fixes that issue by
adding the right keyword for 'plan'.

This patch also adds a space in the output string in order to be
well-formatted.

Signed-off-by: Gael Chamoulaud (Strider) <gchamoul@redhat.com>
Change-Id: I4baff7b179c483726208b237df939087c595c165
This commit is contained in:
Gael Chamoulaud (Strider) 2021-01-25 18:17:37 +01:00
parent 949cc1c18e
commit d46d61fecb
No known key found for this signature in database
GPG Key ID: 4119D0305C651D66
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
tripleo_validations/tripleo_validator.py
View File

@ -348,11 +348,11 @@ class TripleOValidatorRun(command.Command):
"inventory or use authentication."))
else:
if plan:
status = deployment.get_deployment_status(clients, plan)
status = deployment.get_deployment_status(clients, plan=plan)
if not status or status in NO_VALIDATION_STATE:
raise exceptions.CommandError(
_("The plan and the stack '{}' doesn't exist OR are "
"in 'failed' or 'deploying' state."
"in 'failed' or 'deploying' state. "
"Please use a valid plan".format(plan)))
else:
msg = "Running Validations without Overcloud settings."