From af453f727341863d2f21b9d7e50a3b876a598136 Mon Sep 17 00:00:00 2001 From: Neil Massey Date: Tue, 4 Feb 2025 13:48:15 +0000 Subject: [PATCH] removed commas from error messages as it was creating lists --- nlds_processors/archive/s3_to_tarfile_tape.py | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/nlds_processors/archive/s3_to_tarfile_tape.py b/nlds_processors/archive/s3_to_tarfile_tape.py index 4328e1e..ecaf0e1 100644 --- a/nlds_processors/archive/s3_to_tarfile_tape.py +++ b/nlds_processors/archive/s3_to_tarfile_tape.py @@ -377,8 +377,8 @@ def _verify_tape_server(self): status, _ = self.tape_client.ping() if status.status != 0: msg = ( - f"Failed status message: {status.message}. ", - f"Could not ping cta server at {self.tape_server_url}.", + f"Failed status message: {status.message}. " + f"Could not ping cta server at {self.tape_server_url}." ) raise S3StreamError(msg) @@ -386,17 +386,17 @@ def _verify_tape_server(self): status, resp = self.tape_client.stat(self.tape_base_dir) if status.status != 0: msg = ( - f"Failed status message: {status.message}. ", - f"Base dir {self.tape_base_dir} could not be statted", + f"Failed status message: {status.message}. " + f"Base dir {self.tape_base_dir} could not be statted" ) raise S3StreamError(msg) # Check whether the flag indicates it's a directory elif not bool(resp.flags & StatInfoFlags.IS_DIR): msg = ( - f"Failed status message: {status.message}. ", - f"Full status object: {status}. ", - f"Stat result for base dir {self.tape_base_dir} ", - f"indicates it is not a directory.", + f"Failed status message: {status.message}. " + f"Full status object: {status}. " + f"Stat result for base dir {self.tape_base_dir} " + f"indicates it is not a directory." ) raise S3StreamError(msg)