Merge pull request #747 from MikroElektronika/update/removed-github-api-retry-approach

Updated scripts to exclude api fetching of all releases

Author: StrahinjaJacimovic

scripts/compare_assets.py

@@ -13,60 +13,6 @@ def get_headers(api, token):
             'Accept': 'application/octet-stream'
         }
 
-def fetch_specified_release_version(repo, token, tag):
-    api_headers = get_headers(True, token)
-    url = f'https://api.github.com/repos/{repo}/releases'
-
-    # First: 5 fast attempts (5s timeout)
-    for attempt in range(1, 6):
-        try:
-            print(f'GitHub API attempt {attempt}/5 (timeout=10s)')
-            response = requests.get(url, headers=api_headers, timeout=10)
-            response.raise_for_status()
-            return support.get_specified_release(repo, api_headers, tag)
-
-        except requests.exceptions.RequestException as e:
-            last_exception = e
-            print(f'\033[93mAttempt {attempt} failed:\033[0m {e}')
-
-    # Final fallback attempt (600s timeout)
-    try:
-        print('Final attempt with extended timeout (600s)')
-        response = requests.get(url, headers=api_headers, timeout=600)
-        response.raise_for_status()
-        return support.get_specified_release(repo, api_headers, tag)
-
-    except requests.exceptions.RequestException as e:
-        print('\033[91mFinal attempt failed too\033[0m')
-        raise last_exception from e
-
-def fetch_latest_release_version(repo, token):
-    api_headers = get_headers(True, token)
-    url = f'https://api.github.com/repos/{repo}/releases/latest'
-
-    # First: 5 fast attempts (5s timeout)
-    for attempt in range(1, 6):
-        try:
-            print(f'GitHub API attempt {attempt}/5 (timeout=10s)')
-            response = requests.get(url, headers=api_headers, timeout=10)
-            response.raise_for_status()
-            return response.json()
-
-        except requests.exceptions.RequestException as e:
-            last_exception = e
-            print(f'\033[93mAttempt {attempt} failed:\033[0m {e}')
-
-    # Final fallback attempt (600s timeout)
-    try:
-        print('Final attempt with extended timeout (600s)')
-        response = requests.get(url, headers=api_headers, timeout=600)
-        response.raise_for_status()
-        return response.json()
-
-    except requests.exceptions.RequestException as e:
-        print('\033[91mFinal attempt failed too\033[0m')
-        raise last_exception from e
-
 def fetch_existing_asset_names(release):
     return [asset['name'] for asset in release['assets']]
 
@@ -75,15 +21,15 @@ def main(token, repo, current_tag, previous_tag):
 
     print('\033[96mFetching the current tag release info...\033[0m')
     if current_tag != "latest":
-        current_release = fetch_specified_release_version(repo, token, current_tag)
+        current_release = support.get_specified_release(repo, get_headers(True, token), current_tag)
     else:
-        current_release = fetch_latest_release_version(repo, token)
+        current_release = support.get_latest_release(repo, get_headers(True, token))
 
     print('\033[96mFetching the current tag release info...\033[0m')
     if previous_tag != "latest":
-        previous_release = fetch_specified_release_version(repo, token, previous_tag)
+        previous_release = support.get_specified_release(repo, get_headers(True, token), current_tag)
     else:
-        previous_release = fetch_latest_release_version(repo, token)
+        previous_release = support.get_latest_release(repo, get_headers(True, token))
 
     current_assets = [package.replace('.7z', '') for package in fetch_existing_asset_names(current_release)]
     previous_assets = [package.replace('.7z', '') for package in fetch_existing_asset_names(previous_release)]

scripts/update_board_changelog.py

@@ -27,36 +27,6 @@ def find_file(root_folder, filename):
             return os.path.join(dirpath, filename), dirpath
     return None, None
 
-def fetch_latest_release_version(repo, token):
-    api_headers = get_headers(True, token)
-    url = f'https://api.github.com/repos/{repo}/releases'
-    response_acquired = False
-
-    # First: 5 fast attempts (10s timeout)
-    for attempt in range(1, 6):
-        try:
-            print(f'GitHub API attempt {attempt}/5 (timeout=10s)')
-            response = requests.get(url, headers=api_headers, timeout=10)
-            response.raise_for_status()
-            response_acquired = True
-            break
-
-        except requests.exceptions.RequestException as e:
-            last_exception = e
-            print(f'\033[93mAttempt {attempt} failed:\033[0m {e}')
-
-    if not response_acquired:
-        # Final fallback attempt (600s timeout)
-        try:
-            print('Final attempt with extended timeout (600s)')
-            response = requests.get(url, headers=api_headers, timeout=600)
-            response.raise_for_status()
-
-        except requests.exceptions.RequestException as e:
-            print('\033[91mFinal attempt failed too\033[0m')
-            raise last_exception from e
-    return support.get_latest_release(repo, api_headers)
-
 def edit_changelog(version):
     found_file, file_dir = find_file(os.path.join(os.getcwd(), "changelog"), "new_hw.md")
 
@@ -107,7 +77,7 @@ def edit_bsp_header(changelog_path):
     parser.add_argument("repo", type=str, help="Repository name, e.g., 'username/repo'")
     args = parser.parse_args()
 
-    release = fetch_latest_release_version(args.repo, args.token)
+    release = support.get_latest_release(args.repo, get_headers(True, args.token))
     changelog_path = edit_changelog(release['tag_name'].replace('mikroSDK-', ''))
 
     # Update bsp header for board/card clock release