Merge pull request #762 from python-discord/raise-static-builds

Static Builds Small Improvements

Author: HassanAbouelela

pydis_site/apps/api/github_utils.py

@@ -9,7 +9,7 @@
 
 from pydis_site import settings
 
-MAX_RUN_TIME = datetime.timedelta(minutes=3)
+MAX_RUN_TIME = datetime.timedelta(minutes=10)
 """The maximum time allowed before an action is declared timed out."""
 ISO_FORMAT_STRING = "%Y-%m-%dT%H:%M:%SZ"
 """The datetime string format GitHub uses."""

static-builds/netlify_build.py

@@ -15,6 +15,18 @@
 
 import httpx
 
+
+def raise_response(response: httpx.Response) -> None:
+    """Raise an exception from a response if necessary."""
+    if response.status_code // 100 != 2:
+        try:
+            print(response.json())
+        except json.JSONDecodeError:
+            pass
+
+    response.raise_for_status()
+
+
 if __name__ == "__main__":
     owner, repo = parse.urlparse(os.getenv("REPOSITORY_URL")).path.lstrip("/").split("/")[0:2]
 
@@ -29,24 +41,18 @@
     ])
     print(f"Fetching download URL from {download_url}")
     response = httpx.get(download_url, follow_redirects=True)
-
-    if response.status_code // 100 != 2:
-        try:
-            print(response.json())
-        except json.JSONDecodeError:
-            pass
-
-        response.raise_for_status()
+    raise_response(response)
 
     # The workflow is still pending, retry in a bit
     while response.status_code == 202:
         print(f"{response.json()['error']}. Retrying in 10 seconds.")
         time.sleep(10)
         response = httpx.get(download_url, follow_redirects=True)
 
+    raise_response(response)
     url = response.json()["url"]
     print(f"Downloading build from {url}")
-    zipped_content = httpx.get(url, follow_redirects=True)
+    zipped_content = httpx.get(url, follow_redirects=True, timeout=3 * 60)
     zipped_content.raise_for_status()
 
     zip_file = Path("temp.zip")