Add timeout to github actions (#1716)

elronbandel · web-flow · commit 25cd4ea6bc75 · 2025-04-06T16:44:24.000+03:00
* Add timeout to github actions

Signed-off-by: elronbandel &lt;elronbandel@gmail.com&gt;

* Add UrllibHTTPError to retry exceptions in exponential backoff decorator

Signed-off-by: elronbandel &lt;elronbandel@gmail.com&gt;

---------

Signed-off-by: elronbandel &lt;elronbandel@gmail.com&gt;
diff --git a/.github/workflows/catalog_consistency.yml b/.github/workflows/catalog_consistency.yml
@@ -14,6 +14,7 @@ jobs:
    consistency:
 
      runs-on: ubuntu-latest
+     timeout-minutes: 30
      env:
        OS: ubuntu-latest
        UNITXT_DEFAULT_VERBOSITY: error
diff --git a/.github/workflows/catalog_preparation.yml b/.github/workflows/catalog_preparation.yml
@@ -6,10 +6,15 @@ on:
   pull_request:
     branches: [ main ]
 
+concurrency:
+    group: ${{ github.workflow }}-${{ github.event_name == 'pull_request' && github.event.pull_request.number || github.ref_name }}
+    cancel-in-progress: true
+
 jobs:
   preparation:
 
     runs-on: ubuntu-latest
+    timeout-minutes: 30
     env:
       OS: ubuntu-latest
       UNITXT_DEFAULT_VERBOSITY: error
diff --git a/.github/workflows/docs.yml b/.github/workflows/docs.yml
@@ -14,6 +14,7 @@ jobs:
     docs:
 
       runs-on: ubuntu-latest
+      timeout-minutes: 30
       env:
         OS: ubuntu-latest
         PYTHONPATH: ./docs
diff --git a/.github/workflows/examples_tests.yml b/.github/workflows/examples_tests.yml
@@ -17,6 +17,7 @@ jobs:
    examples:
 
      runs-on: ubuntu-latest
+     timeout-minutes: 30
      env:
        OS: ubuntu-latest
        UNITXT_DEFAULT_VERBOSITY: error
diff --git a/.github/workflows/inference_tests.yml b/.github/workflows/inference_tests.yml
@@ -17,6 +17,7 @@ concurrency:
 jobs:
   inference:
     runs-on: ubuntu-latest
+    timeout-minutes: 30
     env:
       OS: ubuntu-latest
       UNITXT_DEFAULT_VERBOSITY: error
diff --git a/.github/workflows/library_eager_execution_tests.yml b/.github/workflows/library_eager_execution_tests.yml
@@ -14,6 +14,7 @@ jobs:
   eager:
 
     runs-on: ubuntu-latest
+    timeout-minutes: 30
     env:
       OS: ubuntu-latest
       UNITXT_USE_EAGER_EXECUTION: True
diff --git a/.github/workflows/library_tests.yml b/.github/workflows/library_tests.yml
@@ -14,6 +14,7 @@ jobs:
   unittests:
 
     runs-on: ubuntu-latest
+    timeout-minutes: 30
     env:
       OS: ubuntu-latest
       UNITXT_DEFAULT_VERBOSITY: error
diff --git a/.github/workflows/performance.yml b/.github/workflows/performance.yml
@@ -12,6 +12,7 @@ jobs:
   performance:
 
     runs-on: ubuntu-latest
+    timeout-minutes: 30
     env:
       OS: ubuntu-latest
       UNITXT_DEFAULT_VERBOSITY: error
diff --git a/.github/workflows/quality.yml b/.github/workflows/quality.yml
@@ -13,6 +13,7 @@ concurrency:
 jobs:
   quality:
     runs-on: ubuntu-latest
+    timeout-minutes: 30
     steps:
       - uses: actions/checkout@v4
       - uses: actions/setup-python@v5
diff --git a/.github/workflows/test_helm.yml b/.github/workflows/test_helm.yml
@@ -14,6 +14,7 @@ jobs:
     helm:
 
       runs-on: ubuntu-latest
+      timeout-minutes: 30
       env:
         OS: ubuntu-latest
 
diff --git a/src/unitxt/utils.py b/src/unitxt/utils.py
@@ -11,6 +11,7 @@
 from collections import OrderedDict
 from functools import lru_cache
 from typing import Any, Dict
+from urllib.error import HTTPError as UrllibHTTPError
 
 from requests.exceptions import ConnectionError, HTTPError
 from requests.exceptions import Timeout as TimeoutError
@@ -21,7 +22,7 @@
 settings = get_settings()
 
 def retry_connection_with_exponential_backoff(max_retries=None,
-                                  retry_exceptions=(ConnectionError, TimeoutError, HTTPError, FileNotFoundError),
+                                  retry_exceptions=(ConnectionError, TimeoutError, HTTPError, FileNotFoundError, UrllibHTTPError),
                                   backoff_factor=1):
     """Decorator that implements retry with exponential backoff for network operations.
 
