Fix using separate timeout for remote rule (#235)

JIRA: RHELWF-10541
release-engineering · Feb 20, 2024 · 75c86a4 · 75c86a4
1 parent 670802f
commit 75c86a4
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/greenwave/config.py b/greenwave/config.py
@@ -36,7 +36,6 @@ class Config(object):
         '*': 'https://src.fedoraproject.org/{pkg_namespace}{pkg_name}/raw/{rev}/f/gating.yaml'
     }
     REMOTE_RULE_GIT_TIMEOUT = 30
-    REMOTE_RULE_GIT_MAX_RETRY = 3
     KOJI_BASE_URL = 'https://koji.fedoraproject.org/kojihub'
     # Options for outbound HTTP requests made by python-requests
     REQUESTS_TIMEOUT = (6.1, 15)

diff --git a/greenwave/resources.py b/greenwave/resources.py
@@ -305,7 +305,8 @@ def retrieve_scm_from_koji_build(nvr: str, source: str, koji_url: str):
 @cached
 def retrieve_yaml_remote_rule(url: str):
     """ Retrieve a remote rule file content from the git web UI. """
-    response = requests_session.request('GET', url)
+    timeout = current_app.config['REMOTE_RULE_GIT_TIMEOUT']
+    response = requests_session.get(url, timeout=timeout)
 
     if response.status_code == 404:
         log.debug('Remote rule not found: %s', url)