a2nr
/
velxio
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge pull request #148 from davidmonterocrespo24/fix-espidf-build-timeout 

fix(espidf): bump ninja timeout 300s → 600s for cold first builds
This commit is contained in:
David Montero Crespo 2026-05-08 23:31:38 -03:00 committed by GitHub
parent b42f815b49 14737eb2db
commit e812371c49
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 9 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
backend/app/services/espidf_compiler.py
View File

@ -986,6 +986,13 @@ class ESPIDFCompiler:
ninja_cmd = ['ninja'] ninja_cmd = ['ninja']
logger.info('[espidf] Building with ninja...') logger.info('[espidf] Building with ninja...')
# Cold ESP-IDF builds with external Arduino libraries (e.g. Adafruit
# BMP280 + BusIO + Unified Sensor → ~1480 build steps) regularly take
# 5-7 minutes on modest hardware. 300s used to cut them off at 98%;
# bump to 600s so first-run cold compiles complete. Subsequent
# builds reuse ninja's cache and finish in seconds.
NINJA_TIMEOUT_S = 600
def _run_ninja(): def _run_ninja():
return subprocess.run( return subprocess.run(
ninja_cmd, ninja_cmd,
@ -993,7 +1000,7 @@ class ESPIDFCompiler:
capture_output=True, capture_output=True,
text=True, text=True,
env=env, env=env,
timeout=300, timeout=NINJA_TIMEOUT_S,
) )
try: try:
@ -1001,7 +1008,7 @@ class ESPIDFCompiler:
except subprocess.TimeoutExpired: except subprocess.TimeoutExpired:
return { return {
'success': False, 'success': False,
'error': 'ESP-IDF build timed out (300s)', 'error': f'ESP-IDF build timed out ({NINJA_TIMEOUT_S}s)',
'stdout': '', 'stdout': '',
'stderr': '', 'stderr': '',
} }