Make warnings more verbose

Author: shin-

depends/docker-registry-core/docker_registry/core/lru.py

@@ -67,8 +67,8 @@ def wrapper(*args):
         key = cache_key(key)
         try:
             redis_conn.set(key, content)
-        except redis.exceptions.ConnectionError:
-            logging.warning("LRU: Redis connection error")
+        except redis.exceptions.ConnectionError as e:
+            logging.warning("LRU: Redis connection error: {0}".format(e))
 
         return f(*args)
     if redis_conn is None:
@@ -83,8 +83,8 @@ def wrapper(*args):
         key = cache_key(key)
         try:
             content = redis_conn.get(key)
-        except redis.exceptions.ConnectionError:
-            logging.warning("LRU: Redis connection error")
+        except redis.exceptions.ConnectionError as e:
+            logging.warning("LRU: Redis connection error: {0}".format(e))
             content = None
 
         if content is not None:
@@ -94,8 +94,8 @@ def wrapper(*args):
         if content is not None:
             try:
                 redis_conn.set(key, content)
-            except redis.exceptions.ConnectionError:
-                logging.warning("LRU: Redis connection error")
+            except redis.exceptions.ConnectionError as e:
+                logging.warning("LRU: Redis connection error: {0}".format(e))
         return content
     if redis_conn is None:
         return f
@@ -109,8 +109,8 @@ def wrapper(*args):
         key = cache_key(key)
         try:
             redis_conn.delete(key)
-        except redis.exceptions.ConnectionError:
-            logging.warning("LRU: Redis connection error")
+        except redis.exceptions.ConnectionError as e:
+            logging.warning("LRU: Redis connection error: {0}".format(e))
         return f(*args)
     if redis_conn is None:
         return f

docker_registry/lib/layers.py

@@ -39,8 +39,8 @@ def enqueue_diff(image_id):
     try:
         if cache.redis_conn:
             diff_queue.push(image_id)
-    except cache.redis.exceptions.ConnectionError:
-        logger.warning('Diff queue: Redis connection error')
+    except redis.exceptions.ConnectionError as e:
+        logger.warning("Diff queue: Redis connection error: {0}".format(e))
 
 
 def generate_ancestry(image_id, parent_id=None):

docker_registry/lib/mirroring.py

@@ -109,9 +109,11 @@ def wrapper(namespace, repository, *args, **kwargs):
             data = cache.redis_conn.get('{0}:{1}'.format(
                 cache.cache_prefix, tag_path
             ))
-        except cache.redis.exceptions.ConnectionError:
+        except cache.redis.exceptions.ConnectionError as e:
             data = None
-            logger.warning('Tags cache: Redis connection error')
+            logger.warning("Diff queue: Redis connection error: {0}".format(
+                e
+            ))
 
         if data is not None:
             return toolkit.response(data=data, raw=True)
@@ -126,8 +128,10 @@ def wrapper(namespace, repository, *args, **kwargs):
             cache.redis_conn.setex('{0}:{1}'.format(
                 cache.cache_prefix, tag_path
             ), tags_cache_ttl, data)
-        except cache.redis.exceptions.ConnectionError:
-            logger.warning('Tags cache: Redis connection error')
+        except redis.exceptions.ConnectionError as e:
+            logger.warning("Diff queue: Redis connection error: {0}".format(
+                e
+            ))
 
         return toolkit.response(data=data, headers=headers,
                                 raw=True)