From 461fbd9df0d0165c349150dbe3fd05a3aa76e7a4 Mon Sep 17 00:00:00 2001 From: allegroai <> Date: Thu, 12 Mar 2020 17:13:36 +0200 Subject: [PATCH] Better warning messages for storage errors --- trains/storage/helper.py | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/trains/storage/helper.py b/trains/storage/helper.py index cf0f76c1..9cc9d965 100644 --- a/trains/storage/helper.py +++ b/trains/storage/helper.py @@ -260,8 +260,11 @@ class StorageHelper(object): # Don't canonize URL since we already did it try: instance = cls(base_url=base_url, url=url, logger=logger, canonize_url=False, **kwargs) - except Exception: - log.error("Failed credentials for {}".format(base_url or url)) + except (StorageError, UsageError) as ex: + log.error(str(ex)) + return None + except Exception as ex: + log.error("Failed credentials for {}. Reason: {}".format(base_url or url, ex)) return None cls._helpers[instance_key] = instance