From b860265e26d5c38f701810db3be0d2a50f1bf557 Mon Sep 17 00:00:00 2001
From: Olivier Le Thanh Duong <olivier@lethanh.be>
Date: Mon, 13 Jan 2025 18:33:52 +0100
Subject: [PATCH] Enhance verbosity on failed downloads

---
 src/aleph/vm/storage.py | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/src/aleph/vm/storage.py b/src/aleph/vm/storage.py
index df0505ca..15d3b938 100644
--- a/src/aleph/vm/storage.py
+++ b/src/aleph/vm/storage.py
@@ -110,9 +110,12 @@ async def download_file(url: str, local_path: Path) -> None:
                 await asyncio.wait_for(file_downloaded_by_another_task(local_path), timeout=30)
             except TimeoutError as error:
                 if attempt < (download_attempts - 1):
-                    logger.warning(f"Download failed, retrying attempt {attempt + 1}/{download_attempts}...")
+                    logger.warning(
+                        f"Download failed (waiting for another taks), retrying attempt {attempt + 1}/{download_attempts}..."
+                    )
                     continue
                 else:
+                    logger.warning(f"Download of {url} failed  (waiting for another task), aborting...")
                     raise error from file_exists_error
         except (
             aiohttp.ClientConnectionError,
@@ -123,6 +126,7 @@ async def download_file(url: str, local_path: Path) -> None:
                 logger.warning(f"Download failed, retrying attempt {attempt + 1}/{download_attempts}...")
                 # continue  #  continue inside try/finally block is unimplemented in `mypyc`
             else:
+                logger.warning(f"Download of {url} failed  (aborting...")
                 raise error
         finally:
             # Ensure no partial file is left behind