Skip to content

Commit

Permalink
vdk-control-cli: fix warning message not to crash (#353)
Browse files Browse the repository at this point in the history
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 <[email protected]>
  • Loading branch information
antoniivanov authored Oct 6, 2021
1 parent c0f773c commit 72bf97a
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -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:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -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(
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -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
)


Expand Down

0 comments on commit 72bf97a

Please sign in to comment.