decorator: retry on 500

Signed-off-by: Isabella do Amaral <[email protected]>

oras/decorator.py

@@ -37,7 +37,17 @@ def inner(*args, **kwargs):
             attempt = 0
             while attempt < attempts:
                 try:
-                    return func(*args, **kwargs)
+                    res = func(*args, **kwargs)
+                    if res.status_code == 500:
+                        try:
+                            msg = res.json()
+                            for error in msg.get("errors", []):
+                                if isinstance(error, dict) and "message" in error:
+                                    logger.error(error["message"])
+                        except Exception:
+                            pass
+                        raise ValueError(f"Issue with {res.request.url}: {res.reason}")
+                    return res
                 except oras.auth.AuthenticationException as e:
                     raise e
                 except Exception as e:

oras/provider.py

@@ -840,9 +840,6 @@ def push(
         response = self.upload_manifest(
             manifest, container
         )  # make the returned response from this method, the one pertaining to the uploaded Manifest
-        if response.status_code == 500:
-            # retry once
-            response = self.upload_manifest(manifest, container)
         self._check_200_response(response)
         print(f"Successfully pushed {container}")
         return response