Skip to content
Snippets Groups Projects
Verified Commit 0ccdca31 authored by Karel van Klink's avatar Karel van Klink :smiley_cat:
Browse files

move api timeout to a static variable in config.py

parent 7a652eda
No related branches found
No related tags found
1 merge request!51Feature/update callback
Pipeline #84369 passed
...@@ -19,6 +19,7 @@ CONFIG_SCHEMA = { ...@@ -19,6 +19,7 @@ CONFIG_SCHEMA = {
"required": ["ansible_playbooks_root_dir"], "required": ["ansible_playbooks_root_dir"],
"additionalProperties": False, "additionalProperties": False,
} }
DEFAULT_REQUEST_TIMEOUT = 10
class Config(BaseModel): class Config(BaseModel):
......
...@@ -12,6 +12,8 @@ import xmltodict ...@@ -12,6 +12,8 @@ import xmltodict
from dictdiffer import diff from dictdiffer import diff
from pydantic import BaseModel, HttpUrl from pydantic import BaseModel, HttpUrl
from lso.config import DEFAULT_REQUEST_TIMEOUT
logger = logging.getLogger(__name__) logger = logging.getLogger(__name__)
...@@ -153,7 +155,7 @@ def _run_playbook_proc(job_id: str, playbook_path: str, extra_vars: dict, invent ...@@ -153,7 +155,7 @@ def _run_playbook_proc(job_id: str, playbook_path: str, extra_vars: dict, invent
"return_code": int(ansible_playbook_run.rc), "return_code": int(ansible_playbook_run.rc),
} }
request_result = requests.post(callback, json=payload, timeout=10000) request_result = requests.post(callback, json=payload, timeout=DEFAULT_REQUEST_TIMEOUT)
assert request_result.status_code == 200 assert request_result.status_code == 200
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment