From 72bf97a47767ba726bbd990b871bfa860ed76bb2 Mon Sep 17 00:00:00 2001 From: Antoni Ivanov Date: Wed, 6 Oct 2021 13:32:39 +0300 Subject: [PATCH] vdk-control-cli: fix warning message not to crash (#353) Some warning messages were causing issues sinc we use VDKEXception to format the log message and the type is not string. Signed-off-by: Antoni Ivanov --- .../src/vdk/internal/control/command_groups/job/create.py | 2 +- .../vdk/internal/control/command_groups/job/deploy_cli_impl.py | 2 +- .../src/vdk/internal/control/command_groups/job/download_job.py | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/projects/vdk-control-cli/src/vdk/internal/control/command_groups/job/create.py b/projects/vdk-control-cli/src/vdk/internal/control/command_groups/job/create.py index 155ed240e7..afe8b56384 100644 --- a/projects/vdk-control-cli/src/vdk/internal/control/command_groups/job/create.py +++ b/projects/vdk-control-cli/src/vdk/internal/control/command_groups/job/create.py @@ -84,7 +84,7 @@ def __download_key(self, team, name, path): why=f"Error is: {e}", consequence="Local execution of the job might fail since the job may not have permissions to some resources", countermeasure="Try to manually download the data job keytab with vdk", - ) + ).message ) def validate_job_path(self, path: str, name: str) -> None: diff --git a/projects/vdk-control-cli/src/vdk/internal/control/command_groups/job/deploy_cli_impl.py b/projects/vdk-control-cli/src/vdk/internal/control/command_groups/job/deploy_cli_impl.py index bc72c72514..7d4750c198 100644 --- a/projects/vdk-control-cli/src/vdk/internal/control/command_groups/job/deploy_cli_impl.py +++ b/projects/vdk-control-cli/src/vdk/internal/control/command_groups/job/deploy_cli_impl.py @@ -116,7 +116,7 @@ def __cleanup_archive(archive_path: str) -> None: why=f"VDK CLI did not clean up after deploying: {e}", consequence="There is a leftover archive file next to the folder containing the data job", countermeasure="Clean up the archive file manually or leave it", - ) + ).message ) def __update_data_job_deploy_configuration( diff --git a/projects/vdk-control-cli/src/vdk/internal/control/command_groups/job/download_job.py b/projects/vdk-control-cli/src/vdk/internal/control/command_groups/job/download_job.py index 66e4b4f5b7..b17ecda170 100644 --- a/projects/vdk-control-cli/src/vdk/internal/control/command_groups/job/download_job.py +++ b/projects/vdk-control-cli/src/vdk/internal/control/command_groups/job/download_job.py @@ -70,7 +70,7 @@ def __cleanup_archive(archive_path: str): why=f"VDK CLI did not clean up after deploying: {e}", consequence="There is a leftover archive file next to the folder containing the data job", countermeasure="Clean up the archive file manually or leave it", - ) + ).message )