Add logging and proper raise on too many tries

bopen · May 29, 2024 · 2942527 · 2942527
1 parent 2da64f2
commit 2942527
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/xarray_ecmwf/engine_ecmwf.py b/xarray_ecmwf/engine_ecmwf.py
@@ -87,8 +87,10 @@ def retrieve(
                 with self.retrieve_once(request, override_cache_file) as ds:
                     yield ds
                 break
-            except RuntimeError:
-                LOGGER.exception("Failed retrieve")
+            except RuntimeError as ex:
+                LOGGER.exception(f"Failed retrieve: {try_} / {tries}")
+        else:
+            raise RuntimeError(f"too many retries {tries}")
 
     @contextlib.contextmanager
     def retrieve_once(