Addressing CR comment

openlawlibrary · May 11, 2020 · 22469ca · 22469ca
1 parent cb8179c
commit 22469ca
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/taf/git.py b/taf/git.py
@@ -63,7 +63,7 @@ def __init__(
         logging.INFO: taf_logger.info,
         logging.WARNING: taf_logger.warning,
         logging.ERROR: taf_logger.error,
-        logging.CRITICAL: taf_logger.critical
+        logging.CRITICAL: taf_logger.critical,
     }
 
     log_template = "{}{}"

diff --git a/taf/repositoriesdb.py b/taf/repositoriesdb.py
@@ -98,11 +98,11 @@ def load_repositories(
             repositories = _get_json_file(auth_repo, repositories_path, commit)
             targets = _get_json_file(auth_repo, targets_path, commit)
         except InvalidOrMissingMetadataError as e:
-            log_func = taf_logger.warning
             if "targets/repositories.json not available at revision" in str(e):
-                log_func = taf_logger.debug
-            log_func("Skipping commit {} due to error: {}", commit, str(e))
-            continue
+                taf_logger.debug("Skipping commit {} due to: {}", commit, str(e))
+                continue
+            else:
+                raise
 
         # target repositories are defined in both mirrors.json and targets.json
         repositories = repositories["repositories"]